Question

所以我已经阅读了很多关于SqlDataReaders没有在.Net中被正确处理的内容 - 而且我已经和＃34; Timeout过期了。从池中获取连接之前经过的超时时间。这可能是因为所有池连接都在使用中并且达到了最大池大小＆＃34;错误几天了。显然，我可以将最大池大小提升到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的代码非常相似，因此以这种方式修改应用程序是相对无痛的。

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

2 个答案: