fgets не работает после scanf

scanf() читает именно то, что вы просили, оставляя следующую \n с конца этой строки в буфере, где ее будет читать fgets(). Либо сделайте что-нибудь, чтобы использовать новую строку, либо (мое предпочтительное решение) fgets(), а затем sscanf() из этой строки.

scanf оставляет пробелы во входном буфере, включая символы новой строки. Чтобы использовать fgets для чтения следующей строки, вам нужно вручную удалить оставшуюся часть текущей строки:

int c;
do{
    c = getchar();
}while(c != EOF && c != '\n');

Это более простое решение

scanf("%d",&loops);
while ((getchar()) != '\n'); //This will consume the '\n' char
//now you're free to use fgets
fgets(string,sizeof(string),stdin);

Вайн,

Geekoaur хорошо ответил на ваш вопрос, я просто указываю на еще одну «проблему» с вашим кодом.

Строка s1[strlen(s1)] = '\0'; является no-op, если s1 уже корректно завершается нулем ДО того, как она будет выполнена.

Но если s1 НЕ уже правильно завершается нулем ДО того, как эта строка будет выполнена (и вам не повезло), это вызовет:

• SIGSEGV в системе POSIX (*nix).
• GPF в Windows.

Это связано с тем, что strlen в основном находит индекс существующего нулевого терминатора и возвращает его! Вот правильная, неоптимизированная реализация strlen:

int strlen(char *string) {
    int i = 0;
    while(string[i] != '\0') {
        ++i;
    }
    return i;
}

Итак... Если вы ДЕЙСТВИТЕЛЬНО беспокоитесь о том, что строки НЕ заканчиваются нулем, вы должны сделать что-то вроде:

• string[sizeof(string)]='\0'; для локальных автоматических строк (где компилятор "знает" размер строки);
• или string[SIZE_OF_STRING] для всех других строк, где SIZE_OF_STRING (чаще всего) представляет собой константу #define или переменную, которую вы поддерживаете специально для хранения текущего размера (а не длины) динамически выделяемой строки.

И если вы ДЕЙСТВИТЕЛЬНО, ДЕЙСТВИТЕЛЬНО, ДЕЙСТВИТЕЛЬНО беспокоитесь о том, что строки не заканчиваются нулем (например, вы имеете дело с «грязными» библиотечными методами (например, с ATMI Tuxedo), вы ТАКЖЕ «очищаете» свой «вернуть строки» перед передачей их подозрительным библиотечным методам с помощью:

• до:memset(string, NULL, SIZE_OF_STRING);
• вызвать: DirtyFunction(/*out*/string);
• после: string[SIZE_OF_STRING]='\0'

Найти SIG11 очень сложно, потому что (если только вы не «подцепите» их с помощью signal-processor и сказать иначе, они приводят к тому, что Unix принудительно завершает вашу программу, поэтому вы не можете ничего записать (постфактум), чтобы выяснить, где, черт возьми, сделал-это-пришло-от... особенно учитывая, что во многих случаях строка кода, которая выдает SIG11, не является фактической причиной потери строки своего нулевого терминатора.

Это имеет для вас смысл?

Здоровья, приятель. Кейт.

PS: ПРЕДУПРЕЖДЕНИЕ: strncpy НЕ всегда завершается нулем... вы, вероятно, имели в виду strlcpy вместо этого. Я усвоил это на собственном горьком опыте... когда рухнул платеж на 60 миллионов долларов.

ИЗМЕНИТЬ:

К вашему сведению: вот «безопасная» (неоптимизированная) версия strlen, которую я назову strnlen (думаю, она должна быть в stdlib. Эх).

// retuns the length of the string (capped at size-1)
int strnlen(char *string, int size) {
    int i = 0;
    while( i<size && string[i]!='\0' ) {
        ++i;
    }
    return i;
}

Я знаю, что это очень старо. Я новичок в c и хотел проверить свой метод, который использует getchar:

#include <stdio.h>

int main()
{

    printf("Please enter your name\n");
    char string[10];

    scanf("%s", string);
    printf("Hello %s\n", string);

    //getchar();  # un commenting this line, fgets perfectly works!!
    printf("Please enter your name again\n");

    fgets ( string, 10, stdin );     

    printf("Hello again %s", string);

    getchar();
}

просто поставь scanf("%d\n",&loops);

вместо scanf("%d",&loops);

Другой способ игнорировать следующий символ новой строки (из-за нажатия ENTER) после сканирования целого числа в циклах меток переменных:

scanf ("%d%*c", &loops);

Где, согласно справочным страницам:

* Подавляет присвоение. Последующее преобразование происходит как обычно, но без использования указателя; результат преобразования просто отбрасывается.

Это очень маловероятно, но хорошая привычка проверять ошибки во время сканирования:

errno = 0
scanf ("%d%*c", &loops);
if (errno != 0) perror ("scanf");
// act accordingly to avoid un-necessary bug in the code

Например, в вашем коде циклы — это локальная неинициализированная переменная, содержащая мусор. Если scanf не может заполнить желаемое значение, следующий цикл while может выполняться произвольно.

scanf() оставить следующий \n с конца этой строки в буфере, где fgets() будет его читать. Чтобы решить эту проблему, мы должны сделать что-то, чтобы использовать новую строку. Давайте посмотрим на проблему.

Проблема

#include<stdio.h>
void main()
{
    char input[10];
    printf("Enter in fgets: ");
    scanf("%s", input);
    getchar();
    printf("Enter in scanf: ");
    fgets(input, 10, stdin);
}

Enter in scanf: Hello
Enter in fgets:

Как видите, он не показывает часть fgets. Давайте посмотрим решение.

Поместите getchar() между scanf и fgets()

#include<stdio.h>
void main()
{
    char input[10];
    printf("Enter in fgets: ");
    scanf("%s", input);
    getchar();
    printf("Enter in scanf: ");
    fgets(input, 10, stdin);
}

Enter in scanf: Hello
Enter in fgets: Hello

Если возможно, используйте scanf() после fgets()

#include<stdio.h>
void main()
{
    char input[10];
    printf("Enter in fgets: ");
    fgets(input, 10, stdin);
    getchar();
    printf("Enter in scanf: ");
    scanf("%s", input);
}

Enter in fgets: Hello
Enter in scanf: Hello

Поставьте fget() 2 раза (этот метод иногда не работает)

#include<stdio.h>
void main()
{
    char input[10];
    printf("Enter in fgets: ");
    scanf("%s", input);
    getchar();
    printf("Enter in scanf: ");
    fgets(input, 10, stdin);
    fgets(input, 10, stdin)
}

Enter in scanf: Hello
Enter in fgets: Hello

Вставить sscanf() из scanf()

#include<stdio.h>
void main()
{
    char input[10];
    printf("Enter in fgets: ");
    sscanf(hello, "%s", input);
    getchar();
    printf("Enter in scanf: ");
    fgets(input, 10, stdin);
}

Enter in sscanf: Hello
Enter in fgets: Hello

Следующее работает, если fgets() "пропускается" после использования scanf()

Сказав:

scanf("%d", &loops);

Сказать:

char garbage[100];

fgets(garbage,100,stdin);

Это сохранит все, что осталось во входном буфере, в переменную мусора.

Это эффективно очистит входной буфер и позволит вам впоследствии использовать fgets().

РЕДАКТИРОВАТЬ: Недавно я узнал, что есть более простое решение, чем приведенное выше. Если вы скажете getchar() после scanf(), это позволит вам использовать fgets() без проблем. getchar() получит следующий символ из входного буфера, в данном случае это будет '\n'. Как только вы удалите '\n' из входного буфера, fgets должен работать нормально.