Почему считается что неправильно писать while (!input_stream.eof())?
1,00
р.
р.
В разных источниках говорят что использование 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. } <br>Так что если и вызывать 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.