WHILE and INFINITE LOOPS

[iydh]

Hello guys! I need your help. I keep getting infinite loops when ever my answer is out from Y or N.

#include <iostream>
using namespace std;

int main(){
int number;
char answer= 'Y';


while(answer !='N'){
cout << "Type a number: ";
cin >> number;

if(number <0){
cout << number << " is a negative number." << endl;
}

else
if(number >0){
cout << number << " is a positve number." << endl ;

}

cout << "Repeat? [Y/N]: ";
cin >> answer;
answer = toupper(answer);

if(answer == 'n' || answer == 'N'){
cout << "Good bye~";
}

else
if(answer !='N')
}

[LB]

Please edit your post and make sure your code is [code]between code tags[/code] so that it has line numbers and syntax highlighting, as well as proper indentation.

This code does not compile.

[sasauke]

To fix the infinite loop you could put the cin >> answer; statement in a do..while loop which will loop while answer is different from Y, y, N and n.

I edited your code a little bit and I think this is what you were trying to do.

#include <iostream>
using namespace std;

int main()
{
	int number;
	char answer = 'Y';

	while(answer != 'N' && answer != 'n')
	{
		cout << "Type a number: ";
		cin >> number;

		if(number < 0)
		{
			cout << number << " is a negative number." << endl;
		}
		else if(number > 0)
		{
			cout << number << " is a positve number." << endl ;
		}

		do
		{
			cout << "Repeat? [Y/N]: ";
			cin >> answer;
		} while (answer != 'Y' && answer != 'y' && answer != 'N' && answer != 'n');
		
		
		if(answer == 'n' || answer == 'N')
		{
			cout << "Good bye~";
		}				
	}
	return 0;
}

Also, please surround your code with "code" tags, so we can read it better.