ループ条件でiostream::eofを使うのが間違いとされる理由とは？

C++でファイルを読み込む際、while(!file.eof()) のようなEOFチェックをループ条件に使うのは避けるべきだとよく言われます。その最大の理由はシンプルです。まだEOFに達していないからといって、次の読み込みが成功するとは限らないからです。

なぜeof()チェックは問題なのか

ifstreamなどを使ってファイルを読み込む場合、stream.eof()が判定しているのは「すでにEOFに達したかどうか」という事実だけです。つまり、「次の読み込みが成功するかどうか」を予測するものではありません。

eof()フラグは、実際に読み込みを試みて末尾を超えた後に初めてセットされます。そのため、while(!myFile.eof())と書くと、最後のデータを読み終えた時点ではまだeof()がfalseであるため、ループにもう一回入り込んでしまい、必ず失敗する読み込みを追加で実行することになります。この失敗した読み込みの結果までプログラマが再度チェックしなければならず、コードが冗長になり、バグの温床にもなります。

悪い例：eof()をループ条件に使う書き方

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

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

    while(!myFile.eof()) {
        myFile >> x;      // 最後の周回で必ず失敗する読み込みが発生する
        if(myFile) {      // 失敗していないか、ここでもう一度チェックが必要
            // x を使った処理
        }
    }
}

この書き方では、ループ内で読み込みの成否を二重にチェックする必要があり、無駄な反復も発生します。

良い例：ストリームを直接条件に使う書き方

一方、抽出演算子>>はストリーム自身への参照を返します。C++11以降では、ストリームは明示的なbool変換を持つため、whileの条件式に直接置くことで「その読み込みが成功したか」をその場で判定できます。

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

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

    while(myFile >> x) {
        // x を使った処理
        // 追加のチェックは一切不要！
    }
}

この方法なら、読み込みが失敗した瞬間（EOF到達時も含む）にループが自動的に終了するため、余計な反復や二重チェックがまったく発生しません。

まとめ

• eof()は「過去にEOFへ達したか」を示すだけで、将来の読み込み成否は保証しない。
• while(!file.eof())は最後に必ず失敗する読み込みを含み、エラー処理が複雑になる。
• while(file >> x)のようにストリーム自体を評価すれば、読み込みの成否を正確かつ簡潔に判定できる。

ファイルの終端まで読み込む処理を書くときは、EOFを事前に予測しようとせず、読み込み操作そのものをループ条件にするのがC++の定石です。