Почему считается что неправильно писать while (!input_stream.eof())?

В разных источниках говорят что использование std::istream::eof() - это признак плохого кода и что в частности неправильно писать:
while (!input_stream.eof()) { input_stream >> value; process(value); }
Что в этом коде не так? Как писать правильно?

Ответ

Проблема std::istream::eof() в том, что он выставляется только после выполнения какой-либо операции чтения. Поэтому происходит следующее:
std::ifstream input_stream("empty_file.txt"); // Открываем пустой файл if (!input_stream.eof()) { // eof() == false, т.к. мы еще ничего не читали int value; input_stream >> value; // Пытаемся читать число, а его там нет. // Здесь eof() == true, но мы это не проверяем. std::cout << value; // Выведется 0. }
Так что если и вызывать eof(), то это надо делать после операции чтения.
Однако, объекты std::istream умеют преобразовываться в bool, а каждая операция чтения возвращает ссылку на std::istream. Поэтому идиоматичный код выглядит следующим образом:
while (input_stream >> value) { process(value); }
Или для строки:
std::string str; while (std::getline(input_stream, str)) { ... }
Или сразу для нескольких значений:
while (input_stream >> value1 >> value2) { ... }
Здесь если при чтении value1 произойдет ошибка, то все последующие операции чтения (т.е. value2) будут игнорироваться, по этому можно читать сразу несколько значений сразу, и уже потом проверять состояние std::istream