实现 IDisposable 时处理构造函数中的异常

Question

我读到，如果我的类有一个本身就是 IDisposable 的成员变量，我需要实现 IDisposable。好吧，我正在实现 IDisposable 接口，因为我的类包含一个数据库对象（下面的 db 类成员），它本身就是 IDisposable：

    public class CommissionModel : IDisposable
    {
        protected PetaPoco.Database db;

        public CommissionModel()
        {
            string connectionString = "Server=localhost;...";

            // The line below may throw an exception (!!!)
            db = new PetaPoco.Database(connectionString, "mysql");              
        }

        // Automatically close database connection
        public void Dispose()
        {
            if (db != null)
                db.Dispose();

            db = null;
        }

        public void InsertRecord(Record somerecord)
        {
            // ...
            db.Insert(somerecord);
        }

问题是 db 的实例化code> 成员在某些情况下可能会失败。 当构造函数抛出异常并且数据库对象没有创建时该怎么办？我应该重新抛出异常还是检查 InsertRecord 方法中是否 db != null ？

Answer 1

如果构造函数抛出异常，代码的使用者将永远不会收到对类实例的引用。部分初始化的类内存最终会被回收，并且不会调用Dispose。

我通常建议将由于外部情况（而不是由于误用，例如无效的参数值）而可能失败的操作移至单独的方法，例如“Connect”。

Answer 2

理想情况下，您不应在构造函数中执行任何“工作”。构造函数中的工作实际上应该是链接对象引用。这将允许您通过连接模拟类来对此类进行单元测试。

试试这个：

public CommissionModel(PetaPoco.Database db) {
    this.db = db;
}

Answer 3

看起来你什么都不用做。

如果您的构建过程被中断，则不会创建资源。在大多数使用场景中（这位于类之外），Dispose() 永远不会被调用。但即使是这样，您的 if (db != null) 代码也足以起到保护作用。

一些小要点：

• Dispose() 中的db = null; 毫无意义。
• InsertRecord() 应该首先检查 if (db != null)

Answer 4

您应该只捕获您可以处理的异常。从代码来看，初始化变量 db 可能表明与数据库服务器通信存在问题。

我的建议是按原样保留代码并在中心位置处理异常，例如 Global.asax 中的 Application_Error 事件或初始化 CommissionModel 的事件。

Answer 5

如果在构造函数获取资源和将资源返回到应用程序之间不可能发生异常(*)，则无需担心。

如果在这种情况下可能会发生异常，我建议采用如下模式：

 public class CommissionModel : IDisposable
    {
        protected PetaPoco.Database db;
        protected OtherResourceType resource2;

        public CommissionModel()
        {
            Boolean ok;
            string connectionString = "Server=localhost;...";

            ok = false;
            try
            {
                // Either of the next two lines may throw an exception
                db = new PetaPoco.Database(connectionString, "mysql");
                resource2 = new OtherResourceType();
                // Once we make it this far, we should be successful
                ok = true;
            }
            finally
            {
                if (!ok)
                  Dispose();
            }
        }

        // Automatically close database connection
        public void Dispose()
        {
            Zap(ref db);  // Method to Dispose and null out, only if not null
            Zap(ref resource2);
        }
    }

(*) 如果某个卑鄙的食人魔在您的代码上调用 Tread.Abort，则几乎总是有可能发生 ThreadAbortException，但实际上没有什么可做的无论如何，我们已经做到了这一点。