.Net数据库如何正确关闭我的数据库连接？

Question

所以我已经阅读了很多关于SqlDataReaders没有在.Net中正确处理的内容 - 而且我一直在与“Timeout expired”进行斗争。在从池中获取连接之前已经过了超时时间。这可能是因为所有池连接都发生了正在使用并且达到最大池大小“现在错误了几天。 显然，我可以将最大池大小提升到30,000 - 但这并不能解决实际问题。

当我单步执行代码时，我执行以下SQL查询：

select * from sys.dm_os_performance_counters
where counter_name ='User Connections'

之后

cmd.Connection.Open(); 

行，用户连接增加1.但是，除非我在Web服务器上回收应用程序池（此时，网站上的所有活动数据库连接都被终止），否则它永远不会回落。

这是我的代码：

public static DataTable SPExecuteDataTable(string[] ConnectionData, params object[] args)
{
    SqlConnection conn = null;
    SqlCommand cmd = null;
    SqlDataReader dr = null;
    try
    {
        conn = new SqlConnection(ConnectionData[1]);
        cmd = new SqlCommand(ConnectionData[0], new SqlConnection(ConnectionData[1]));
        cmd.CommandType = CommandType.StoredProcedure;

        for (int i = 0; i < args.Length; i++)
        {
            SqlParameter Param = new SqlParameter(ConnectionData[i + 2], DBNullIfNull(args[i]));
            cmd.Parameters.Add(Param);
        }

        cmd.Connection.Open();
        DataTable dt = new DataTable();

        using (dr = cmd.ExecuteReader())
        {
            if (dr != null)
                dt.Load(dr);
            else
                dt = null;
        }

        return dt;
    }
    catch (Exception e)
    {
        Exception x = new Exception(String.Format("DataAccess.SPExecuteDataTable() {0}", e.Message));
        throw x;
    }
    finally
    {
        conn.Close();
        cmd.Connection.Close();
        dr.Close();
        conn.Dispose();
        cmd.Dispose();
        dr.Dispose();
    }

到目前为止，我已经尝试明确关闭连接（就像在我的finally块中），但这不起作用。 我也试过使用这样的语句：

using (SqlDataReader dr = blah blah blah)
{
    //code here
}

但这也行不通。 我的代码出了什么问题，在这里？

Answer 1

首选实践是using块包装连接，命令和读取器：

using(SqlConnection conn = new SqlConnection(ConnectionData[1])
{
    using(SqlCommand cmd = new SqlCommand(ConnectionData[0], conn)
    {                                                     // ^-- re-use connection - see comment below
        cmd.CommandType = CommandType.StoredProcedure;

        for (int i = 0; i < args.Length; i++)
        {
            SqlParameter Param = new SqlParameter(ConnectionData[i + 2], DBNullIfNull(args[i]));
            cmd.Parameters.Add(Param);
        }

        cmd.Connection.Open();
        DataTable dt = new DataTable();

        using (dr = cmd.ExecuteReader())
        {
            if (dr != null)
                dt.Load(dr);
            else
                dt = null;
        }

        return dt;
    }    
}

这样他们都会被关闭并妥善处理。

虽然我认为你问题的核心在于你每次创建两个连接：

conn = new SqlConnection(ConnectionData[1]);
cmd = new SqlCommand(ConnectionData[0], new SqlConnection(ConnectionData[1]));
                                        ^----  creating a second connection

最后，您通过创建新异常并抛出它而不是重新抛出原始异常，丢失了许多可能有价值的信息（堆栈跟踪等）：

catch (Exception e)
{
    Exception x = new Exception(String.Format("DataAccess.SPExecuteDataTable() {0}", e.Message));
    throw x;
}

我要么让原始异常冒泡，要么包含原始异常作为InnerException ：

catch (Exception e)
{
    string message = String.Format("DataAccess.SPExecuteDataTable() {0}", e.Message);
    Exception x = new Exception(message, e);
    throw x;
}

Answer 2

解：

使用DataTables！ 要防止应用程序的非数据访问层与数据库通信，只需在数据访问层中执行此操作：

using (SqlDataReader dr = cmd.ExecuteReader())
                        {
                            if (dr != null)
                                dt.Load(dr);
                            else
                                dt = null;
                        }
                        return dt;

然后，您可以在解决方案的其余部分中操纵dt，并且已经正确处理了连接。 像魅力一样工作，幸运的是，数据表和datareader的代码非常相似，因此以这种方式修改应用程序是相对轻松的。