你如何“正确”地做到这一点？当您的实现是空方法时实施 Dispose() （根据 FxCop）？ (CA1063)

Question

我有一个接口的实现，并且该接口扩展了 IDisposable。在我的接口的特定实现中，我不需要释放任何东西，所以我只有一个空的 Dispose() 方法。

public interface IMyStuff : IDisposable
{
}

public MyStuffImpl : IMyStuff
{
    public void Dispose()
    {
    }
}

现在在 FxCop 中，这会导致 CA1063：

Error, Certainty 95, for ImplementIDisposableCorrectly
{
    Resolution   : "Provide an overridable implementation of Dispose(
                   bool) on 'MyStuffImpl' or mark the type as sealed. 
                   A call to Dispose(false) should only clean up native 
                   resources. A call to Dispose(true) should clean up 
                   both managed and native resources."
}
CriticalWarning, Certainty 75, for CallGCSuppressFinalizeCorrectly
{
    Resolution   : "Change 'MyStuffImpl.Dispose()' to call 'GC.SuppressFinalize(
                   object)'. This will prevent derived types that introduce 
                   a finalizer from needing to re-implement 'IDisposable' 
                   to call it."
}
Error, Certainty 95, for ImplementIDisposableCorrectly
{
    Resolution   : "Modify 'MyStuffImpl.Dispose()' so that it 
                   calls Dispose(true), then calls GC.SuppressFinalize 
                   on the current object instance ('this' or 'Me' in Visual 
                   Basic), and then returns."
}

因此，看起来我可以通过以下两种方式之一解决此问题：

---

使类密封：

public sealed MyStuffImpl : IMyStuff
{
    public void Dispose()
    {
    }
}

---

实现典型模式的一部分：

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

    private void Dispose(bool disposing)
    {
    }
}

---

就我而言，我不这样做我不打算扩展这个实现，所以我可能会通过将其密封来解决它，但我承认我并不真正理解为什么密封与否很重要。

另外，仅仅因为我的类是密封的，FxCop 不再告诉我 Dispose() 应该调用 GC.SupressFinalize(this); 但这真的是这样吗？在 .NET 中，总是在 Dispose 中调用 SupressFinalize 是否“更好”？

Answer 1

除非您的实例具有终结器，否则 SuppressFinalize() 毫无意义。
如果您的类没有终结器，但未sealed，您仍应SuppressFinalize，以防继承类添加终结器。

您的两个选项都是正确的，除了 Dispose(bool) 需要protected virtual。

Answer 2

在“实现典型模式的一部分”选项中，您应该使您的 Dispose(bool) 方法 protected virtual：

protected virtual void Dispose(bool disposing) 
{ 
} 

这将为子类提供处理任何资源的机会他们管理。这就是“提供 Dispose(bool) 的可重写实现”中“可重写”的含义。

当然，public virtual 也能满足 FxCop。