Как реализовать проверку на вводимые символы?

Question

[Олег Филимоненко]

Есть участок кода, в котором я ожидаю от пользователя подтверждения в виде 'Y'/'y' или 'N'/'n'. Перепробовал кучу вариантов, от getch до scanf() совместно с fflush(stdio) в разных циклах и их комбинациях. И всегда программа частично работает, но с подвохом. В частности: в нижеприведенной реализации я хоть и добился того, что проверка реагирует на 'Y'/'y' или 'N'/'n', но делать она это может, например, и при вот таком вводе: "polniyBred". (наткнулась на y в 6 позиции). Честное слово, я прекрасно понимаю, что проблема во мне, что я не совсем соображаю уже в 2 часа ночи, что я не знаю и 10% функций стандартной библиотеки. Поэтому прошу сжалиться и показать, как канонично пишутся такого рода проверки и пользовательские диалоги, избегая по возможности костылей и изоленты. Вы же все сталкивались с такими программами, которые не позволяют вам ввести чушь.

Код

void usrSettings(struct usrSettings_str **usrSet){
    char consent;

    printf("Before changing the settings, read the command manual .help\n");
    printf("Are you sure? (Y/y(for 'Yes') or N/n(for 'No' - setting by default)\n");

    scanf("%c", &consent);
    while(consent != 'Y' && consent != 'y' && consent != 'N' && consent != 'n'){
        printf("Enter Y/y or N/n\n");
        printf("> ");
        scanf("%c", &consent);
    }

    if (consent == ('Y' || 'y'));{
        //Edit somethings settings
        printf("User enter Yes\n");
    }
    
    if (consent == ('N' || 'n')){
        //Return to main menu
        printf("User enter NO\n");
    }
}

В общих чертах, я понимаю, что решение должно быть банальным, но чем дольше я копаю, тем сильнее зарываюсь. Вот, например, scanf() отработала в цикле 4 раза (видать, брала символы из буферизированного ввода), до тех пор, пока не повстречала 'y'. (скриншот 3).

Скриншот 1 (Казалось бы...)

Скриншот 2 (Вот тут начинается)

Скриншот 3 (Полный абзац!)

Comments

[wisgest]

if (consent == ('Y' || 'y'));{
        //Edit somethings settings
        printf("User enter Yes\n");
    }

Точка с запятой после условия приводит к тому, что у if пустое тело, а следующий оператор сам по себе.
В условии, вероятно, имелось в виду что-то вроде consent == 'Y' || consent == 'y'.

что я не совсем соображаю уже в 2 часа ночи

Ну так проспался бы и подумал на трезвую голову, а не засорял Интернет.

Answer 1

[CityCat4]

Использовать fgets() чтобы забрать весь юзерский ввод (чтобы избежать повторного чтения).

#define BUFLEN 80
char *str;

str = (char *) calloc(BUFLEN, 1);

    printf("Before changing the settings, read the command manual .help\n");
    printf("Are you sure? (Y/y(for 'Yes') or N/n(for 'No' - setting by default)\n");

for(;;)
 {
    fgets(str, BUFLEN, stdin);
    *(str + 1) = '\0';
    
    if ((*str == 'Y') || (*str = 'y'))
      {
         // Делаем что-то
        break;
      }
    else
       {
          // Сообщаем юзеру, что он чудак, снова даем приглашение на ввод
          bzero(str, BUFLEN);
       }
}

free(str);

Это если нужно разбирать только ситуацию "y" - да, все остальное - нет. Если нужно разобрать 'y' - да, 'n' - что-то еще, остальное - юзер чудак - добавить еще один if после проверки на 'y'

Comments

[Олег Филимоненко]

Во-первых, да, Ваш код заработал и отлично себя ведет. Но мой неугомонный ум полез искать еще и вот что нарыл:

Пример неплохого варианта

char userGetLine[256]; // строка для чтения
    char confirm; // итоговое целое число

    printf("%s", msg); // выводим приглашение ко вводу
    fgets(userGetLine, sizeof(userGetLine), stdin); // считываем строку

    // пока не будет считано целое число
    while ((sscanf(userGetLine, "%c", &confirm) != 'Y') || (sscanf(userGetLine, "%c", &confirm) != 'y') || (sscanf(userGetLine, "%c", &confirm) != 'N') || (sscanf(userGetLine, "%c", &confirm) != 'n')   ) {
        printf("Incorrect input. Try again: "); // выводим сообщение об ошибке
        fgets(userGetLine, sizeof(userGetLine), stdin); // и заново считываем строку
    }

    // Делаю что-то дальше со своими настройками.

Идея в том, что мы также берем кусок стандартного буфера и банально втюхиваем его с форматированный ввод. И если юзер написал мусор, то scanf просто не подтвердит принятие и вернет 0 (или 1, не суть. Не True, короче).
Что можете сказать по этому поводу. Вы бы таким пользовались? Если нет, то почему?

P.S - Я не совсем уверен в булевых операциях, приведенных в проверке условия. По правилам де-Моргана вроде все должно работать, но это пресловутое '!=' и иже с ним, имеют, на сколько я помню, высший приоритет. Если уж совсем не в тягость, прикиньте в уме, все ли верно, Пожалуйста. Хорошего вечера.

[CityCat4]

Олег Филимоненко, Не надо забивать микроскопом гвозди. Нам нужен только первый байт пользовательского ввода, на остальные можно забить. У Вас для вычисления условия (в котором проверяется один байт) используется четыре sscanf - отнюдь не быстрой операции форматного ввода, в то время, как все делается за два if. Десяток таких проверок - и программа начинает тормозить на пустом месте...

Я всегда стараюсь исключить "дорогой" потоковый ввод-вывод, использую его только там, где нужно пять-шесть параметров подставить с печатью цифр например. Весь анализ можно делать в памяти. Можно было в принципе и без fgets() обойтись, считать строку через read(), но я никогда stdin не пробовал так открывать :)

[Олег Филимоненко]

CityCat4, Принял, понял, закрепил! Когда окончательно во всем разберусь - оберну в отдельный обособленный работающий кусок кода, как назидательный пример и выложу сюда.

[Олег Филимоненко]

CityCat4, и пока я ищу самостоятельно, рискну задать вопрос сюда: У меня для перехода в режим настоек предусмотрена команда .settings, но вот в чем беда: и в Вашем прототипе и в моем, костыльном, какое-то говно остается в буфере ввода stdin. Да, в Win функция fflush(stdin) отрабатывает на ура, но не в .nix-системах. Вот чего сейчас точно не хочется - переводить консоль в другие режимы. Есть велосипед - для очистки буфера применяют зацикленное считывание символов, пока не \n (Выгляело как-то так: scanf("%*[^\n]"), но это же совсем, простите...

[Karpion]

Олег Филимоненко, Какой-то жуткий код.

char userGetLine[256]; // строка для чтения Буфер фиксированного размера? А если строка будет больше?

fgets(answer, sizeof(answer), stdin); // считываем строку

Откуда тут answer? Что это за переменная?

while ((sscanf(userGetLine, "%c", &confirm) != 'Y') || (sscanf(userGetLine, "%c", &confirm) != 'y') || (sscanf(userGetLine, "%c", &confirm) != 'N') || (sscanf(userGetLine, "%c", &confirm) != 'n') )

Это вообще ужас. Поскольку строка ввода у нас в массиве userGetLine - нам не надо делать sscanf, достаточно взять соответствующий символ (элемент массива).

Если есть сомнения в приоритете операций - не жалейте скобок. И даже если нет сомнений - всё равно не жалейте. Считайте это заповедью.

___

CityCat4, read() не работает с stdin, он читает через file-descriptor, а не через file-pointer. Из stdin читает fread()! Вообще- имеет смысл научиться работать с этими функциями. Надо помнить, что read() делает сист.вызов в ядро поэтому с короткими запросами (где-то до одного килобайта) работает дольше; зато на длинных вызовах, когда надо читать много (мегабайты) - не заморачивается буферизацией и работает быстрее, чем fread(). Ну и можно/нужно разбирать ввод самостоятельно.

[Олег Филимоненко]

В общем, Ваше решение самое элегантное. Спасибо огромное!

[Олег Филимоненко]

Олег Филимоненко, что получилось:

- Не принимает больше 1 символа
- Не принимает 2 и более симвоов, начинающихся с удовлетворяющих нас символов Y y N n (прим: yo / nutella) (доп. к п.1)
- Не принимает ничего кроме символов (цифры, знаки и тд, что логично)

Непонятки:
- (лично мой косяк где-то выше по коду) Почему-то буфер остается не пустым, но это из за того, что в настройки я вхожу командой, интерпретируемой как %s, буду играться с длиной массива.
- При проверке на 2 и более символов, позицию байта с символом конца строки пришлось сдвинуть на 2, что странно(((

Рабочий вариант

void usrSettings(struct usrSettings_str **usrSet){

int c;                       // Очистка буфера stdin пока не нашел варианта умнее. А проблема в том, что
do {                         //в буфере остается какой-то хлам, который я не увижу и не уловлю, пока не научусь 
c = getchar();          //пользоваться отладчиком
} while (c != '\n' && c != EOF);

char *str;

str = (char *) calloc(BUFLEN, 1);
    printf("Before changing the settings, read the command manual .help\n");
    printf("Are you sure? (Y/y(for 'Yes') or N/n(for 'No' - setting by default/using last changes)\n");
    bzero(str, BUFLEN);
    printf("settings > ");

for(;;)
 {
    fgets(str, BUFLEN, stdin);
    if (*(str + 2) != '\0')    //Вот тут дичайший костыль. Сделано для проверки "не введено ли несколько
                                     //символов", потому что до этого момента при вводе "yo, common" давало true.
                                     //и работает только со смещенеим 2. Было бы не плохо порассуждать, почему так?
    {
        printf("No, only one symbol!\n");
        printf("settings > ");
        bzero(str, BUFLEN);
        continue;
    }
    
    *(str + 1) = '\0';
    
    if ((*str == 'Y') || (*str == 'y'))
      {
        printf("User set Yes\n");
        break;
      }
      if ((*str == 'N') || (*str == 'n'))
      {
          //Exit to parent function
        break;
      }
    else 
       {
          printf("Stop! You must enter 'Y'/'y' for \"Yes\" or 'N'/'n' for \"No\"\n");
          printf("settings > ");
          bzero(str, BUFLEN);
       }
}

free(str);
}

[CityCat4]

Karpion, Да мне как-то особо не приходилось интерактивку писать на С, даже консольную, я все больше по пакетной обработке :) Надо будет - разберусь :)

[CityCat4]

Олег Филимоненко, Для проверки того, не ввел ли юзер больше одного символа, если таковая нужна - сделать

if (strlen(str) > 1)
  // Написать юзеру что он чудак

Еще.
bzero после calloc не нужен - буфер и так занулен - в этом и фича calloc.
Скобки неоднозначны где проверяется на N. Куда компилер отнесет последнее else (где сообщение об ошибке)? А куда захочет, туда и отнесет. Если уж делать обработку Y и N (хотя я всегда подхожу из принципа - Y - это да, а любая шляпа, кроме Y - это нет), то тогда я бы вот так сделал

for(;;)
 {
    fgets(str, BUFLEN, stdin);
   if (strlen(str) > 1)
    {
        printf("No, only one symbol!\n");
        printf("settings > ");
        bzero(str, BUFLEN);
        continue;
    }
    
    *(str + 1) = '\0';
    
    if ((*str == 'Y') || (*str == 'y'))
      {
        printf("User set Yes\n");
        break;
      }
     else
      if ((*str == 'N') || (*str == 'n'))
      {
          //Exit to parent function
        break;
      }
    else 
       {
          printf("Stop! You must enter 'Y'/'y' for \"Yes\" or 'N'/'n' for \"No\"\n");
          printf("settings > ");
          bzero(str, BUFLEN);
       }
}

Answer 2

[Karpion]

1) Не видно текст программы - работа программы закрывает его.

2) Вместо scanf("%c",&consent); лучше использовать consent=grtchar();

3) Для данной ситуации сравнения с буквами - хорошо бы делать consent=grtchar()|32; Тогда надо сравнивать с маленькой буквой.

4) consent надо делать int, а не char - там м.б. EOF.

5) Я бы заменил while на do..while или на repeat..until, оно тут логичнее, ибо условие можно проверить только после первого выполнения тела цикла - Вы же написали его дважды, что создаёт возможность мерзких ошибок по невнимательности.

6) Изучите операторы break (выход из тела цикла) и continue (переход к следующей итерации). Тогда оба ифа вносим в тело цикла, дописав к ним break - и не надо дублировать условия.

7) А конкретно у Вас ошибка в условии while - там только заглавные буквы. Третий пункт решает эту проблему.

Comments

[Олег Филимоненко]

Огромное спасибо за проявленное внимание и потраченное время. Я учту все Ваши советы.