Simple cin question.

I have a menu in a program I am writing and the user inputs an integer to select from the menu. I am trying to catch any input from the user that is not an integer and have the user select again. I just don't know how to check if the input is a non-integer. For example if the user inputs a character on accident then it should catch it.

Code:

int sel;
cout<<"1: Do This"<<endl; 
cout<<"2: Do this"<<endl;
cout<<"Selection?";cin>>sel;

Thanks For The help

Basically, you need to use a loop. Note that if the user enters invalid input, cin >> sel fails and cin enters an error state. So you can test if the read failed, and if so, you clear the error state, discard what is left on the input stream, (optionally) print an error message, and then print the menu and get the user's selection again.

An example of this would be:

Code:

int sel;
for (;;)
{
    cout << "1: Do This" << endl;
    cout << "2: Do this" << endl;
    cout << "Selection? ";
    if (!(cin >> sel))
    {
        cin.clear();
        cin.ignore(numeric_limits<std::streamsize>::max(), '\n');
        cout << "Invalid selection. Please try again.\n";
    }
    else
    {
        cin.ignore(numeric_limits<std::streamsize>::max(), '\n');
        break;
    }
}

You could also use a do while loop, perhaps with a function.

EDIT:
Incidentally, it occurred to me that my original example does not take into account valid followed by invalid input on the same line. You have a choice in such situations: either read the entire line as a string, parse it, and then report an error if invalid input is found at all, or simply accept the valid portion, then discard the invalid portion. This is what I have done with my corrected example with another cin.ignore() call before the break.

Thanks a lot for your help =).

>> my original example does not take into account valid followed by invalid input on the same line.

Another solution that handles this and still considers it an error is:

Code:

int sel;
for (;;)
{
    cout << "1: Do This" << endl;
    cout << "2: Do this" << endl;
    cout << "Selection? ";
    if (!(cin >> sel) || cin.get() != '\n')
    {
        cin.clear();
        cin.ignore(numeric_limits<std::streamsize>::max(), '\n');
        cout << "Invalid selection. Please try again.\n";
    }
    else
    {
        break;
    }
}

The extra code ignores the trailing newline if the read was successful, but if there's garbage after the int it shows the error and asks the user to try again. This will stop the user from entering something like 1.2 when the input should be an integer.