ADO.NET 和不使用的处理

Question

我有一个项目没有在任何地方使用他们的 ADO.NET 代码。我正在清理他们未关闭的连接。下面的代码是 try/catch/finally 的最佳实践吗？我也有一些包含我在命令和连接处理之间处理的 SqlTransaction。

SqlConnection con = new SqlConnection(ConfigurationManager.AppSettings["MyNGConnectDashBoardConnectionString"].ToString());
            SqlCommand cmd = new SqlCommand();
            DataSet ds = new DataSet();
            try
            {
                con.Open();
                cmd.Connection = con;

                SqlDataAdapter da = new SqlDataAdapter(cmd);
                da.Fill(ds);

            }
            catch (Exception ex)
            {
                throw ex;
            }

            finally
            {
                cmd.Dispose();
                con.Dispose();
            }

Answer 1

其实在使用SqlDataAdapter.Fill(dataset)方法的时候不用担心会关闭连接。此方法在执行每个 Fill 后关闭连接。

此外，无需调用SqlCommand.Dispose()，因为命令本身没有需要清理的非托管资源。您应该关心的是是否在某个时候调用了SqlConnection.Close()。这是在填充之后完成的。

Answer 2

你所拥有的一切都很好。处置使用非托管资源的对象总是一个好主意。但是，如果您厌倦了总是显式调用 Dispose，最好的做法可能是使用 using：

using (SqlConnection con = new SqlConnection(ConfigurationManager.AppSettings["MyNGConnectDashBoardConnectionString"].ToString()))
{
    using (SqlCommand cmd = new SqlCommand())
    {
        DataSet ds = new DataSet();
        try
        {
            con.Open();
            cmd.Connection = con;

            SqlDataAdapter da = new SqlDataAdapter(cmd);
            da.Fill(ds);

        }
        catch (Exception ex)
        {
            throw; // I changed this too!
        }
    }
}

此外，如果您要“重新抛出”异常，您几乎总是只想简单地throw。如果你throw ex;，你会丢失一些堆栈跟踪。

Answer 3

最佳做法是使用 using 而不是 try/finally :)

但是在您的情况下，甚至不需要使用，因为Fill() 会关闭连接：

        SqlConnection con = new SqlConnection(ConfigurationManager.AppSettings["MyNGConnectDashBoardConnectionString"].ToString())
        SqlDataAdapter da = new SqlDataAdapter("your sql is here", con);
        da.Fill(ds);

简单的重新抛出异常也毫无意义。如果您需要记录错误，只需按照@Cory 的建议使用普通的throw;。