endless loop for some reason

The following basic example is creating an endless loop when I put in anything but 1 or 2, whats going on? I just want it to start the program over again if the inputted value isn't 1 or 2. Thanks in advance.

#include <iostream.h>

void main()

{

bool x;
int cat = 0;

do

{

cout << "Is cat true? 0 for yes, 1 for no.\n\n";
cin >> x;

if (x == 0)
cout << "You are correct!!\n";
else if (x == 1)
cout << "You are not correct \n";
else
cout << "Type in a valid number please.\n";
}

while (x != 0 || x != 1);

}

Change
while (x != 0 || x != 1);
to
while (x != 0 && x != 1);

C:\Documents and Settings\Administrator\Desktop\C++ files\options\asdf.cpp(15) : error C2679: binary '>>' : no operator defined which takes a right-hand operand of type 'bool' (or there is no acceptable conversion)
Error executing cl.exe.

Thats what it says for the cin statement. What should I do?

Change your declaration of x from
bool x;
to
int x;

It works, but now when I type something like "sdafsda" or any character, it says its correct. How do I get back to saying its not correct if anything but 0 and 1 are typed? Modified code:

#include <iostream.h>

void main()

{

int x;
int cat = 0;

do

{

cout << "Is cat true? 0 for yes, 1 for no.\n\n";
cin >> x;

if (x == 0)
cout << "You are correct!!\n";
else if (x == 1)
cout << "You are not correct \n";
else
cout << "Type in a valid number please.\n";
}

while ((x != 0) && x (!= 1));

}

Maybe you can change x to type char and alter your if statements to compare characters instead of numbers:

Seriously though, if you're just starting to learn C++, you should try learning other concepts before coming to input validation stuff. Also, while you're at it, void main() violates ISO C++ standards (should return int, according to the standard). Check out this link and this link for more details as to why this is so.

Thanks for your help so far man. But I can't seem to take your suggestions and make my code work. Everything works perfectly unless the user types in a letter. When they type in a letter the program loops endlessly. AHHHH Here is my new code:


#include <iostream>

int main()
{
using namespace std;


int x = 1;
int y;

do
{
cout << "Is x 1 or 2? (If you type anything else then 1 or 2 the program will repeat until a valid value is entered.\n";
cin >> y;
cout << "\n";
if (y == 1)
cout << "Correct! X is one\n";
else if (y==2)
cout << "You are wrong!\n";
}

while ((y !=1 && y !=2));
return 0;
}

ok, i understand why that is happening to u, but exactly how to fix it is tricky. i used to have same problem last year in my cs class. input validation is extremely hard to get correct, at least from my experience. what u need, is a loop inside the do loop that checks the input stream to make sure it is valid. when cin expects an integer, and recieves a character, it sets a flag:
cin.rdstate() to 2, the value of cin.rdstate() is 0 normally. so perhaps u could create a loop using cin.rdstate()?? if ur not familiar with that, have u ever used cin.good(). it is ezier to use than the above mentioned, someting like this:

cin >> y;
while(!cin.good())
{
cin.clear();
cin >> y;
}

hope that helps u out

Hi,

I'm not sure why your loop goes crazy when the user enters a letter instead of the expected integer, but you can validate your input very easily if you use a char type for the input from the user because a char literal can be anything:


In the rest of your program, your if statement conditions should test for char literals because the char variable "answer", in the case where the user inputs 1, stores the ASCII code that corresponds to 1 which is 49, so "answer" will not equal 1 but it will equal '1'.

if(answer == '1') cout<<"You are right!";

Alternatively, if for some reason you need an integer as the input, after validating the user's input, you can convert it to an integer like this:

int int_answer=0;
if(answer == '1') int_answer = 1;
else int_answer = 2;

Thanks man. It works! I couldn't figure out the whole cin.good thing like that other guy said. But thanks for the suggestions.
-andy