Ошибка при чтении из файла в си.Что делать?

Question

[Chemodan228]

#include <stdio.h>
#include <stdlib.h>
int main() {
    FILE *in;
    int temp,sum=0;
    if ((in=fopen("input.txt","r"))==NULL){
        puts("ERROR");
        return 1;
    }
    while(!feof(in)){
        fscanf(in,"%i",&temp);
        printf("%i ",temp);
    }
    fclose(in);

    return 0;
}
выводит:
12 13 45 1 12 1 1

input.txt:
12 13 45 1 12 1
Подскажите почему дублируется последний символ?

Answer 1

[CityCat4]

Не надо использовать эту [beep] конструкцию while(!feof())
От нее только проблемы. Используйте fscanf() в цикле, пока он не вернет EOF.

Да, я знаю, что в примерах так - я сам когда-то писал по таким же примерам и потом долго матерился, не понимая, почему не работает...

Comments

[Mercury13]

Насколько мне известно, fscanf не возвращает EOF. Возвращает 0.

[CityCat4]

Mercury13,

On success, these functions return the number of input items successfully matched and assigned; this can be fewer than provided for, or even zero, in the event of an early matching failure.

The value EOF is returned if the end of input is reached before either the first successful conversion
or a matching failure occurs. EOF is also returned if a read error occurs, in which case the error
indicator for the stream (see ferror(3)) is set, and errno is set to indicate the error.

Можно для гарантии проверять на возврат нуля, конечно же. Мне вот все эти танцы с потоковым чтением видятся осмысленными только для файлов гигантского размера типа сквидовых логов. Файлы осмысленного размера проще читать через read (ну или read-sscanf - для ленивых)

Answer 2

[Mercury13]

feof возвращает true, когда мы попытались прочитать и не смогли, потому что каретка ударилась в конец файла. А не когда каретка аккуратно припарковалась в конце файла — и уж тем более не когда она отделена от конца пробелами.

Лучше прочитайте про scanf…

On success, the function returns the number of items of the argument list successfully filled. This count can match the expected number of items or be less (even zero) due to a matching failure, a reading error, or the reach of the end-of-file.

…и посмотрите, как можно переделать условие цикла.

Answer 3

[Василий Мельников]

1. Стоит проверять что возвращает функция чтения fscanf.
Она явно не смогла прочитать число из потока.
У тебя в файле после последней единицы что-то есть типа возврата строки, пробела или чего-то подобного.

2. Уже к чистоте кода держите переменные ближе к месту использования. temp сразу перед fscanf и ещё и чем нибудь инициализировать. Тогда как минимум в подобных случаях будет не последнее значение, а "маркер"

Comments

[Mercury13]

В Си вроде до 99 переменные можно держать только в начале функции.

[Василий Мельников]

Mercury13, Насколько я помню можно в начале составного оператора. Т.е. открывающей фигурной скобки. Но не суть. пункт 2 это просто рекомендация