引发异常时确保关闭 SQL 连接的正确方法是什么？

Question

我经常使用看起来像这样的模式。我想知道这是否可行，或者是否有我没有在这里应用的最佳实践。

我特别想知道；在抛出异常的情况下，我在 finally 块中的代码是否足以确保正确关闭连接？

public class SomeDataClass : IDisposable
{
    private SqlConnection _conn;

    //constructors and methods

    private DoSomethingWithTheSqlConnection()
    {
        //some code excluded for brevity

        try
        {
            using (SqlCommand cmd = new SqlCommand(SqlQuery.CountSomething, _SqlConnection))
            {
                _SqlConnection.Open();
                countOfSomething = Convert.ToInt32(cmd.ExecuteScalar());
            }
        }
        finally
        {
            //is this the best way?
            if (_SqlConnection.State == ConnectionState.Closed)
                _SqlConnection.Close();
        }

        //some code excluded for brevity
    }

    public Dispose()
    {
        _conn.Dispose();
    }
}

Answer 1

我猜_SqlConnection.State == ConnectionState.Closed 是指!=

这肯定会奏效。我认为将连接对象本身包含在 using 语句中更为习惯，但是如果您出于某种原因想要重用相同的连接对象，那么您所拥有的就很好。

不过，您绝对应该更改的一件事是Dispose() 方法。您不应该在 dispose 中引用连接对象，因为它可能已经在此时完成。您应该改用推荐的 Dispose 模式。

Answer 2

.Net 框架维护一个连接池是有原因的。相信它！ :) 不必为了连接数据库和释放连接而编写这么多代码。

您可以只使用“using”语句并放心“IDBConnection.Release()”将为您关闭连接。

高度复杂的“解决方案”往往会导致代码错误。简单更好。

Answer 3

不需要 try..finally 围绕“使用”，使用 IS 是 try..finally

Answer 4

像您展示的那样，将连接关闭代码放在“Finally”块中。 finally 块在抛出异常之前执行。使用“使用”块同样有效，但我发现显式“最终”方法更清晰。

对于许多开发人员来说，使用语句是老生常谈，但年轻的开发人员可能不知道这一点。

Answer 5

请看这个问题的答案：

Close and Dispose - which to call?

如果您的连接生命周期是单个方法调用，请使用该语言的using 功能来确保正确清理连接。虽然try/finally 块在功能上是相同的，但它需要更多的代码并且 IMO 的可读性较差。无需检查连接状态，您可以拨打Dispose，它会处理清理连接。

如果您的连接生命周期对应于包含类的生命周期，则实现IDisposable 并清理Dispose 中的连接。

Answer 6

MSDN Docs 说得很清楚...

• Close 方法回滚所有挂起的事务。然后它释放到连接池的连接，或者如果连接池被禁用，则关闭连接。

您可能没有（也不想）禁用连接池，因此池最终会在您调用“关闭”后管理连接的状态。这可能很重要，因为从数据库服务器端查看所有打开的连接可能会让您感到困惑。

---

• 应用程序可以多次调用 Close。不会产生异常。

那么，为什么还要为 Closed 测试呢？只需调用 Close()。

---

• Close 和 Dispose 在功能上是等效的。

这就是 using 块导致连接关闭的原因。 使用为你调用 Dispose。

---

• 不要在类的 Finalize 方法中对 Connection、DataReader 或任何其他托管对象调用 Close 或 Dispose。

重要的安全提示。谢谢，埃贡。

Answer 7

我可以建议这个吗：

    class SqlOpener : IDisposable
    {
        SqlConnection _connection;

        public SqlOpener(SqlConnection connection)
        {
            _connection = connection;
            _connection.Open();

        }

        void IDisposable.Dispose()
        {
            _connection.Close();
        }
    }

    public class SomeDataClass : IDisposable
    {
        private SqlConnection _conn;

        //constructors and methods

        private void DoSomethingWithTheSqlConnection()
        {
            //some code excluded for brevity
            using (SqlCommand cmd = new SqlCommand("some sql query", _conn))
            using(new SqlOpener(_conn))
            {
                int countOfSomething = Convert.ToInt32(cmd.ExecuteScalar());
            }
            //some code excluded for brevity
        }

        public void Dispose()
        {
            _conn.Dispose();
        }
    }

希望有帮助:)

Answer 8

因为无论如何您都在使用 IDisposables。可以使用'using'关键字，基本上相当于在finally块中调用dispose，但是看起来更好。

Answer 9

将您的数据库处理代码包装在“使用”中

using (SqlConnection conn = new SqlConnection (...))
{
    // Whatever happens in here, the connection is 
    // disposed of (closed) at the end.
}