CA2000 将对象引用传递给 C# 中的基本构造函数

Question

当我通过 Visual Studio 的代码分析实用程序运行一些代码时收到一条警告，我不确定如何解决。也许这里有人遇到过类似的问题，解决了它，并愿意分享他们的见解。

我正在编写 DataGridView 控件中使用的自定义绘制单元格。代码类似于：

public class DataGridViewMyCustomColumn : DataGridViewColumn
{
    public DataGridViewMyCustomColumn() : base(new DataGridViewMyCustomCell())
    {
    }

它会生成以下警告：

CA2000 : Microsoft.Reliability : 在方法“DataGridViewMyCustomColumn.DataGridViewMyCustomColumn()”中调用 System.IDisposable.Dispose 在对象“new DataGridViewMyCustomCell()”上的所有引用都超出范围。

我知道它在警告我 DataGridViewMyCustomCell（或它继承自的类）实现了 IDisposable 接口，并且应该调用 Dispose() 方法来清理 DataGridViewMyCustomCell 不再声明的任何资源。

我在 Internet 上看到的示例建议使用 using 块来确定对象的生命周期并让系统自动处理它，但是当移入构造函数的主体时无法识别 base，所以我不能在它周围写一个 using 块......我不确定我是否想要这样做，因为这不会指示运行时释放仍然可以稍后在基类中使用的对象吗？

那么我的问题是，代码还可以吗？或者，如何重构它来解决警告？除非确实合适，否则我不想抑制警告。

Answer 1

如果您使用的是 Visual Studio 2010，那么 CA2000 将完全损坏。它也可能在其他版本的 FxCop（又名代码分析）中被破坏，但 VS2010 是我唯一可以保证的。我们的代码库正在为这样的代码发出 CA2000 警告...

internal static class ConnectionManager 
{
    public static SqlConnection CreateConnection()
    {
         return new SqlConnection("our connection string");
    }
}

...表明连接在超出方法范围之前没有被释放。嗯，是的，没错，但它并没有超出应用程序的范围，因为它返回给调用者——这就是方法的重点！同样，您的构造函数参数并没有超出范围，而是被传递给基类，因此这是规则的误报，而不是实际问题。

这曾经是一个有用的规则，但现在你真正能做的就是关闭它，直到他们修复它。这是不幸的，因为（极少数）实际的积极因素是应该修复的。

Answer 2

没有安全和优雅的方式让链式构造函数将新的IDisposable 对象传递给基本构造函数，因为您注意到不可能将链式构造函数调用包装在任何类型的try finally 块中。有一种安全的方法，但它并不优雅：定义一个实用方法，例如：

internal static TV storeAndReturn<TR,TV>(ref TR dest, TV value) where TV:TR
{ 
  dest = value; return value;
}

让构造函数看起来像：

protected DataGridViewMyCustomColumn(ref IDisposable cleaner) : 
   base(storeAndReturn(ref cleaner, new DataGridViewMyCustomCell()))
{
}

需要一个新对象的代码必须调用一个公共静态工厂方法，该方法将在 try/finally 块中调用适当的构造函数，其主线将在完成之前将 cleaner 清空，并且其finally 块将在cleaner 上调用Dispose，如果它不为空。如果每个子类都定义了一个类似的工厂方法，那么这种方法将确保新的IDisposable 对象将被释放，即使在它创建和封装对象暴露给客户端代码之间发生异常也是如此。该模式很难看，但我不确定其他更好的模式能否保证正确性。