通过引用传递 IDisposable 对象会导致错误？

Question

我正在尝试创建一个通用方法来处理实现IDisposable 的对象，称为DisposeObject()

为了确保我正在处理原始引用指向的对象，我试图通过引用传递一个对象。

但我收到一个编译错误提示

“ref”参数类型与参数类型不匹配

在下面的（简化的）代码中，_Baz 和 _Bar 都实现了 IDisposable。

所以问题是，

1. 为什么会出现此错误？
2. 有没有办法绕过它？

[更新] 从到目前为止提供的答案来看，只要我不将 IDisposable 参数设置为 null，我就可以简单地按值传递对象，而无需使用 ref。 我现在遇到了另一个麻烦，是否在DisposeObject 方法中将一次性对象设置为null。

这是完整的完整来源：

public class Foo : IDisposable
{
    private Bar _Bar;
    private Baz _Baz;
    private bool _IsDisposed;

    ~Foo() { Dispose(false); }

    public void Dispose(bool disposing)
    {
        if (!_IsDisposed)
        {
            if (disposing)
            {
                DisposeObject(ref _Baz);
                DisposeObject(ref _Bar);
            }
        }

        _IsDisposed = true;
    }

    private void DisposeObject(ref IDisposable obj)
    {
        try
        {
            if (obj == null) 
                return;
            obj.Dispose();
            obj = null;
        } catch (ObjectDisposedException) { /* Already Disposed... */ }
    }

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }
}

public class Bar : IDisposable
{
    public void Dispose() {}
}

public class Baz : IDisposable
{
    public void Dispose() {}
}

[结果]
我删除了在DisposeObject 中将参数设置为 null (obj = null;) 的代码 所以最终的代码就变成了。

    public void Dispose(bool disposing)
    {
        if (!_IsDisposed)
        {
            if (disposing)
            {
                DisposeObject(_Baz);
                DisposeObject(_Bar);
            }
        }

        _IsDisposed = true;
    }

    private void DisposeObject(IDisposable obj)
    {
        try
        {
            if (obj == null) 
                return;
            obj.Dispose();
        } catch (ObjectDisposedException) { /* Already Disposed... */ }
    }

Answer 1

这是您示例的一个选项（目前无法针对编译器进行验证，但您会明白的）：

private void DisposeObject<T>(ref T obj) where T : IDisposable
{
    // same implementation
}

要调用它，请使用

DisposeObject<Baz>(ref _Baz);
DisposeObject<Bar>(ref _Bar);

正如在其他 cmets 中所指出的，您得到的编译器错误有其自身的目的（阻止您在方法中分配其他类型的 IDisposable，从而导致不一致的状态）。

Answer 2

您无需通过引用传递，因为您传递的是引用类型。您应该从方法定义中删除 ref 关键字。这样做你应该不会有任何问题，尽管我不确定这比简单地调用Dispose() 更有效或更清晰（除了你不必为显式实现而强制转换它的事实之外） null 为您检查）。

编辑

舞蹈，虽然我希望围绕这个主题的讨论对你有所帮助，但你的初衷似乎并不可行。为了传递ref 的内容，您不能传递类型不同于ref 参数所期望的类型的变量（换句话说，您不能传递声明为class 或其他类型的变量） interface，如果 ref 参数是 IDisposable，则实现 IDisposable。因为ref 参数允许赋值传播回调用者，所以您可能会允许将不兼容的类型存储在您的变量中。

如果你想要的话，最好的办法是自己分配null。如果您想将 null 检查并忽略异常封装到函数中，那很好，但不幸的是，ref 在这种情况下无论您如何切片都不适合您。

Answer 3

试试这个：

IDisposable d = (IDisposable)_Baz;
DisposeObject(ref d);

编辑： 正如亚当指出的那样，您的代码不需要将其作为参考。对象总是作为引用传递。

Answer 4

这种方法听起来很有趣，但我暂时忽略它。

要解决您的问题，您需要使用“（IDisposable）”转换您传递的对象

我承认编译器和 Jon Skeet 的意愿。为此，您需要一个实际对象：

IDisposable _BazD = (IDisposable)_Baz;
DisposeObject(ref _BazD);

除了 try/catch 之外，我还会在 DisposeObject() 中添加一个空检查。与昂贵的异常捕获相比，“obj==null”将是一个快速而简单的检查，如果这会为同一个对象多次命中。嗯......一分钟前有吗？没关系。

Answer 5

谢谢 Dan C。我还没有足够的代表来添加 cmets，所以我必须添加这个作为答案。但是，此解决方案完全归功于 Dan C。

这是工作代码：

public override void Dispose()
{
    base.Dispose();

    DisposeOf<UserTableAdapter>(ref userAdapter);
    DisposeOf<ProductsTableAdapter>(ref productsAdapter);

    if (connection != null)
    {
        if (connection.State == ConnectionState.Open)
        {
            connection.Close();
        }
        DisposeOf<SqlConnection>(ref connection);
    }
}

private void DisposeOf<T>(ref T objectToDispose) where T : IDisposable
{
    if (objectToDispose != null)
    {
        objectToDispose.Dispose();
        objectToDispose = default(T);
    }
}