Вложенные условные операторы в C

Question

Я имею немного проблемы при работе с вложенными условными операторами в С.

int is_correct() { 

    char YN ; 

    printf("Y or N : ") ; 
    scanf("%c", &YN) ; 
    YN = toupper(YN) ; 

    return (YN == 'Y')? 1 : (YN == 'N')? 0 : is_correct() ; 
}  

Я был под впечатлением, что последняя строка кода будет возвращать 1 или 0, если в «Y» или «N» было введено или называть себя снова, если был введен неожиданный символ. Вместо этого он постоянно называет себя независимо от ввода.

Answer 1

Возможно, сканирование неисправно, и вы не проверяете его.

Вы не указываете, что «непрерывно» означает «без остановки, чтобы читать больше ввода», что он, конечно, должен делать.

Заметим, например, что toupper() использует int -typed аргумент и результаты, и ожидающие значения типа unsigned char вы можете ударить неопределенное поведение там.

Это действительно запутанный аспект функций ctype.h. Я склоняюсь к тому, чтобы передать в unsigned char вызов, если данные поступают из текстового (char) буфера.

Добавить printf() позвонить, чтобы распечатать значение YN перед окончательной строкой.

Answer 2

Преобразование %c не считывает символ новой строки, который вы должны вводить после односимвольного ответа.

В общем, scanf() вызывает гораздо больше проблем, чем того стоит. Попробуйте использовать getline(), чтобы прочитать строку из stdin в строку, а затем использовать sscanf() для извлечения символа из строки.

Помимо прочего, getline() позволяет указать максимальную длину ввода, поэтому вы можете легко избежать переполнения буфера строк.

Answer 3

В scanf("%c",&YN) месте пробел перед спецификатором преобразования %c как scanf(" %c",&YN) съедать символ новой строки (\n)

#include <stdio.h> 
#include <ctype.h> 

int is_correct(void) 
{ 
    char YN ; 
    printf("Y or N : ") ; 
    scanf(" %c",&YN); 
    YN = toupper(YN); 

    return YN == 'Y' ? 1 : YN == 'N' ? 0 : is_correct() ; 
}  

int main() 
{ 
    printf("%d",is_correct()); 
    return 0; 
} 

Я проверял. Работает отлично, если вы набираете только один символ (исключая \n)!
Более эффективным способом вы можете это сделать; сохраните первый символ в char ch, а затем используйте петлю while((YN = getchar()) != '\n'), чтобы съесть все остальные символы, включая \n. Пример: Если вы наберете ynabcd, первый символ y будет сохранен в ch как Y, а отдых будет поедать while(.

int is_correct(void) 
{ 
    char YN ; 
    printf("Y or N : ") ; 
    scanf("%c",&YN); 
    char ch = toupper(YN); 
    while((YN = getchar()) != '\n') 
    ; 
    return ch == 'Y' ? 1 : ch == 'N' ? 0 : is_correct() ; 
}