c# - 私有(private)无效处置( bool )？

Question

在一些地方，人们建议对 IDisposable 模式使用 private void Dispose(bool)。这似乎已经过时了(至少对于未密封的类而言)，因为新的建议模式(根据 Microsoft 的说法)是 protected virtual void Dispose(bool)。

问题是，代码分析不会报告 private void Dispose(bool) 违反 CA1063 , 尽管它似乎直接违反了模式。

这是怎么回事？ private void Dispose(bool) 是否以某种方式被调用(或编译成看起来像 protected virtual Dispose(bool) 的东西？

如果这是代码分析的某个问题并且是不正确的模式，有没有办法检测到这一点？可能与 StyleCop 一起使用？

编辑:经过思考，是不是基类可以调用base.Dispose() 会命中private void Dispose(bool)？即使它无法传递参数？

编辑:示例

public class A : IDisposable
{
    ~A()
    {
        this.Dispose(false);
    }

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

    private void Dispose(bool disposing) // Should be protected virtual void Dispose(bool)
    {
        Console.WriteLine("A");
    }
}

public class B : A
{
    protected virtual void Dispose(bool disposing) // Proper pattern.
    {
        Console.WriteLine("B");
    }
}

public static class Program
{
    static void Main(string[] args)
    {
        A a = new A();
        a.Dispose(); // Prints "A"

        B b = new B();
        b.Dispose(); // Prints "A"!
    }
}

正如您从中看到的，它使处置模式的使用变得非常笨拙。

您可以通过隐藏 public void Dispose(void) 然后在某处调用 base.Dispose() 来稍微解决这个问题。这与调用 B b = new B(); 时的正确处理模式“相似”。 b.dispose(); except 当调用 A b = new B(); b.Dispose();，它只调用A的Dispose方法。

public class B : A
{
    public void Dispose() // Causes CA error with or without "new".
    {
        this.Dispose(true);
        GC.SuppressFinalize(this);
    }

    protected virtual void Dispose(bool disposing) // Proper pattern.
    {
        base.Dispose(); // Writes "A" (without quotes).
        Console.WriteLine("B");
    }
}

基本上，这整件事看起来很糟糕。我们是否知道 CA 接受 private void Dispose(bool) 是否是一个错误，是否有办法至少用 StyleCop 发出警告？

编辑:我认为我不应该接受 Alexandre 的回答，因为相对于我提出的问题，它基本上可以归结为“可能是一个错误”，以及一些应该作为评论的内容。如果其他人有更确凿的结论，我认为那将是一个更合适的答案。

Answer 1

Implementing a Dispose Method

The IDisposable interface requires the implementation of a single parameterless method, Dispose. However, the dispose pattern requires two Dispose methods to be implemented:

• A public non-virtual (NonInheritable in Visual Basic) IDisposable.Dispose implementation that has no parameters.
• A protected virtual (Overridable in Visual Basic) Dispose method.

Because the public, non-virtual (NonInheritable in Visual Basic), parameterless Dispose method is called by a consumer of the type, its purpose is to free unmanaged resources and to indicate that the finalizer, if one is present, doesn't have to run. Because of this, it has a standard implementation:

public void Dispose()
{
   // Dispose of unmanaged resources.
   Dispose (true);
   // Suppress finalization.
   GC.SuppressFinalize (this);
}

In the second overload, the disposing parameter is a Boolean that indicates whether the method call comes from a Dispose method (its value is true) or from a finalizer (its value is false).

当垃圾收集器决定不再需要您的对象时，它会尝试最终确定它，以防您忘记调用无参数 dispose 方法，因为如果您这样做并且您遵循该模式，调用将被抑制.

参见:How Finalization Works

私有(private)与 protected 虚拟:

如果您想支持正确遵循该模式的子类，您应该始终按照文档所述使用 protected 虚拟。

为什么有人用私有(private)版？也许是因为继承从来都不是他们的意图，特别是如果您只是使用 Resharper 等工具即时生成方法，那么大多数时候这些方法都是私有(private)的。

为什么代码分析没有报告问题？

可能是一个错误。提供一个给出问题的小样本，以便其他人可以在他们的机器上进行测试。