在 c# 中创建一个 IDisposable 类，该类在完成后清理 SqlConnection

Question

在对某人推荐的previous question 的回答中：

将 SqlConnection 作为类的成员变量，但使类 IDisposable 并在释放类时释放 SqlConnection

我已经整理了这个建议的实现（如下），但想检查这个实现是否正确（显然它目前除了打开连接之外没有做任何事情，但我的想法是那里会有一些方法可以使用连接，并且能够依赖它存在并打开）。

public class DatabaseRecord : IDisposable
{
    protected SqlConnection connection;

    public DatabaseRecord()
    {
        connection = new SqlConnection("ConnectionString");
        connection.Open();
    }

    // IDisposable implementation

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


    private void Dispose(bool disposing)
    {
        if (!this.disposed)
        {
            if (disposing)
            {
                connection.Dispose();
            }
            disposed = true;
        }

    }

    // Destructor
    ~DatabaseRecord()
    {
        Dispose(false);
    }
}

这行得通吗？使用 DatabaseRecord 实例的类是否需要做任何特殊的事情，或者一旦不再使用/引用实例，会自动调用 Dispose 吗？这是否比在需要连接的每个单独的方法主体中使用 using (var connection = new SqlConnection("...")) { } 更有效/更好？

Answer 1

SqlConnection 是一个托管资源，应该在if (disposing) 块中进行处理。使用你的类的类应该处理它，最好使用 using 块。这是否优于 SqlConnections 的单个 using 块将取决于此类的其他方法以及它们的使用方式。

Answer 2

我看到的所有建议都说DbConnections 应该保留最短时间，所以我希望在我正在审查的代码中看到的格式是

using (var connection = new SqlConnection("...")) { ... }

Answer 3

connection.Dispose() 应移动到 if (disposing) { ... } 块。不需要调用Close()，因为Dispose()会在连接打开时关闭连接。

Answer 4

这将起作用，并且比多个 using 语句更有效。使用 DatabaseRecord 类的代码可以在 using 语句中执行此操作，以便在离开循环时自动清理那里。

然而，一个建议是在 Dispose 方法中检查 Connection 对象的状态，如果在调用 dispose 之前它仍处于打开状态，则将其关闭。