使用 IDisposable 时在哪里放置 try/catch

Question

我只是建议将整个 using 块放入 try 中，否则 using 范围将阻止捕获异常。但是，如果抛出异常，这是否会阻止 using 正确处置其资源？如果我有下面的代码，我应该将 try 块放在哪里？

using (connSQL = new SqlConnection(strConn)) 
{
    connSQL.Open();
    using (SqlCommand commSQL = new SqlCommand(strPreparedStatement, connSQL)) 
    {
        if (sqlParameters != null)
        {
            for (int i = sqlParameters.GetLowerBound(0); i <= sqlParameters.GetUpperBound(0); i++)
            {
                commSQL.Parameters.Add(sqlParameters[i]);
            }
        }
        drSQL = commSQL.ExecuteReader();
        dtReturn.Load(drSQL);

        commSQL.Parameters.Clear();
    }
}

在此应用程序中，确保与数据库的连接不会因异常而开始累积更为重要。

Answer 1

using 语句已经可以防止连接累积 - 它通过有效地成为一个 try/finally 块来为您完成清理工作对 finally 块中的 Dispose 的调用。如果您还想要一个 try/catch 块，您可以将其放在内部或外部 - 但您确定它不应该位于更高的水平？您实际上期望如何处理异常？

顺便说一句，当命令即将被释放时，尚不清楚为什么要清除命令中的参数...

我鼓励您也在 using 语句中声明变量，这样您就不会最终尝试在块之外读取它们：

using (SqlConnection connSQL = new SqlConnection(strConn)) {

通常，更愿意为变量提供尽可能窄的范围。哦，您的 SqlDataReader 也应该位于 using 语句中。无论如何，这可能并不重要，因为您要关闭连接和命令，但我会按照原则这样做 - 它实现了 IDisposable，因此您应该处置它。

哦，目前您迭代 sqlParameters 的方式相当冗长。 foreach 使它更简单，即使没有，我也会使用 for (int i = 0; i 除非我有理由相信这不是一个“简单”的数组。

所以，我的等效代码看起来像这样：

using (SqlConnection connection = new SqlConnection(connectionString))
{
    connection.Open();
    using (SqlCommand command = new SqlCommand(strPreparedStatement, connection))
    {
        if (sqlParameters != null)
        {
            // If sqlParameter is an array, you can just use
            // command.Parameters.AddRange(sqlParameters) instead
            foreach (SqlParameter parameter in sqlParameters)
            {
                command.Parameters.Add(parameter);
            }
        }
        using (SqlDataReader reader = command.ExecuteReader())
        {
            DataTable table = new DataTable();
            // Perform any extra initialization here
            table.Load(reader);
            return table;
        }
    }
}

Answer 2

您还需要将数据读取器包装在 using 语句中，因为它是一次性资源：

using (var connSQL = new SqlConnection(strConn)) 
using (var commSQL = connSQL.CreateCommand()) 
{
    connSQL.Open();
    commSQL.CommandText = strPreparedStatement;
    if (sqlParameters != null)
    {
        for (int i = sqlParameters.GetLowerBound(0); i <= sqlParameters.GetUpperBound(0); i++)
        {
            commSQL.Parameters.Add(sqlParameters[i]);
        }
    }
    using (var drSQL = commSQL.ExecuteReader())
    {
        dtReturn.Load(drSQL);
    }
}

我还在该块作用域中本地定义了连接、命令和数据读取器对象。

就 try/finally 语句而言，您不再需要它，因为 using 语句确保即使在发生异常时也会调用 Dispose 方法。这种 SQL 连接和命令的方法可以确保正确释放它们。

Answer 3

不需要放置 try catch 因为 using 隐式实现它，实际上它使用 try finally 并确保它会释放对象。

这是关于 try catch 和使用的 MSDN 使用示例

{
  Font font1 = new Font("Arial", 10.0f);
  try
  {
    byte charset = font1.GdiCharSet;
  }
  finally
  {
    if (font1 != null)
      ((IDisposable)font1).Dispose();
  }
}

等于：

using (Font font3 = new Font("Arial", 10.0f),
            font4 = new Font("Arial", 10.0f))
{
    // Use font3 and font4.
}