Veri kaydı satırını verir ve tüm kayıtları okumazsak Veritabanı bağlantısı kapatılır mı?

yieldAnahtar kelimenin nasıl çalıştığını anlarken , StackOverflow üzerinde Link1 ve link2 ile karşılaştım yield return. Ama yield returnaşağıda gösterildiği gibi kullanırsam ve tüm DataReader üzerinden yineleme yapmazsam, DB bağlantısı sonsuza kadar açık kalacak mı?

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;
  }
}

Aynı örneği bir örnek Konsol App denedim ve sonunda bloğunun GetRecords()yürütülmediğini hata ayıklama sırasında fark ettim . DB Bağlantısının kapatılmasını nasıl sağlayabilirim? yieldAnahtar kelime kullanmaktan daha iyi bir yol var mı ? Ben DB üzerinde seçili SQL'leri ve saklı yordamları yürütmekten sorumlu olacak ve sonuç dönecektir özel bir sınıf tasarlamak çalışıyorum. Ancak DataReader'ı arayana geri vermek istemiyorum. Ayrıca bağlantının tüm senaryolarda kapatıldığından emin olmak istiyorum.

Düzenle Yöntem arayanların yöntemi doğru kullanmasını beklemek yanlış olduğu için Ben'in cevabının cevabını değiştirdi ve DB bağlantısına göre, yöntemin sebepsiz olarak birden çok kez çağrılması daha pahalı olacak.

Ayrıntılı açıklama için Jakob ve Ben'e teşekkür ederiz.

Evet, açıkladığınız sorunla karşılaşacaksınız: sonucun yinelemesini bitirinceye kadar bağlantıyı açık tutacaksınız. Bununla başa çıkmayı düşünebileceğim iki genel yaklaşım var:

İt, çekme

Şu anda IEnumerable<IDataRecord>, çekebileceğiniz bir veri yapısı döndürüyorsunuz . Bunun yerine, sonuçlarını dışarı itmek için yönteminizi değiştirebilirsiniz . En basit yol Action<IDataRecord>, her bir yinelemede çağrılan bir geçiş yapmaktır:

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

Bir öğe koleksiyonu ile uğraştığınız göz önüne alındığında, IObservable/ IObserverbiraz daha uygun bir veri yapısı olabilir, ancak ihtiyacınız yoksa, basit Actionbir çok daha basittir.

Hevesle Değerlendirin

Bir alternatif, geri dönmeden önce yinelemenin tamamen tamamlandığından emin olmaktır.

Normalde, sonuçları yalnızca bir listeye koyarak sonra geri göndererek yapabilirsiniz, ancak bu durumda her öğenin okuyucuya aynı referans olmasıyla ilgili ek bir karmaşıklık vardır. Dolayısıyla, okuyucudan ihtiyacınız olan sonucu çıkarmak için bir şeye ihtiyacınız var:

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;
}

Kişisel finallybloğu hep çalıştırır.

yield returnDerleyiciyi kullandığınızda durum makinesi uygulamak için yeni bir iç içe sınıf oluşturur.

Bu sınıf, finallyayrı yöntemler olarak tüm kod bloklarını içerecektir . finallyDuruma bağlı olarak yapılması gereken blokları takip edecektir . Gerekli tüm finallybloklar Disposeyöntemde yürütülecektir .

C # dili şartnamesine göre, foreach (V v in x) embedded-statementgenişletilir

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

listeleyicisi Birlikte döngü çıkmak bile bertaraf edilecektir böylece breakya return.

Yineleyici uygulaması hakkında daha fazla bilgi almak için Jon Skeet'in bu makalesini okuyabilirsiniz.

Düzenle

Bu yaklaşımla ilgili sorun, yöntemi doğru kullanmak için sınıfınızın istemcilerine güvenmenizdir. Örneğin, numaralandırıcıyı doğrudan alabilir ve whileatmadan bir döngü içinde yineleyebilirler .

@BenAaronson tarafından önerilen çözümlerden birini düşünmelisiniz.

Belirli yielddavranıştan bağımsız olarak, kodunuz kaynaklarınızı doğru bir şekilde atmayacak yürütme yolları içerir. İkinci satırınız bir istisna veya üçüncünüz atarsa ​​ne olur? Ya da dördüncüsü? Çok karmaşık bir try / nihayet zincirine ihtiyacınız olacak veya usingblokları kullanabilirsiniz .

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;
               }
            }
        }
    }
}

İnsanlar bunun sezgisel olmadığını, yönteminizi çağıran kişilerin sonucu iki kez numaralandırmanın yöntemi iki kez arayacağını bilmeyebileceğini belirtti. İyi şanslar. Dil böyle çalışır. Gönderen sinyal budur IEnumerable<T>. Bu dönmez bir sebep yoktur List<T>ya T[]. Bunu bilmeyen insanlar eğitilmeli, çalışılmamalıdır.

Visual Studio'nun statik kod analizi adı verilen bir özelliği vardır. Kaynakları doğru bir şekilde bertaraf edip etmediğinizi öğrenmek için kullanabilirsiniz.

Yaptığınız yanlış görünüyor, ama iyi çalışacak.

Bir IEnumerator <> kullanmalısınız , çünkü IDisposable'ı da devralır .

Ancak bir foreach kullandığınız için, derleyici yine de foreach için bir IEnumerator <> oluşturmak için bir IDisposable kullanmaktadır .

gerçekte, foreach dahili olarak birçok şey içerir.

Https://stackoverflow.com/questions/13459447/do-i-need-to-consider-disposing-of-any-ienumerablet-i-use adresini kontrol edin