Почему использовать `while (!stream.eof())` в C++ — это ошибка?

Почему `while (!stream.eof())` не работает?

Ваша ошибка — классический пример неправильного использования eof(). Проблема в том, что флаг eof устанавливается только после неудачной попытки чтения, а не до неё. Давайте разберём по шагам, что происходит в вашем цикле:

1. Итерация 1: Файл не закончен, eof() == false. Читаем 10 — успешно. Добавляем 10.
2. Итерация 2: eof() всё ещё false. Читаем 20 — успешно. Добавляем 20.
3. Итерация 3: eof() false. Читаем 30 — успешно. Добавляем 30.
4. Итерация 4: eof() false. Читаем 40 — успешно. Добавляем 40.
5. Итерация 5: eof() всё ещё false! Почему? Потому что мы ещё не пытались читать после последнего числа. Выполняется file >> value — чтение не удаётся (файл закончился), флаг eof устанавливается, value остаётся равным 40 (предыдущее значение). Вы добавляете 40 снова.
6. Итерация 6: eof() == true, цикл завершается.

В результате у вас дублируется последнее значение. Ключевой момент: eof() не предсказывает будущее, он сообщает о прошлом — была ли уже попытка чтения за концом файла.

Правильный подход: проверяйте результат операции чтения

Вместо проверки eof() нужно проверять, удалось ли выполнить само чтение. Самый простой и надёжный способ — использовать сам поток как условие:

#include <iostream>
#include <fstream>
#include <vector>

int main() {
    std::ifstream file("numbers.txt");
    std::vector<int> numbers;
    int value;

    while (file >> value) {  // Попытка чтения и проверка успеха
        numbers.push_back(value);
    }

    for (int n : numbers) {
        std::cout << n << " ";
    }
    return 0;
}

Как это работает: оператор >> возвращает ссылку на поток, а поток неявно преобразуется в bool. Если последняя операция прошла успешно — возвращается true, если нет (конец файла, ошибка формата) — false. Таким образом, вы читаете и сразу проверяете успех чтения.

Когда всё же использовать eof()?

eof() может быть полезен в редких случаях, например:

• Для диагностики: проверить, действительно ли файл закончился, а не произошла другая ошибка.
• В сочетании с другими проверками, если вам нужно различать конец файла и ошибку формата.

Пример корректного использования:

while (file >> value) {
    numbers.push_back(value);
}
if (file.eof()) {
    std::cout << "Файл прочитан полностью\n";
} else if (file.fail()) {
    std::cout << "Ошибка формата данных\n";
}

Золотое правило: проверяйте состояние потока после операции чтения, а не до неё. Это избавит вас от множества трудноуловимых багов.

#include <iostream>
#include <fstream>
#include <vector>
#include <iterator>

int main() {
    std::ifstream file("numbers.txt");
    std::vector<int> numbers;

    // Читаем все int из файла в вектор
    std::copy(
        std::istream_iterator<int>(file),
        std::istream_iterator<int>(),  // Конечный итератор (конец потока)
        std::back_inserter(numbers)
    );

    for (int n : numbers) {
        std::cout << n << " ";
    }
    return 0;
}