판독 루프를 종료하는 방법은 선호되는 방법입니까?

읽을 항목 수를 알 수없는 판독기를 반복해야하는 경우 끝까지 닿을 때까지 계속 읽으십시오.

이것은 종종 끝없는 루프가 필요한 곳입니다.

1. 블록 안에 어딘가에 또는 문 true이 있어야 함을 나타내는 항상 있습니다 .breakreturn

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

2. 이 더블 루프 방법 읽기.

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

3. 단일 읽기 while 루프가 있습니다. 모든 언어가이 방법을 지원하지는 않습니다 .

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

어떤 접근 방식이 가장 읽기 쉽고 일반적으로 사용되는 버그를 도입 할 가능성이 가장 적은지 궁금합니다.

이 중 하나를 작성해야 할 때마다 "더 나은 방법이 있어야 한다 " 고 생각 합니다.

여기서 물러서겠습니다 코드의 까다로운 세부 사항에 집중하고 있지만 더 큰 그림이 누락되었습니다. 예제 루프 중 하나를 살펴 보겠습니다.

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

이 코드 의 의미 는 무엇입니까 ? 의미는 "파일의 각 레코드에 일부 작업을 수행합니다"입니다. 그러나 그것은 코드의 모습 이 아닙니다 . 코드는 "오프셋을 유지합니다. 파일을 엽니 다. 종료 조건없이 루프를 입력하십시오. 레코드를 읽습니다. null을 테스트하십시오."와 같은 코드입니다. 우리가 일하기 전에 모든 것! " 이 코드의 모양이 의미와 일치하도록하려면 어떻게 해야 합니까? "입니다.이 코드는 다음과 같아야합니다.

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

이제 코드는 의도 한대로 읽습니다. 의미론에서 메커니즘을 분리하십시오 . 원래 코드에서는 매커니즘 (루프의 세부 사항)과 시맨틱 (semantics)을 각 레코드에 대해 수행 한 작업을 혼합합니다.

이제 구현해야합니다 RecordsFromFile(). 그것을 구현하는 가장 좋은 방법은 무엇입니까? 무슨 상관이야? 그것은 누군가가 볼 코드가 아닙니다. 기본 메커니즘 코드이며 10 줄 길이입니다. 그러나 원하는대로 작성하십시오. 이건 어때?

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

이제 우리는 느리게 계산 된 레코드 시퀀스를 조작하여 모든 종류의 시나리오가 가능해졌습니다.

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

등등.

루프를 작성할 때마다 "이 루프를 메커니즘이나 코드의 의미처럼 읽습니까?" 대답이 "메커니즘과 같다"면 해당 메카니즘을 자체 메서드로 옮기고 의미를 더 잘 보이도록 코드를 작성하십시오.

무한 루프가 필요하지 않습니다. C # 읽기 시나리오에서는 필요하지 않습니다. 오프셋을 유지해야한다고 가정하면 이것이 내가 선호하는 접근법입니다.

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

이 방법은 독자를위한 설정 단계가 있다는 사실을 인정하므로 두 가지 읽기 메소드 호출이 있습니다. while조건 대비해 리더에 전혀 데이터가없는, 루프의 상부에있다.

글쎄, 그것은 당신의 상황에 달려 있습니다. 그러나 제가 생각할 수있는 "C # -ish"솔루션 중 하나는 내장 IEnumerable 인터페이스와 foreach 루프를 사용하는 것입니다. IEnumerator 인터페이스는 true 또는 false로 MoveNext 만 호출하므로 크기를 알 수 없습니다. 그런 다음 종료 논리가 열거 자에 한 번 작성되므로 둘 이상의 지점에서 반복 할 필요가 없습니다.

MSDN은 IEnumerator <T> 의 예를 제공합니다 . IEnumerator <T>를 반환하려면 IEnumerable <T>을 만들어야합니다.

초기화, 조건 및 증가 작업이있을 때 C, C ++ 및 C #과 같은 언어의 for 루프를 사용하고 싶습니다. 이처럼 :

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

또는 이것이 더 읽기 쉽다고 생각되면 말입니다. 나는 개인적으로 첫 번째 것을 선호합니다.

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