IDisposable::Dispose() 是否应该是虚拟的

Question

说 SomeDisposable 的工厂实际上是在创建/返回一种看门狗 Wrapper

public class Wrapper : SomeDisposable
{
    public new /*:(*/ Dispose() { ... };
}

调用者使用like

using (SomeDisposable sd = SomeDisposableFactory.Create(...))
{
} // Wrapper.Dispose() never called.

Wrapper.Dispose() 永远不会被调用。如果Dispose() 是虚拟的，那么Wrapper.Dispose() 将被调用。

IDisposable 接口不保证其他最佳实践方法virtual Dispose(bool) 确实存在或强制其是虚拟的，因此通常不能依赖它存在（这只是推荐的模式）。接口目前不允许对虚拟进行约束。

不将推荐的Dispose() 模式设为虚拟有哪些优点和缺点，这可以解决这个特殊的困境。 C# 是否应该允许通过接口强制使用虚拟方法（因为抽象类不像契约定义那样流行）。

Answer 1

没有。该模式实际上表示Dispose()（非虚拟）应该调用protected virtual void Dispose(bool) 方法。这保证了基类Dispose 调用可以正确地向上传递层次结构。

IDisposable 的文档中有详细说明：

• 它应该提供一个公共的、非虚拟的 Dispose() 方法和一个受保护的虚拟 Dispose(Boolean disposing) 方法。
• Dispose() 方法必须调用 Dispose(true) 并且应该禁止完成以提高性能。
• 基本类型不应包含任何终结器。

Answer 2

这已经解决了。

非sealed的一次性类型应该使用通用的dispose pattern：

public class DisposableResourceHolder : IDisposable
{
    private SafeHandle resource; // handle to a resource

    public DisposableResourceHolder()
    {
        this.resource = ... // allocates the resource
    }

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

    protected virtual void Dispose(bool disposing)
    {
        if (disposing)
        {
            // dispose managed resources.
            if (resource != null) resource.Dispose();
        }

        // free unmanaged resources.
    }
}