C++でループ条件にiostream::eof()を使うのが間違いとされる理由

なぜeof()によるループ制御が問題なのか

ストリームのeof()をループ条件に使うことが好ましくない最大の理由は、「まだEOFに達していない」という状態は「次の読み込みが必ず成功する」ことを意味しないからです。

C++でfstream（ファイルストリーム）を使ってファイルを読み込む際、while文の中でstream.eof()によって終端を判定すると、実際には「ファイルの末尾に到達したかどうか」を確認しているだけにすぎません。

eof()のフラグは、実際にEOFを超えて読み込みを試みた後でなければセットされません。つまり、最後のデータを読み終えた直後の時点ではeof()はまだfalseのままです。その結果、ループがもう一周回ってしまい、失敗した読み込みによって変数が前回の値や空のまま放置されるため、同じデータを二重に処理したり、無効なデータを扱ってしまう恐れがあります。

誤った例：eof()を使ったループ

#include<iostream>
#include<fstream>
using namespace std;

int main() {
   ifstream myFile("myfile.txt");
   string x;

   while(!myFile.eof()) {
      myFile >> x;
      // xが有効かどうか、再びチェックが必要になる
      if(!x.empty()) {
         // xに対して何らかの処理を行う
      }
   }
}

この書き方では、ループ内で読み込みの成否を改めて確認する必要があり、コードが冗長になるだけでなく、チェック漏れによるバグも発生しやすくなります。

正しい例：読み込み操作を直接条件にする

ストリームをループの条件式に直接使えば、追加のチェックは一切不要になります。抽出演算子>>の結果であるストリームオブジェクトはbool値へ変換でき、読み込みが成功したかどうかをそのまま評価できるためです。

#include<iostream>
#include<fstream>
using namespace std;

int main() {
   ifstream myFile("myfile.txt");
   string x;

   while(myFile >> x) {
      // xに対して何らかの処理を行う
      // 追加のチェックは不要！
   }
}

この方法なら、読み込みが失敗した瞬間（EOFに達したときやエラーが発生したとき）に自動的にループを抜けられるため、安全で簡潔なコードになります。C++でファイルを単語や行ごとに読み込む場合は、このイディオムを使うのが定石です。