Problem with [Y/N] option in console.

[Y4kuzi]

Hi guys,

I am not sure how to describe this problem, so I'll just keep it simple.
In my console app, users can choose if they want to exit the program or not by entering 'Y' or 'N'.
This works fine, but whenever the user enters more characters, it starts to act weird.
For example, if a user enters 'hello' where he/she should've put 'Y' or 'N', the output is as follows:

Would you like to exit the program? [N/Y]
Option: hello
Invalid option - please enter a valid value. Variable holds: h
Would you like to exit the program? [N/Y]
Option: Invalid option - please enter a valid value. Variable holds: e
Would you like to exit the program? [N/Y]
Option: Invalid option - please enter a valid value. Variable holds: l
Would you like to exit the program? [N/Y]
Option: Invalid option - please enter a valid value. Variable holds: l
Would you like to exit the program? [N/Y]
Option: Invalid option - please enter a valid value. Variable holds: o
Would you like to exit the program? [N/Y]
Option:

The code is as follows:

	end:
	std::cout << "Would you like to exit the program? [N/Y]\n";
	char exit;
	std::cout << "Option: ";
	std::cin >> exit;
	if (exit == 'y') { return 0; }
	else if (exit == 'n') { goto keyoption; }
	else {
	std::cout << "Invalid option - please enter a valid value. Variable holds: " << exit << "\n";
	goto end;

Any suggestions on how to fix this?
Thanks in advance.

[Moschops]

Clear the input before using it.

http://www.cplusplus.com/forum/general/11070/
http://stackoverflow.com/questions/257091/how-do-i-flush-the-cin-buffer

[jimas13]

first of all why do you use goto?
a simple do-while, or even while, loop should do the job perfectly.

char exit;
do{
cin>>exit;
if(exit!='y')
cout<<"wrong option"<<endl;
}while(exit=='y')

and if its a "no" i advise making another subroutine and linking it to the main programm inside the loop.

[Y4kuzi]

Thank you guys so much, a simple std::cin.sync(); did the trick.

@jimas13; I am using goto because I just started C++ last night, I know very little about it so far. Thanks for the advice, tho.

[Moschops]

What are you learning C++ from that teaches goto instead of loops?

[Y4kuzi]

I used goto since I started scripting in mIRC. But I am familiar with while, I just never used do-while before.

[S G H]

For security reasons, do not ever use goto's in C++.

[Y4kuzi]

Thanks, I will remember that.
I will try to find another way to jump to other parts of the code.

[Moschops]

The most common way of jumping to other parts of code are "functions". We also use loops ( for, while, and do-while ) and branches (if, switch).