Cách nào để chấm dứt vòng lặp đọc là cách tiếp cận ưa thích?

Khi bạn phải lặp lại một trình đọc mà số lượng mục cần đọc là không xác định, và cách duy nhất để làm là tiếp tục đọc cho đến khi bạn kết thúc.

Đây thường là nơi bạn cần một vòng lặp vô tận.

1. Luôn luôn truechỉ ra rằng phải có một breakhoặc một returncâu lệnh ở đâu đó bên trong khối.

   int offset = 0;
   while(true)
   {
       Record r = Read(offset);
       if(r == null)
       {
           break;
       }
       // do work
       offset++;
   }

2. Có hai lần đọc cho phương thức vòng lặp.

   Record r = Read(0);
   for(int offset = 0; r != null; offset++)
   {
       r = Read(offset);
       if(r != null)
       {
           // do work
       }
   }

3. Có vòng lặp đọc đơn. Không phải tất cả các ngôn ngữ đều hỗ trợ phương pháp này .

   int offset = 0;
   Record r = null;
   while((r = Read(++offset)) != null)
   {
       // do work
   }

Tôi đang tự hỏi phương pháp nào ít có khả năng giới thiệu một lỗi nhất, dễ đọc nhất và thường được sử dụng.

Mỗi lần tôi phải viết một trong những điều này tôi nghĩ rằng "phải có một cách tốt hơn" .

Tôi sẽ lùi một bước ở đây. Bạn đang tập trung vào các chi tiết cầu kỳ của mã nhưng thiếu hình ảnh lớn hơn. Chúng ta hãy xem một trong các vòng lặp ví dụ của bạn:

int offset = 0;
while(true)
{
    Record r = Read(offset);
    if(r == null)
    {
        break;
    }
    // do work
    offset++;
}

Ý nghĩa của mã này là gì? Ý nghĩa là "thực hiện một số công việc cho mỗi bản ghi trong một tệp". Nhưng đó không phải là những gì mã vẻ thích . Mã này trông giống như "duy trì bù. Mở tệp. Nhập vòng lặp không có điều kiện kết thúc. Đọc bản ghi. Kiểm tra tính vô hiệu." Tất cả điều đó trước khi chúng tôi có được công việc! Câu hỏi bạn nên đặt ra là " làm thế nào tôi có thể làm cho diện mạo của mã này khớp với ngữ nghĩa của nó? " Mã này phải là:

foreach(Record record in RecordsFromFile())
    DoWork(record);

Bây giờ mã đọc như ý định của nó. Tách các cơ chế của bạn khỏi ngữ nghĩa của bạn . Trong mã gốc của bạn, bạn trộn lẫn cơ chế - các chi tiết của vòng lặp - với ngữ nghĩa - công việc được thực hiện cho mỗi bản ghi.

Bây giờ chúng ta phải thực hiện RecordsFromFile(). Cách tốt nhất để thực hiện điều đó là gì? Ai quan tâm? Đó không phải là mã mà bất cứ ai sẽ nhìn vào. Đó là mã cơ chế cơ bản và dài mười dòng. Viết nó theo cách bạn muốn. Còn cái này thì sao?

public IEnumerable<Record> RecordsFromFile()
{
    int offset = 0;
    while(true)
    {
        Record record = Read(offset);
        if (record == null) yield break;
        yield return record;
        offset += 1;
    }
}

Bây giờ chúng ta đang thao tác một chuỗi các bản ghi được tính toán một cách lười biếng, tất cả các loại kịch bản đều có thể xảy ra:

foreach(Record record in RecordsFromFile().Take(10))
    DoWork(record);

foreach(Record record in RecordsFromFile().OrderBy(r=>r.LastName))
    DoWork(record);

foreach(Record record in RecordsFromFile().Where(r=>r.City == "London")
    DoWork(record);

Và như thế.

Bất cứ khi nào bạn viết một vòng lặp, hãy tự hỏi mình "vòng lặp này có đọc giống như một cơ chế hay giống như ý nghĩa của mã không?" Nếu câu trả lời là "giống như một cơ chế", thì hãy thử chuyển cơ chế đó sang phương thức riêng của nó và viết mã để làm cho ý nghĩa rõ ràng hơn.

Bạn không cần một vòng lặp vô tận. Bạn không bao giờ cần một trong các kịch bản đọc C #. Đây là cách tiếp cận ưa thích của tôi, giả sử rằng bạn thực sự cần phải duy trì bù đắp:

Record r = Read(0);
offset=1;
while(r != null)
{
    // Do work
    r = Read(offset);
    offset++
}

Cách tiếp cận này thừa nhận thực tế là có một bước thiết lập cho người đọc, vì vậy có hai cuộc gọi phương thức đọc. Điều whilekiện là ở đầu vòng lặp, chỉ trong trường hợp không có dữ liệu nào trong đầu đọc.

Vâng, nó phụ thuộc vào tình hình của bạn. Nhưng một trong những giải pháp "C # -ish" hơn mà tôi có thể nghĩ đến là sử dụng giao diện IEnumerable tích hợp và vòng lặp foreach. Giao diện cho IEnumerator chỉ gọi MoveNext bằng đúng hoặc sai, vì vậy kích thước có thể không xác định. Sau đó, logic chấm dứt của bạn được viết một lần - trong bảng liệt kê - và bạn không phải lặp lại ở nhiều vị trí.

MSDN cung cấp một ví dụ về IEnumerator <T> . Bạn cũng sẽ cần tạo một IEnumerable <T> để trả về IEnumerator <T>.

Khi tôi có các hoạt động khởi tạo, điều kiện và gia tăng, tôi muốn sử dụng các vòng lặp cho các ngôn ngữ như C, C ++ và C #. Như thế này:

for (int offset = 0, Record r = Read(offset); r != null; r = Read(++offset)){
    // loop here!
}

Hoặc điều này nếu bạn nghĩ rằng điều này là dễ đọc hơn. Cá nhân tôi thích cái đầu tiên.

for (int offset = 0, Record r = Read(offset); r != null; offset++, r = Read(offset)){
    // loop here!
}