Kết nối cơ sở dữ liệu sẽ bị đóng nếu chúng ta mang lại hàng dữ liệu và không đọc tất cả các bản ghi?

Trong khi hiểu cách yieldhoạt động của từ khóa, tôi đã xem qua link1 và link2 trên StackOverflow, chủ trương sử dụng yield returntrong khi lặp qua DataReader và nó cũng phù hợp với nhu cầu của tôi. Nhưng nó làm tôi tự hỏi điều gì sẽ xảy ra, nếu tôi sử dụng yield returnnhư được hiển thị bên dưới và nếu tôi không lặp qua toàn bộ DataReader, liệu kết nối DB có mở mãi mãi không?

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

Tôi đã thử ví dụ tương tự trong Ứng dụng Console mẫu và nhận thấy trong khi gỡ lỗi rằng khối cuối cùng GetRecords()không được thực thi. Làm thế nào tôi có thể đảm bảo sau đó đóng kết nối DB? Có cách nào tốt hơn là sử dụng yieldtừ khóa không? Tôi đang cố gắng thiết kế một lớp tùy chỉnh sẽ chịu trách nhiệm thực thi các SQL được chọn và các thủ tục được lưu trữ trên DB và sẽ trả về kết quả. Nhưng tôi không muốn trả lại DataReader cho người gọi. Ngoài ra tôi muốn đảm bảo rằng kết nối sẽ được đóng lại trong tất cả các tình huống.

Biên tập Thay đổi câu trả lời cho câu trả lời của Ben vì không chính xác khi mong đợi người gọi phương thức sử dụng phương thức này một cách chính xác và đối với kết nối DB sẽ tốn kém hơn nếu phương thức được gọi nhiều lần mà không có lý do.

Cảm ơn Jakob và Ben đã giải thích chi tiết.

Có, bạn sẽ phải đối mặt với vấn đề bạn mô tả: cho đến khi bạn hoàn thành việc lặp lại kết quả, bạn sẽ giữ kết nối mở. Có hai cách tiếp cận chung mà tôi có thể nghĩ ra để giải quyết vấn đề này:

Đẩy, đừng kéo

Hiện tại bạn đang trả về một IEnumerable<IDataRecord>cấu trúc dữ liệu bạn có thể lấy từ đó. Thay vào đó, bạn có thể chuyển đổi phương pháp của mình để đẩy kết quả của nó ra. Cách đơn giản nhất là vượt qua trong một lệnh Action<IDataRecord>được gọi trên mỗi lần lặp:

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

Lưu ý rằng bạn đang xử lý một bộ sưu tập các mặt hàng, IObservable/ IObservercó thể là một cấu trúc dữ liệu phù hợp hơn một chút, nhưng trừ khi bạn cần nó, một cách đơn giản Actionsẽ đơn giản hơn nhiều.

Đánh giá háo hức

Một cách khác là chỉ để đảm bảo việc lặp lại hoàn thành trước khi quay trở lại.

Bạn thường có thể làm điều này bằng cách chỉ đưa các kết quả vào một danh sách sau đó trả lại, nhưng trong trường hợp này có sự phức tạp thêm của mỗi mục là cùng một tham chiếu đến người đọc. Vì vậy, bạn cần một cái gì đó để trích xuất kết quả bạn cần từ người đọc:

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

finallyKhối của bạn sẽ luôn luôn thực thi.

Khi bạn sử dụng yield returntrình biên dịch sẽ tạo một lớp lồng nhau mới để thực hiện một máy trạng thái.

Lớp này sẽ chứa tất cả các mã từ finallycác khối như các phương thức riêng biệt. Nó sẽ theo dõi các finallykhối cần được thực hiện tùy thuộc vào trạng thái. Tất cả finallycác khối cần thiết sẽ được thực hiện trong Disposephương thức.

Theo đặc tả ngôn ngữ C #, foreach (V v in x) embedded-statementđược mở rộng thành

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

vì vậy, điều tra viên sẽ được xử lý ngay cả khi bạn thoát khỏi vòng lặp với breakhoặc return.

Để có thêm thông tin về việc thực hiện iterator, bạn có thể đọc bài viết này của Jon Skeet

Biên tập

Vấn đề với cách tiếp cận như vậy là bạn dựa vào các khách hàng của lớp để sử dụng phương thức này một cách chính xác. Ví dụ, họ có thể lấy điều tra viên trực tiếp và lặp qua nó trong một whilevòng lặp mà không cần xử lý nó.

Bạn nên xem xét một trong những giải pháp được đề xuất bởi @BenAaronson.

Bất kể yieldhành vi cụ thể , mã của bạn chứa các đường dẫn thực thi sẽ không xử lý tài nguyên của bạn một cách chính xác. Điều gì nếu dòng thứ hai của bạn ném một ngoại lệ hoặc thứ ba của bạn? Hay thậm chí là thứ tư của bạn? Bạn sẽ cần một chuỗi thử / cuối cùng rất phức tạp hoặc bạn có thể sử dụng usingcác khối.

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

Mọi người đã nhận xét rằng điều này không trực quan, rằng mọi người gọi phương thức của bạn có thể không biết rằng việc liệt kê kết quả hai lần sẽ gọi phương thức đó hai lần. Chà, may mắn khó khăn. Đó là cách ngôn ngữ hoạt động. Đó là tín hiệu IEnumerable<T>gửi đi. Có một lý do này không trở lại List<T>hoặc T[]. Những người không biết điều này cần phải được giáo dục, không được làm việc xung quanh.

Visual Studio có một tính năng gọi là phân tích mã tĩnh. Bạn có thể sử dụng nó để tìm hiểu xem bạn đã xử lý tài nguyên đúng cách chưa.

Những gì bạn đã làm có vẻ sai, nhưng sẽ làm việc tốt.

Bạn nên sử dụng IEnumerator <> , vì nó cũng thừa hưởng IDis Dùng một lần .

Nhưng bởi vì bạn đang sử dụng một foreach, trình biên dịch vẫn đang sử dụng IDis Dùng để tạo một IEnumerator <> cho foreach.

trong thực tế, các bài giảng liên quan đến rất nhiều điều.

Kiểm tra /programming/13459447/do-i-need-to-consider-disposeing-of-any-ienumerablet-i-use