C# 抽象 Dispose 方法

Question

我有一个实现 IDisposable 的抽象类，如下所示：

public abstract class ConnectionAccessor : IDisposable
{
    public abstract void Dispose();
}

在 Visual Studio 2008 Team System 中，我对我的项目运行代码分析，出现的警告之一如下：

Microsoft.Design ：修改“ConnectionAccessor.Dispose()”，以便它调用 Dispose(true)，然后对当前对象实例（Visual Basic 中的“this”或“Me”）调用 GC.SuppressFinalize，然后返回.

它只是愚蠢地告诉我修改抽象方法的主体，还是应该在 Dispose 的任何派生实例中做进一步的事情？

Answer 1

您应该遵循实现 Dispose 的传统模式。将 Dispose() 设为虚拟被认为是不好的做法，因为传统模式强调在“托管清理”中重用代码（API 客户端直接或通过 调用 Dispose()使用）和“非托管清理”（GC 调用终结器）。提醒一下，模式是这样的：

public class Base
{
    ~Base()
    {
        Dispose(false);
    }

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this); // so that Dispose(false) isn't called later
    }

    protected virtual void Dispose(bool disposing)
    {
        if (disposing)
        {
             // Dispose all owned managed objects
        }

        // Release unmanaged resources
    }
}

这里的关键是，用于非托管清理的终结器和 Dispose 之间没有重复，但任何派生类都可以扩展托管和非托管清理。

对于您的情况，您应该做的是：

protected abstract void Dispose(bool disposing)

并将其他一切保留原样。即使这也具有可疑的价值，因为您现在强制派生类实现 Dispose，并且您如何知道所有派生类都需要它？如果您的基类没有什么可处理的，但大多数派生类可能会这样做（也许有一些例外），那么只需提供一个空的实现即可。这就是 System.IO.Stream（本身是抽象的）所做的事情，因此有先例。

Answer 2

该警告基本上告诉您实施Dispose 模式 在你的班级里。

结果代码应如下所示：

public abstract class ConnectionAccessor : IDisposable
{
    ~ConnectionAccessor()
    {
        Dispose(false);
    }

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

    protected virtual void Dispose(bool disposing)
    {
    }
}

Answer 3

虽然这看起来确实有点挑剔，但这个建议是有效的。您已经表明您希望 ConnectionAccessor 的任何子类型都将拥有需要处置的某些内容。因此，确保基类完成正确的清理（就 GC.SuppressFinalize 调用而言）似乎比依赖每个子类型来完成似乎更好。

我使用 Bruce Wagners 书中提到的处置模式 Effective C# 基本上是：

public class BaseClass : IDisposable
{
    private bool _disposed = false;
    ~BaseClass()
    {
        Dispose(false);
    }

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

    protected virtual void Dispose(bool disposing)
    {
        if (_disposed)
            return;

        if (disposing)
        {
            //release managed resources
        }

        //release unmanaged resources

        _disposed = true;
    }
}

public void Derived : BaseClass
{
    private bool _disposed = false;

    protected override void Dispose(bool disposing)
    {
        if (_disposed) 
            return;

        if (disposing)
        {
            //release managed resources
        }

        //release unmanaged resources

        base.Dispose(disposing);
        _disposed = true;
    }

Answer 4

我对迄今为止提供的答案的唯一抱怨是，他们都假设您需要有一个终结器，但事实并非如此。与终结相关的性能开销相当大，如果没有必要，我不想将其强加于我的所有派生类。

请参阅 Joe 的此博文 Duffy，它解释了您何时可能需要或不需要终结器，以及在任何情况下如何正确实现 Dispose 模式。
总结 Joe 的博客文章，除非您正在做一些相当低级的处理非托管内存的事情，否则您不应该实现终结器。作为一般经验法则，如果您的类仅保存对本身实现 IDisposable 的托管类型的引用，则不需要终结器（但应该实现 IDisposable 并处置这些资源）。如果您直接从代码分配非托管资源（PInvoke？）并且必须释放这些资源，那么您需要一个。如果派生类确实需要，则始终可以添加终结器，但是通过将其放入基类来强制所有派生类具有终结器会导致所有派生类受到可终结对象的性能影响，而这种开销可能不会增加。必要的。

Answer 5

不过这个警告很有趣。 C# 设计师之一 Eric Lippert 在博客中阐述了为什么错误消息应该是“诊断性而非规范性：描述问题，而不是解决方案”。
在此处阅读.