Menu problem.

[And3rs]

Hello.

creating a simple menu so the player can insert what difficulty he would like to play at.

int main()
{
	char choice = 'y';
	do
	{
	enum difficulty{Easy = 1, Medium, Hard};
	cout << "\t\tWelcome to my Menu.\n\n";
	cout << "1 - Easy\n";
	cout << "2 - Normal\n";
	cout << "3 - Hard\n\n";
	

	int choose;
	cout << "Choice: ";
	cin >> choose;

	switch(choose)
	{
	case 1:
		cout << "You picked the difficulty: Easy";
		break;
	case 2:
		cout << "You picked the difficulty: Normal";
		break;
	case 3:
		cout << "You picked the difficulty: Hard";
	            break;
	default:
		cout << " You can only choose from 1-3, try again? (Y/N)\n: ";
		cin >> choice;
		system("cls");
	
	}
	}
	while(choice == 'y');

	cout << "Okey then, bye.";

		getch();
        return 0;
}

How come the program reloops the program in an instant after I have picked 1-3?
works like it should do when I pick 4.

[tabstop]

Well, in case 1, it prints the difficulty line, and then ... well, that's it; that's all you put in the case, so that's all that happens.

[And3rs]

What did u just say? ^^

If I pick 1, then it will say, You choose the difficulty: Easy
and then the break; should.. break out.

where is the endless looping comming from? ^^

[tabstop]

From the do...while loop? break only breaks out of the switch, it doesn't break out of the outer loop too (you wouldn't want it to, anyway, 99999999 times out of 100000000).

[And3rs]

Well, im out of ideas, can u give me a hint to solve my problem?

I want the program to end if I pick 1-3.

[And3rs]

or not end, display the message, then wait for the getch() and then quit.

[tabstop]

If you don't want it to loop, don't put it in a loop. Why have the do...while if you only want it to happen once? Or, if you want the loop for "not 1-3" case, then your loop condition needs to be based on the value of choose.

[scwizzo]

Of course it works when you choose 4 cause it snaps to the default one. If you choose 1-3 it will display, then you never give the user the option to say Y or N. So... choice is always equal to Y until you choose something other than 1-3.

[And3rs]

ofc. I want the program to loop, otherwise I wouldnt put in a (y/n) question.

but now it seems that everything loops.

[anon]

Just put choice = 'n'; at the beginning of the loop. You only set it to 'y' in the default case and it would loop again.

[And3rs]

Well if I do that, the compiler will most likley whine about that choice have been initialized more than once, or something like that.

Im so bad at fixing errors ^^

[And3rs]

Im so stuck, gah, getting irritated.

[anon]

Assignment is not initialization - there is no such thing as initializing something twice. You could even leave it uninitialized initially and the compiler would still see that you assign it a value two lines down.

[And3rs]

Now when I put in the choice = 'n'; the code works atleast better.

now another problem came up, now is the line cout << "Okey then, bye.";
showing everytime.

[anon]

Yes, too bad now there's no way to tell after the loop whether anything was picked or not. And when you think about it, there's also no way to tell what difficulty you chose, since that variable is also local to the do-loop (should you want to do more than that after selecting difficulty). Perhaps move the choose variable out of the loop too (and rename it to something more descriptive like difficulty) and initialize it to 0. Later you can check if it is still 0 (no difficulty was chosen).