为什么SqlConnection不被处理/关闭？

Question

鉴于这一方法：

internal static DataSet SelectDataSet(String commandText, DataBaseEnum dataBase)
{
    var dataset = new DataSet();

    SqlConnection sqlc = dataBase == DataBaseEnum.ZipCodeDb
                             ? new SqlConnection(ConfigurationManager.AppSettings["ZipcodeDB"])
                             : new SqlConnection(ConfigurationManager.AppSettings["WeatherDB"]);
    SqlCommand sqlcmd = sqlc.CreateCommand();
    sqlcmd.CommandText = commandText;
    var adapter = new SqlDataAdapter(sqlcmd.CommandText, sqlc);
    adapter.Fill(dataset);


    return dataset;
}

为什么在调用方法超出范围或sqlc没有更多引用之后，sqlc ( SqlConnection)没有被释放/关闭？

编辑1:甚至在使用中包装它，我仍然可以看到连接使用(我关闭了连接池)：

SELECT DB_NAME(dbid) as 'Database Name',
COUNT(dbid) as 'Total Connections'
FROM sys.sysprocesses WITH (nolock)
WHERE dbid > 0
GROUP BY dbid

编辑2:我从这里得到的帮助下，还有一些更多的调试--答案是有人硬编码了一个带有池的连接字符串。谢谢你的帮助--如果可以的话，我会把所有的回复都标记为答案。

Answer 1

C#的垃圾收集是不确定的，但该语言确实为资源处理提供了如下确定性结构：

using (SqlConnection connection = new SqlConnection(...))
{
    // ...  
}

这将创建一个try/finally块，它将确保释放连接对象，而不管方法中发生了什么。您确实应该在这样的using块中包装实现IDisposable的任何类型的实例，因为它将确保负责任的资源管理(对非托管资源，如数据库连接)，并且它还为您提供您正在寻找的确定性控制。

Answer 2

因为c#是一种垃圾收集语言，而垃圾收集不是确定性的。它的事实是您的The连接被释放了。你只是不能选择什么时候。

Sql连接是一个有限的资源，您很容易就可以创建足够多的连接来运行。把它写成这样：

internal static DataSet SelectDataSet(String commandText, DataBaseEnum dataBase)
{
    var dataset = new DataSet();

    using (SqlConnection sqlc = dataBase == DataBaseEnum.ZipCodeDb
                             ? new SqlConnection(ConfigurationManager.AppSettings["ZipcodeDB"])
                             : new SqlConnection(ConfigurationManager.AppSettings["WeatherDB"]))
    using (SqlCommand sqlcmd = sqlc.CreateCommand())
    {
        sqlcmd.CommandText = commandText;
        var adapter = new SqlDataAdapter(sqlcmd.CommandText, sqlc);
        adapter.Fill(dataset);

    }
    return dataset;
}

虽然在这种情况下，您可能会逃脱它，因为.Fill() method is a strange beast:

如果在调用Fill之前关闭了IDbConnection，则打开IDbConnection以检索数据，然后关闭。

因此，这意味着数据适配器应该为您处理它，如果您从一个封闭的连接开始。我更担心的是，您将sql命令作为普通字符串传递。查询中必须不时有用户参数，这意味着要将数据直接连接到命令字符串中。不要这样做！！使用SqlCommand的Paramters集合。

Answer 3

我同意这里的所有答案，加上一个连接可能是一个更广泛的范围，而不仅仅是一种方法。当您需要在不同的地方使用您现有的连接时，场景会发生一些变化。始终确保在使用完IDisposable之后，为实现IDisposable的所有对象调用它们。这是一个很好的实践，这样您就不会得到垃圾收集器无法决定如何处理它们的未使用对象。