Чтение ввода из fgetc() и печать с помощью printf()

Question

Хорошо, я уверен, что кое-что мне здесь не хватает, но я понятия не имею, что это такое, и я надеюсь, что кто-то может помочь мне разобраться.

Я читаю на входе из командной строки и писал функцию, которая использует fgetc() для этого. Однако, казалось бы, поверхностное изменение в функции заставляет его вести себя совершенно по-другому.

Это функция main ():

while(1)
{
     char* cmd = malloc(sizeof(char) * 80);
     printf("Enter command: ");
     read_flush(cmd, 80);
     printf("%s\n", cmd);
     free(cmd);
}

И это одна из версий read_flush ():

int read_flush(char* buffer, int count)
{
    int i, c;
    for(i = 0; i < count; i++)
    {
        c = fgetc(stdin);
        if(c == '\n' || c == EOF)
            return i;
        buffer[i] = (char)c;
    }

    return i;
}

Этот работает отлично. Вы вводите ввод, и он выплевывает его обратно. Однако следующая версия заставляет main просто печатать "Enter command:" снова и снова, не давая пользователю возможности ввести ввод.

int read_flush(char* buffer, int count)
{
    int i, c;
    while(i < count)    
    {
        c = fgetc(stdin);
        if(c == '\n' || c == EOF)
            return i;
        buffer[i] = (char)c;
        i++;
    }

    return i;
}

Какие тонкости с fgetc() я здесь упускаю?

Answer 1

Попробуйте инициализировать i во второй read_flush реализация.

Answer 2

Обе версии имеют одинаковую ошибку. Вы не добавляете NUL в конец строки. malloc не инициализирует память, которую он возвращает.

Answer 3

Во второй версии похоже, что вы не инициализируете i в ноль, как в первой версии. Таким образом, он может начинаться со значения мусора, большего, чем count, и, таким образом, цикл никогда не выполняется.