SqlDataAdapter 关闭连接方法

Question

我的程序中有这样的代码，我相信在填充数据之后它不会关闭连接。

public static string ConnectionInfo = System.Configuration.ConfigurationManager.ConnectionStrings["Default"].ConnectionString;
public static DataTable ExecuteQuery(string query, string table)
    {
        SqlConnection cnn = new SqlConnection(ConnectionInfo);
        SqlDataAdapter Adp = new SqlDataAdapter(query, cnn);
        DataSet Ds = new DataSet();
        Adp.Fill(Ds, table);
        return Ds.Tables[table];
    }

这段代码有什么问题吗？

Answer 1

唯一的问题是您没有对SqlConnection 和DataAdapter 使用using 语句。但是，DbDataAdapter.Fill 隐式打开和关闭连接。

public static DataTable ExecuteQuery(string query, string table)
{
    using(SqlConnection cnn = new SqlConnection(ConnectionInfo))
    using(SqlDataAdapter Adp = new SqlDataAdapter(query, cnn))
    {
        DataTable tbl = new DataTable();
        Adp.Fill(tbl);
        return tbl;
    }
}

与 SELECT 语句关联的连接对象必须是 有效，但不需要打开。如果连接关闭 在调用 Fill 之前，打开它以检索数据，然后关闭。如果 在调用 Fill 之前连接是打开的，它保持打开状态。

注意

• using 语句将隐式关闭连接，即使发生错误
• 我使用了DataAdapter.Fill(DataTable)，因为无论如何你都在使用单个表

编辑：我刚刚注意到您正在使用表名的参数。您也可以改用DbDataAdapter.Fill(DataSet, String)。这不会改变任何事情。

Answer 2

添加 using 语句以可靠地关闭连接。这确保即使发生异常也关闭连接。更改代码如下：

public static DataTable ExecuteQuery(string query, string table)
    {
        using(SqlConnection cnn = new SqlConnection(ConnectionInfo))
        {
            SqlDataAdapter Adp = new SqlDataAdapter(query, cnn);
            DataSet Ds = new DataSet();
            Adp.Fill(Ds, table);
            return Ds.Tables[table];
        }
    }

Answer 3

无论连接的打开/关闭都应该在 try-catch-finally 块中完成。

我们不应该使用“using” [使用 (SqlConnection connection = new SqlConnection(connectionString))] 堵塞。因为如果网络出现问题或任何异常原因。连接未关闭。所以最好使用 try-catch 块。

    public static DataTable ExecuteQuery(string query, string table)
    {
        DataSet Ds = new DataSet();

        SqlConnection cnn = new SqlConnection(ConnectionInfo);

        try{
            SqlDataAdapter Adp = new SqlDataAdapter(query, cnn);
            Adp.Fill(Ds, table);
            return Ds.Tables[table]; 
        }
        catch{
            throw;
        }
        finally{
            cnn.Close();
        }

    }