处置实现 IDisposable 的成员

Question

在我的 Dispose 方法（如下所示）中，每次我想调用 someObj.Dispose() 时，我也会检查 someObj!=null。

那是因为我的设计不好吗？ 他们是否是一种更简洁的方法来确定调用对象中使用的所有成员（实现 IDisposable）的 Dispose 而不存在 NullReference 异常的风险？

protected void Dispose(bool disposing)
        {
            if (disposing)
            {
               if (_splitTradePopupManager != null)
                {
                    _splitTradePopupManager.Dispose();
                }
             }
        }

感谢您的关注。

Answer 1

我喜欢@Dan Tao的解决方案，但它作为扩展方法要好得多，imo：

public static void SafeDispose(this IDisposable obj)
{
    if (obj != null)
        obj.Dispose();
}

现在您可以在程序中的任何IDisposable 上调用member.SafeDispose() 而无需担心。 :)

Answer 2

也许其他人可以插话，但我个人认为这不是设计缺陷——这是最安全的做法。

也就是说，没有什么能阻止您以方便的方法包装您的 null 支票和 Dispose 电话：

private void DisposeMember(IDisposable member)
{
    if (member != null)
        member.Dispose();
}

那么你的Dispose 方法可能看起来更干净一些：

protected void Dispose(bool disposing)
{
    if (disposing)
    {
        DisposeMember(_splitTradePopupManager);
        DisposeMember(_disposableMember2);
        DisposeMember(_disposableMember3);
    }
}

另外，这还解决了原始代码中的潜在竞争条件。如果在多线程上下文中运行，if (_field != null) _field.Dispose() 模式可能会在检查和处理之间将_field 设置为null 时导致NullReferenceException（很少见，但可能）。将_field 作为参数传递给诸如DisposeMember 之类的方法会复制对方法中局部变量的引用，从而消除了这种可能性，尽管这种可能性不大。

Answer 3

只有你知道这个问题的答案！

如果没有看到你的整个班级，其他人很难判断这些成员是否有可能在调用Dispose 时为空。

（当然，作为一般规则，引用类型或可空值类型总是可能为空，因此始终包含这些空检查可能是一种好习惯。）

Answer 4

我能想到的唯一其他选择是创建一个DisposeParameter 辅助方法，它有一个对象作为它的参数，并且只检查它是否为空，否则处置它。这样你只需要一行代码来处理它，但我不确定它是否会使其更具可读性。

Answer 5

试试这个。

    protected void Dispose(bool disposing) 
    { 
        if (disposing) 
        {
           //for all members.. 
           if (null != member && member is IDisposible) 
            { 
                member.Dispose(); 
            } 
         } 
    }