如果我们产生datareader行而不读取所有记录，数据库连接会关闭吗？

在了解yield关键字的工作原理的同时，我遇到了StackOverflow 上的link1和link2，它主张yield return在DataReader 上进行迭代时使用它，它也适合我的需求。但是，这使我感到奇怪，如果发生如下情况，如果我yield return按如下所示使用，并且如果我不遍历整个DataReader，那么数据库连接会永远保持打开状态吗？

IEnumerable<IDataRecord> GetRecords()
{
    SqlConnection myConnection = new SqlConnection(@"...");
    SqlCommand myCommand = new SqlCommand(@"...", myConnection);
    myCommand.CommandType = System.Data.CommandType.Text;
    myConnection.Open();
    myReader = myCommand.ExecuteReader(CommandBehavior.CloseConnection);
    try
       {               
          while (myReader.Read())
          {
            yield return myReader;          
          }
        }
    finally
        {
            myReader.Close();
        }
}


void AnotherMethod()
{
    foreach(var rec in GetRecords())
    {
       i++;
       System.Console.WriteLine(rec.GetString(1));
       if (i == 5)
         break;
  }
}

我在示例控制台应用程序中尝试了相同的示例，并在调试时注意到GetRecords()未执行的finally块。我怎样才能确保关闭数据库连接？有没有比使用yield关键字更好的方法？我正在尝试设计一个自定义类，该类将负责在数据库上执行选择的SQL和存储过程，并将返回结果。但是我不想将DataReader返回给调用者。我还要确保在所有情况下都将关闭连接。

编辑将答案更改为Ben的答案，因为期望方法调用者正确使用该方法是不正确的，并且对于DB连接，如果无缘无故多次调用该方法，则将更加昂贵。

感谢Jakob和Ben提供详细说明。

是的，您将遇到您描述的问题：在完成对结果的迭代之前，将保持连接打开。我可以考虑以下两种一般方法：

推，不要拉

目前，您正在返回IEnumerable<IDataRecord>，可以从中提取数据结构。相反，您可以切换方法以推出其结果。最简单的方法是传递在Action<IDataRecord>每次迭代中调用的an ：

void GetRecords(Action<IDataRecord> callback)
{
    // ...
      while (myReader.Read())
      {
        callback(myReader);
      }
}

请注意，鉴于您要处理的是项目集合，IObservable/ IObserver可能是稍微合适的数据结构，但是除非您需要它，否则简单Action得多。

认真评估

另一种选择是确保返回之前迭代完全完成。

通常，您可以通过将结果放在列表中然后返回来完成此操作，但是在这种情况下，每个项目的额外复杂性是对读者的相同引用。因此，您需要一些东西来从阅读器中提取所需的结果：

IEnumerable<T> GetRecords<T>(Func<IDataRecord,T> extractor)
{
    // ...
     var result = new List<T>();
     try
     {
       while (myReader.Read())
       {
         result.Add(extractor(myReader));         
       }
     }
     finally
     {
         myReader.Close();
     }
     return result;
}

您的finally块将始终执行。

使用yield return编译器时，将创建一个新的嵌套类以实现状态机。

此类将包含来自finally块的所有代码作为单独的方法。它将finally根据状态跟踪需要执行的块。所有必要的finally块将在Dispose方法中执行。

根据C＃语言规范，foreach (V v in x) embedded-statement扩展到

{
    E e = ((C)(x)).GetEnumerator();
    try
    {
        while (e.MoveNext())
        {
            V v = (V)(T)e.Current;
            embedded - statement
        }
    }
    finally {
        … // Dispose e
    }
}

因此即使您使用break或退出循环，枚举器也将被处置return。

要获得有关迭代器实现的更多信息，您可以阅读Jon Skeet的这篇文章。

编辑

这种方法的问题在于，您依赖类的客户端来正确使用该方法。例如，他们可以直接获取枚举数，并在while不处理它的情况下循环遍历该枚举数。

您应该考虑@BenAaronson建议的解决方案之一。

不论具体 yield行为如何，您的代码都包含无法正确处理资源的执行路径。如果第二行或第三行抛出异常怎么办？甚至是您的第四个？您将需要一个非常复杂的try / finally链，也可以使用using块。

IEnumerable<IDataRecord> GetRecords()
{
    using(var connection = new SqlConnection(@"..."))
    {
        connection.Open();

        using(var command = new SqlCommand(@"...", connection);
        {
            using(var reader = command.ExecuteReader())
            {
               while(reader.Read())
               {
                   // your code here.
                   yield return reader;
               }
            }
        }
    }
}

人们指出这不是很直观，人们调用您的方法可能不知道两次枚举结果将两次调用该方法。好吧，祝你好运。这就是语言的工作方式。那是信号IEnumerable<T>发出。这是不返回List<T>或的原因T[]。不知道这一点的人需要受教育，不能解决。

Visual Studio具有一项称为静态代码分析的功能。您可以使用它来查找是否正确配置了资源。

您所做的事情似乎错了，但可以正常工作。

您应该使用IEnumerator <>，因为它还继承了IDisposable。

但是因为您使用的是foreach，所以编译器仍在使用IDisposable生成IEnumerator <>为foreach。

实际上，foreach涉及很多内部问题。

检查/programming/13459447/do-i-need-to-consider-dispose-of-any-ienumerablet-i-use