Проверка ввода переменной char

Question

Я пытаюсь проверить вход для вопроса quit/return в своей программе, поэтому пользователь вынужден ввести либо 'r', либо 'q'.

Мне удалось заставить его почти работать. Проблема в том, что пользователь вводит 'r' или 'q' в начале, за которым следуют случайные буквы, тогда программа принимает этот ввод. Любые идеи о том, как получить программу, чтобы разрешить только один 'r' или 'q'?

void exit() 
{ 
    char choice; 
    bool badInput; 

    do 
    { 
     cout << "Press 'r' to return to the menu\nPress 'q' to quit the program\n\n" << endl; 
     cin >> choice; 

     badInput = cin.fail(); 
     cin.clear(); 
     cin.ignore(numeric_limits<streamsize>::max(), '\n'); 
    } while (badInput == true && choice == 'r' && choice == 'q' && (cin.peek() == EOF)); 


     if (choice == 'q') 
     { 
      system("CLS"); 
      cout << "Bye!\n"; 
      system("PAUSE"); 
     } 
     else if (choice == 'r') 
     { 
      system("CLS"); 
      main(); 
     } 
     else 
     { 
      exit(); 
     } 
} 

Answer 1

У вас очень странный способ приблизиться к этой проблеме, и у вас много проблем. В частности, создание функции с именем exit() проблематично, так как это основная функция и вызов ее рекурсивно, чтобы попытаться получить вход, также не является хорошим планом. У вас уже есть цикл, вам просто нужно использовать его более эффективно.

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

Вот первый переписывают проход:

void getInput() 
{ 
    char choice; 

    while (true) 
    { 
     cout << "Press 'r' to return to the menu\nPress 'q' to quit the program\n\n" << endl; 

     cin.clear(); 
     cin.ignore(numeric_limits<streamsize>::max(), '\n'); 

     cin >> choice; 

     switch (choice) 
     { 
      case 'q': 
       system("CLS"); 
       cout << "Bye!\n"; 
       system("PAUSE"); 
       exit(0); 
       break; 
      case 'r': 
       system("CLS"); 
       doMain(); 
       break; 
     } 
    } 
} 

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

Я также рекомендую вам прекратить делать такие вещи, как system("CLS") и system("PAUSE"), и вместо этого сделать что-то в C++ изначально. Это не переносимый код, и он ужасно неуклюжим, поскольку это зависит от команд с DOS 1980-х годов.

Answer 2

Обработка входа используйте

Большинство реализации забуференный cin, так что вход будет возвращен только после того, как пользователь нажимает кнопку ввода. Если это неприемлемо для вас, вам придется использовать функции, зависящие от ОС.

Если это нормально для вас, то если вы прочитаете символ, будет передан только первый из них: оставшиеся символы до тех пор, пока ввод не будет ждать последующих чтений. Поэтому, вместо того, чтобы читать один символ, я предлагаю вам прочитать полную строку в строку:

void want_exit() 
{ 
    const string message="Press 'r' to return to the menu\nPress 'q' to quit the program\n"; 
    string line; 

    cout << message << endl; 
    while (getline(cin, line) && line.length()!=1 
      && tolower(line[0])!='y' && tolower(line[0])!='n') 
    { 
     cout << "Invalid input" <<endl << message <<endl; 
     line.resize(0); 
    } 

Теперь строка содержит либо один действительный символ, или пустой (в случае преждевременного ВФ, а это означает, что было перенаправление ввода и что в любом случае не будет больше ввода).

Обработка ввода

Вы не можете называть рекурсивно main(): вместо этого вы должны вернуться из функции, а также организовать вызывающую функцию так, чтобы продолжить процесс

if (line.empty() || tolower(line[0])!='y') { 
     system("CLS"); // This is non portable 
     cout << "Bye!\nPress enter..."; 
     cin.getch(); // as good as system("PAUSE"); 
     std::exit(0); // be aware of the naming issue ! 
    } 
    return; 
} 

Функция вызова (main()?) Затем использовал бы его в петле:

while (...) { 
    ... 
    if (...) 
     want_exit();  
}