SqlConnection SqlCommand SqlDataReader IDisposable

Question

SqlConnection、SqlCommand 和SqlDataReader 都实现了IDisposable 接口。我读到了始终将IDisposables 包装到using 块中的最佳实践。

所以，我查询数据的常见场景看起来像这样（在更大的上下文中，像 linq2sql 这样的映射工具当然是合适的，但假设我们想在这里使用这种方法）：

using (SqlConnection cn = new SqlConnection("myConnectionstring"))
{
    using (SqlCommand cm = new SqlCommand("myQuery", cn))
    {
        // maybe add sql parameters
        using (SqlDataReader reader = cm.ExecuteReader())
        {
             // read values from reader object
             return myReadValues;
        }
    }
}

这是正确的方法还是被认为是矫枉过正？我有点不确定这种级别的嵌套using 块，但我当然想以正确的方式来做。 谢谢！

Answer 1

这是 100% 正确的方法。如果一个类利用了IDisposable，它应该被包装在一个using 语句中，以确保调用Dispose() 方法。此外，不应掉以轻心，与诸如 SQL Server 之类的不受管理的外部技术进行通信。 SqlCommand 对象实现 IDisposable 是有充分理由的。下面的代码是SqlCommand对象的Dispose()方法：

protected override void Dispose(bool disposing)
{
    if (disposing)
    {
        this._cachedMetaData = null;
    }
    base.Dispose(disposing);
}

如您所见，它正在释放对 _cachedMetaData 对象的引用，以便清理它。

Answer 2

您可以使用以下排版方式使代码更靠近左侧：

using (SqlConnection cn = new SqlConnection("myConnectionstring"))
using (SqlCommand cm = new SqlCommand("myQuery", cn))
using (SqlDataReader reader = cm.ExecuteReader())
{
     // read values from reader object
     return myReadValues;
}

正如其他人已经指出的那样，使用三个嵌套的 using 块是正确的。

Answer 3

这是正确的方法，如果您将与读者一起完成。有时，您需要阅读器保持打开状态（也许您的方法返回它），因此立即处理阅读器是行不通的。在这些情况下，ExecuteReader 的重载可以帮助您：

var cn = new SqlConnection("myConnectionstring");
var cm = new SqlCommand("myQuery", cn);
var reader = cm.ExecuteReader(CommandBehavior.CloseConnection);
return reader;

这将保持连接和阅读器打开。一旦阅读器关闭/处置，它也会关闭（并处置）连接。

using(var reader = GetReader()) //which includes the code above
{
   ...
} // reader is disposed, and so is the connection.

Answer 4

是的，这是正确的。您可能会错过嵌套使用中的大括号，因为它们是一个语句，但我认为这不会增加可读性。

Answer 5

这并不过分。 using 块是一种很好的做法，因为它可以确保调用对象的 Dispose() 方法，即使抛出异常也是如此。

不过，这种东西是有名字的。它被称为代码糖。所以：

using (foo bar = new foo()) { //...snip }

是以下的简写代码：

foo bar = null;
Exception error = null;
try {
    bar = new foo();
    // ...snip
}
catch (Exception ex) {
    error = ex;
}
finally {
    if (bar != null) bar.Dispose();
    if (error != null) throw error;
}

任何一种形式都等同于另一种形式，它们只是写同一件事的不同方式。换句话说，for 和 while 之间的区别相同：它们的作用基本相同，但使用方式不同。

using 是首选，因为它使代码更短且更具可读性，并为您自动处理。不过，至于你是否应该使用它，不要听那些说你应该总是做某事的人。这是很好的做法，当然，但知道何时使用、为什么使用以及使用或不使用的好处和后果是值得的不仅仅是因为人们说你应该做某事。

编辑： Eren 的答案有一个示例，说明您不希望为 reader 设置一个 using 块。

Answer 6

我认为处置SqlCommand 没有任何意义。 SqlConnection 和 SqlDataReader 应该被丢弃。

SqlConnection，因为它在超出范围时不会自行关闭。

SqlDataReader 可以让SqlConnection 保持忙碌，直到阅读器关闭。

甚至 MSDN 示例都没有处理 SqlCommand。

Answer 7

我不是专家，但我知道 using 被转换为 try/finally 块，也许您可​​以将 SqlConnection、SqlCommand 和 SqlDataReader 包装成一个独特的 try/finally

try {
     // code here with SqlConnection, SqlCommand and SqlDataReader
}
finally
{
  // Dispose call on SqlConnection, SqlCommand and SqlDataReader
 }