Invite en toute sécurité pour oui / non avec CIN

Je vois deux "problèmes" ici. Le premier est l'utilisation de temps (vrai). Je ne pense pas que ce soit une bonne pratique (bien que cela soit probablement une question de goût pour de nombreuses personnes). La rupture à l'intérieur d'une boucle est cependant intéressante pour la recherche:

int main()
{
    int ch;
    std::string type;

    do {
        getline( std::cin, type );

        if ( cin.fail() ) {
            ch = EOF;
            break;
        }

        ch = tolower( type[ 0 ] );
    } while( ch != 'y' && ch != 'n' );

    // interesting things here...

    return 0;
}

L'entrée basée sur la ligne ne doit pas nécessairement être verbose, vous pouvez le faire succinct , avec une seule fonction que vous écrivez une fois, qui gère toujours les cas d'angle: xxx

Modification de ce que @mcaden a dit, essayez ceci corrige le bogue où si vous entrez plusieurs caractères, le test ne vérifie que la première lettre.

    char type;
    char buffer[128];
    do
    {
      cout << "Were you admitted? [y/n]" << endl;
      cin >> buffer;
      type = buffer[0];
      cout << type << "\n";
    }while( !cin.fail() && type!='y' && type!='n' );

Voici une autre façon de demander à Y / N. "Alors que" vous n'obtenez pas ce que vous voulez, "commutateur" sur votre entrée. Faire tout ce que vous aimez avec elle; Cela fonctionne simplement.

#include "stdafx.h"
#include <iostream>

int main()
{
    bool play = true; // Initialize play to true.
    char choice;
    while (play) // While play is true, do the following:
    {
        std::cout << "Play again? [y/n] -> ";
        std::cin >> choice;
        switch (choice)
        {
        case 'n':           // We can fall through here because we
            play = false;   // don't do anything with 'y' anyway.
        case 'y':           // Remember; play is already true unless
            break;          // we changed it to false with 'n'.
        default:  // We'll simply assume anything else is a fail!
            std::cin.clear();
            std::cin.ignore(1024, '\n');
            break;
        }
        // Break out here and check the "while" condition again...
    }
    // When play becomes false, we exit the while statement.
}

Un autre moyen de demander à Y / N. "Faites" cela "alors que" vous n'obtenez pas ce que vous voulez. Faire tout ce que vous aimez avec elle; Cela fonctionne simplement.

#include "stdafx.h"
#include <iostream>

int main()
{
    bool accepted;
    char answer;
    do
    { // Ask for 'y' or 'n' at least once
        std::cout << "Accepted? [y/n] -> ";
        std::cin >> answer;
        if (std::cin.fail())
        {                                // not a valid character?
            std::cin.clear();
            std::cin.ignore(1024, '\n');
        }
        if (answer == 'n')
        {                                // character is 'n'?
            accepted = false;
        }
        else
        {                                // character is 'y'?
            accepted = true;
        }
         //    not valid    |or|    not 'n'     |or|    not 'y'
    } while (std::cin.fail() || !(answer == 'n') || !(answer == 'y'));  
}

Voici un chemin plus court

char type;
while (type != 'y') 
{
    cout << "Were you admitted? [y/n]" << endl;
    cin >> type;   
}

char yn;
bool choice;
do
    {
           
        cout<<"Would you like try again?\n";
        cout<<"Y for yes N for no: ";
        cin>>yn;
    
        while (yn != 'n'&& yn != 'N'&&yn != 'y'&&yn != 'Y')
        {
            cout<<"Y or N please: ";
            cin>>yn;
        }
        
        if (yn == 'n'||yn == 'N')
            choice = false;
        if(yn == 'y'||yn == 'Y')
            choice = true;
        
    } while (choice == true);