Coverity、Enumerable.Where(this ...) 和 IDisposable

Question

因此，代码分析告诉我 Enumarble.Where(this ...) 正在返回 WhereListIterator 的实例，它（看起来是）内部类型在实现 IDisposable 的 .NET 框架内。

Coverity 不喜欢 IDisposable 未被处置，因此建议我处置所述实例。显然，如果不进行一些类型检查，我就无法处置该实例，因为 Enumerable.Where(this ...) 据说返回 IEnumerable，这确实不是来自 IDisposable 的 ihnerit。

我的问题是：.NET 是否期望我处理 WhereListIterator，或者迭代器自行处理（例如，在每次枚举之后）。如果我不希望处置它，那么为什么要实现该接口？这引出了第三个稍微不相关的问题：如果显式实现了 IDisposable，Coverity（代码分析）是否仍然认为我应该处置它？

代码示例：

var myList = new List<int>{ 1, 2, 3, 4 };

var evenNumbers = myList.Where(x => x % 2 == 0);

foreach(var number in evenNumbers)
{
    Console.WriteLine(number);
}

if(evenNumbers is IDisposable)
{
    ((IDisposable)evenNumbers).Dispose(); // This line will be executed
}

Answer 1

不，您不需要自己处理它。请注意，您无需使用 LINQ 即可演示此类事情。在这种情况下，我相信 WhereListIterator 实际上是一个手写的类，但迭代器块显示了类似的东西：

using System;
using System.Collections.Generic;

public class Program
{
    static void Main(string[] args)
    {
        var empty = Empty();
        Console.WriteLine(empty is IDisposable); // Prints True
    }

    static IEnumerable<string> Empty()
    {
        yield break;
    }
}

它确实实现了 IDisposable 因为它不仅实现了 IEnumerable 而且 IEnumerator 作为一种优化 - 可迭代也充当迭代器，在常见情况下，您只需要迭代一次。 foreach 循环将隐式处置 IEnumerator，除非您进行迭代，否则您不需要处置它。

基本上，你在这里没问题 - 尽管遗憾的是 Coverity 正在警告你这一点。 （老实说，我自己没有使用过 Coverity - 我不知道是否可以做一些事情来调整它的行为。）

Answer 2

如果您不使用 foreach 循环并使用旧的迭代方式

var v = new List<int>() { 1,2,3};
var enumerator = v.GetEnumerator();
while (enumerator.MoveNext())
{

    Console.WriteLine(enumerator.Current);
}

，那么您应该调用 Dispose 方法