在C#中的父类上正确实现IDisposable

Question

我有一个实现C#的类,SerialPort它曾经是这样的:

public class AsyncSerial : IDisposable
{
    SerialPort newPort; //Parameters declared in my constructor
    //Constructor and other methods

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

    protected virtual void Dispose(bool disposing)
    {
        if(disposing)
        {
            this.Close();
            this.Dispose();
        }
    }
}

这在代码分析中不会引发任何警告(我从MSDN获取代码作为如何正确执行此操作的示例).

因为我只是要宣布一个SerialPort我认为我会让我的班级成为孩子SerialPort,但现在我得到警告,我似乎无法解决.

public class AsyncSerial : SerialPort
{
    //Constructor and other methods

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

    protected new virtual void Dispose(bool disposing)
    {
        if(disposing)
        {
            this.Close();
            this.Dispose();
        }
    }
}

代码警告说处理方法应该是new隐藏成员,我做了,但我也得到:

"警告CA1063确保'AsyncSerial.Dispose()'被声明为公共密封"

使其密封意味着它必须被标记override(或者我得到编译器错误),使它覆盖意味着它可以是新的,所以我最终得到:

错误CS0506'AsyncSerial.Dispose()':无法覆盖继承的成员'Component.Dispose()',因为它未标记为虚拟,抽象或覆盖

我不知道IDisposable在父类中实现处理类的"正确"方法.我发现的每个例子都只适合IDisposable作为基础,但是我的课程

public class AsyncSerial : SerialPort, IDisposable
{
    //code
}

给我一个代码分析警告因为SerialPort已经实现了IDisposable.

我是否应该禁止关于确保'AsyncSerial.Dispose()'被声明为公共和密封的警告,或者是否有正确的方法来执行此操作而不提供代码分析警告.

Answer 1

你的子类应该覆盖Dispose(bool disposing)任何东西 - 这就是拥有该方法的全部意义,真的.

但是,我怀疑基类无论如何都会进行正确的调用,所以你不需要做任何事情,除非你有额外的资源要发布,而不是发布的Close().如果是这种情况,请执行以下操作Dispose(bool disposing):

protected override void Dispose(bool disposing)
{
    // Allow the base class to release resources
    base.Dispose(disposing);
    // Release any extra resources here 
}

请注意,当前两个Dispose重载相互调用时,您当前的实现将导致StackOverflowException .