如何处理通过 Next 属性迭代的对象？

Question

我有一个对象，它使用一些底层本机资源，并且有一个指向下一个实例的指针，我对其进行迭代，类似于：

MyObject begin = null;

try
{
    begin = GetFirst();

    while (begin != null)
    {
        MyObject next = begin.Next();
        // do something with begin
        begin.Dispose();
        begin = next;
    }
}
finally
{    
    if (begin != null)
    {
        begin.Dispose();
    }
}

我遇到代码分析问题：

CA2202：Microsoft.Usage：对象“begin”可以在方法“x()”中多次处理。为了避免生成 System.ObjectDisposeException，您不应在一个对象上多次调用 Dispose。

知道如何在不抑制它的情况下摆脱这个错误吗？

Answer 1

显然，您有某种机制来识别链中的第一项，也许是其他一些对象或存储第一项的静态变量？

在最初调用 dispose 的代码中怎么样：

GetFirst().Dispose();

那么 dispose 方法的唯一责任是处置当前项及其子项：

public void Dispose()
{
    if (Next() != null)
    {
        Next().Dispose();
    }
}

这消除了在 dispose 方法中循环的任何需要。我还会看一下 处置模式

Answer 2

代码分析似乎认为在 Dispose() 方法期间可能会发生异常。如果是这种情况，您将输入带有对 begin 的已处置但非空引用的finally 块。

请注意，如果您计划将对 begin.Dispose() 的调用包装在额外的错误捕获和处理中，我只会比 @Dan 更喜欢这种方法。 IMO，丹的解决方案更加优雅。

这是消除警告的 try-finally 方法：

MyObject begin = GetFirst();
MyObject next = null;

while (begin != null)
{
    try
    {
        next = begin.Next();
        // do something with begin
    }
    finally
    {
        begin.Dispose();
        begin = next;
    }
}

Answer 3

在我看来，你的最后一段代码确实是不必要的。如果 begin != null，那么你的 while 循环应该继续，不是吗？

更新：看起来您正在尝试确保在引发异常的情况下处理最后获得的 begin 值。试试这个：

MyObject begin = GetFirst();

while (begin != null)
{
    MyObject next;
    using (begin)
    {
        next = begin.Next();
        // do something with begin
    }

    begin = next;
}

请注意，在上面的建议中，实际上仍然可能会出现未处理的对象：在 using 结束之前分配给 next 的最后一个值> 阻止。您原来的问题中没有涵盖这种情况，因此我没有在上述建议中解决它。不过，如果这是一个潜在的问题，那就值得考虑。