13

我正在使用 dotCover 来分析我的单元测试的代码覆盖率，我得到了一些奇怪的结果......我有一个覆盖不完整的迭代器方法，但没有覆盖的语句只是右括号在方法结束时。

这是我正在测试的方法：

    public static IEnumerable<T> CommonPrefix<T>(
        this IEnumerable<T> source,
        IEnumerable<T> other,
        IEqualityComparer<T> comparer)
    {
        source.CheckArgumentNull("source");
        other.CheckArgumentNull("other");

        return source.CommonPrefixImpl(other, comparer);
    }

    private static IEnumerable<T> CommonPrefixImpl<T>(
        this IEnumerable<T> source,
        IEnumerable<T> other,
        IEqualityComparer<T> comparer)
    {
        comparer = comparer ?? EqualityComparer<T>.Default;

        using (IEnumerator<T> en1 = source.GetEnumerator(),
                              en2 = other.GetEnumerator())
        {
            while (en1.MoveNext() && en2.MoveNext())
            {
                if (comparer.Equals(en1.Current, en2.Current))
                    yield return en1.Current;
                else
                    yield break;
            }
        } // not covered
    } // not covered

单元测试：

    [Test]
    public void Test_CommonPrefix_SpecificComparer()
    {
        var first = new[] { "Foo", "Bar", "Baz", "Titi", "Tata", "Toto" };
        var second = new[] { "FOO", "bAR", "baz", "tata", "Toto" };

        var expected = new[] { "Foo", "Bar", "Baz" };
        var actual = first.CommonPrefix(second, StringComparer.CurrentCultureIgnoreCase);
        Assert.That(actual, Is.EquivalentTo(expected));
    }

和覆盖结果：

覆盖结果

我假设块的右大括号using实际上是Dispose对枚举器的调用；但是，为什么不执行呢？我首先怀疑 NUnit 没有处理枚举器，但如果我在actual.

至于第二个未覆盖的右大括号，我不知道它代表什么......我想这与编译器如何转换迭代器块有关。

谁能阐明这两个“声明”是什么，以及为什么不执行它们？

编辑：Peter 提出了一个非常好的问题：上面显示的结果是在debug build上运行测试时获得的。如果我在release build上运行测试，则该方法的覆盖率CommonPrefixImpl为 100%，因此它可能与编译器优化有关。

4

2 回答 2

12

迭代器方法的问题之一是编译器会生成一个相当大且复杂的状态机来管理迭代器方法中代码的延迟执行。这通常会生成一两个类。这些类旨在处理一般情况而不是您的特定情况，因此其中可能至少有一些从未使用过的代码。您可以通过使用 ILSpy、JustDecompile 或 Reflector 等工具查看程序集来查看生成的内容。它将显示由 C# 编译器生成的程序集中的类（通常包含“<”等的类名）

分析器所知道的是 PDB 如何与您的代码相关联，尽管您编写的所有代码都可能被执行，但仍有可能并非编译器生成的所有代码都被执行。分析器可能不知道这一点，只是说执行了特定百分比（小于 100）的特定迭代器方法。

可能生成的其中一件事是异常处理代码。因为编译器不知道您的代码不会或可能无法生成异常，它仍然会生成代码来补偿异常——它需要防止其状态被破坏。我敢打赌，如果您在迭代器方法中基于某个标志包含在不同位置抛出异常的方法并运行该方法两次（一次没有异常，一次在同一运行中出现异常），那么百分比会不同——可能更高因为生成的异常处理代码将被执行。

方法的结尾“似乎”没有被执行的事实可能是因为该代码是状态机中被执行的不同方法的一部分，并且编译器永远不会生成从生成的代码到您的类中的代码的关联.

更新：为了更好地理解编译器正在做什么并查看它生成的代码类型的示例，请参阅C# 规范中的第10.14 节迭代器（ http://www.microsoft.com/en-us/download/details .aspx?id=7029 )

于 2012-08-15T02:14:36.557 回答
0

除了您的问题和详细答案之外，我还有以下行为。

    // less than 100% coverage
    public static IEnumerable<T> ForEachYieldDo<T>(this IEnumerable<T> source, Action<T> action)
    {
        foreach (var x in source)
        {
            action(x);
            yield return x;
        }
    }

    // 100% code coverage
    public static IEnumerable<T> ForEachSelectDo<T>(this IEnumerable<T> source, Action<T> action)
    {
        return source.Select(x =>
        {
            action(x);
            return x;
        });
    }

这两个函数具有相同的行为。仅在处理项目时才执行该操作。如果停止检索项目，则不执行操作。

于 2019-09-22T14:32:14.770 回答