Input validation loop in C++ with strings

Question

I'm just learning C++ (1 week of experience) and was trying to write an input validation loop that ask the user to enter "Yes" or "No". I figured it out but have a feeling there's a better way of approaching this. Here's what I came up with:

{
    char temp[5]; // to store the input in a string
    int test; // to be tested in the while loop

    cout << "Yes or No\n";
    cin.getline(temp, 5);

    if (!(strcmp(temp, "Yes")) || !(strcmp(temp, "No")))    // checks the string if says Yes or No
        cout << "Acceptable input";                         // displays if string is indeed Yes or No
    else                                                    //if not, intiate input validation loop
    {
        test = 0;
        while (test == 0) // loop
        {
            cout << "Invalid, try again.\n";
            cin.getline(temp, 5);               // attempts to get Yes or No again
            if (!(strcmp(temp, "Yes")) || !(strcmp(temp, "No")))  // checks the string if says Yes or No
                test = 1;         // changes test to 1 so that the loop is canceled
            else test = 0;        // keeps test at 0 so that the loop iterates and ask for a valid input again
        }
        cout << "Acceptable input";
    }

    cin.ignore();
    cin.get();

    return 0;
}

I apologize for my poor notes, not sure what is relevant. I'm also using the cstring header.

Do not use the cstring header unless you must. #include <string> is a much better solution. — hrnt
– hrnt, Commented Mar 28, 2011 at 8:08
Since you're working in C++, look into using the std::string class. — Cody Gray
– Cody Gray ♦, Commented Mar 28, 2011 at 8:08

Community · Accepted Answer · 2017-05-23 11:55:25Z

5

Even better IMO:

std::string answer;

for(;;) {
    std::cout << "Please, type Yes or No\n";
    getline(std::cin, answer);

    if (answer == "Yes" || answer == "No") break;
}

You also can transform answer into lower case that allows user to type not only "Yes", but also "yes", "yEs", etc. See this question

edited May 23, 2017 at 11:55

CommunityBot

11 silver badge

answered Mar 28, 2011 at 8:27

maverik

5,6565 gold badges38 silver badges55 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

johnsyweb Over a year ago

Using std::string is certainly better as not only is it C++, it reduces the risk of buffer overflow and it makes string comparisons easier to read. However, this answer does not allow "no" (and its case variants) as valid input.

ChrisWue · Accepted Answer · 2011-03-28 08:06:39Z

0

I think you want a do while loop:

bool test = false;
do 
{
   cout << "Yes or No\n";
   cin.getline(temp, 5);
   if (!(strcmp(temp, "Yes")) || !(strcmp(temp, "No"))) // checks the string if says Yes or No
   {
       cout << "Acceptable input"; // displays if string is indeed Yes or No
       test = true;
   }
   else
   {
       cout << "Invalid, try again.\n";
   }

} while (!test);

answered Mar 28, 2011 at 8:06

ChrisWue

19.1k4 gold badges62 silver badges86 bronze badges

1 Comment

ChrisWue Over a year ago

It was more meant to show the transformation into a do {} while loop.

Collectives™ on Stack Overflow

Input validation loop in C++ with strings

2 Answers 2

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related