データリーダー行を取得し、すべてのレコードを読み取らない場合、データベース接続は閉じられますか？

方法を理解しながらyieldキーワード作品を、私が出会ったリンク1とリンク2の使用を提唱StackOverflowの上yield returnDataReaderオブジェクトを反復処理しながら、をし、それは私のニーズに合ったとしても。しかし、yield return以下に示すように使用し、DataReader全体を反復処理しない場合、DB接続はいつまでも開いたままになるのか、疑問に思います。

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()実行されないことに気付きました。DB接続の終了を確認するにはどうすればよいですか？yieldキーワードを使用するよりも良い方法はありますか？DBで選択SQLとストアドプロシージャを実行し、結果を返すカスタムクラスを設計しようとしています。しかし、DataReaderを呼び出し元に返したくありません。また、すべてのシナリオで接続が閉じられるようにします。

編集メソッドの呼び出し元がメソッドを正しく使用することを期待するのは間違っているため、Benの答えへの答えを変更しました。

詳細な説明をしてくれたJakobとBenに感謝します。

はい、説明した問題に直面します。結果の反復処理が完了するまで、接続を開いたままにします。これに対処するために考えられる2つの一般的なアプローチがあります：

プッシュ、プルしないでください

現在IEnumerable<IDataRecord>、から取得できるデータ構造であるを返しています。代わりに、メソッドを切り替えて結果をプッシュすることができます。最も簡単な方法はAction<IDataRecord>、各反復で呼び出されるを渡すことです。

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ブロックがDisposemethod で実行されます。

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によって提案された解決策の1つを検討する必要があります。

特定のyield動作に関係なく、コードにはリソースを正しく破棄しない実行パスが含まれています。2行目が例外をスローした場合、または3行目が例外をスローした場合はどうなりますか？それとも4番目ですか？非常に複雑な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;
               }
            }
        }
    }
}

これは直観的ではなく、メソッドを呼び出す人は、結果を2回列挙するとメソッドが2回呼び出されることを知らないかもしれないと人々は言っています。まあ、タフな運。それが言語の仕組みです。それがIEnumerable<T>送信する信号です。これは戻らない理由があるList<T>かがT[]。これを知らない人々は教育を受け、回避する必要はありません。

Visual Studioには、静的コード分析と呼ばれる機能があります。リソースを適切に破棄したかどうかを調べるために使用できます。

あなたがしたことは間違っているように見えますが、うまくいきます。

IDisposableも継承するため、IEnumerator <>を使用する必要があります。

ただし、foreachを使用しているため、コンパイラはまだIDisposableを使用してforeachのIEnumerator <>を生成しています。

実際、foreachは内部的に多くのことを伴います。

/programming/13459447/do-i-need-to-consider-disposing-of-any-ienumerablet-i-useを確認してください