Play aga`in loop broken

[johndough]

OK, so the program is working minus the play again loop. I even tried a goto statement for it but no luck. Basically what its doing is after the game runs its course it asks if you would like to play again? If you hit Y it starts over but if you it N it starts over. I want it to out put thank you for playing and close after user hits a button.

// C// Guess My Number
// The classic number guessing game

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;
int main()
{
	
    srand(static_cast<unsigned int>(time(0)));  //seed random number generator

	int secretNumberE = rand() % 10 + 1; 
	int secretNumberM = rand() % 100 + 1;// random number between 1 and 100
	int secretNumberH = rand() % 1000 + 1;
	int tries = 0;
	int guess;
    char play;
	cout << "\tWelcome to Guess My Number\n\n";
START:
	cout << "Difficulty Levels\n\n";
    cout << "1 - Easy\n";
    cout << "2 - Normal\n";
    cout << "3 - Hard\n\n";

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

    switch (choice)
{
    case 1:
    cout << "You picked Easy.\n";
	do
	{
		cout << "Enter a guess 1-10: ";
		cin >> guess;
		++tries;

		if (guess > secretNumberE)
		{
			cout << "Too high!\n\n";
		}
		else if (guess < secretNumberE)
		{
			cout << "Too low!\n\n";
		}
		else
		{
			cout << "\nThat's it! You got it in " << tries << " guesses!\n";
		}

	} while (guess != secretNumberE);
	goto BEGIN;
	break;
	case 2:
	cout << "You picked Normal.\n";
	do
	{
		cout << "Enter a guess 1-100: ";
		cin >> guess;
		++tries;

		if (guess > secretNumberM)
		{
			cout << "Too high!\n\n";
		}
		else if (guess < secretNumberM)
		{
			cout << "Too low!\n\n";
		}
		else
		{
			cout << "\nThat's it! You got it in " << tries << " guesses!\n";
		}

	} while (guess != secretNumberM);
	goto BEGIN;
	break;
	case 3:
	cout << "You picked Hard.\n";
do
	{
		cout << "Enter a guess 1-10: ";
		cin >> guess;
		++tries;

		if (guess > secretNumberH)
		{
			cout << "Too high!\n\n";
		}
		else if (guess < secretNumberH)
		{
			cout << "Too low!\n\n";
		}
		else
		{
			cout << "\nThat's it! You got it in " << tries << " guesses!\n";
		}

	} while (guess != secretNumberH);
	goto BEGIN;
	break;
	BEGIN:
		cout << "Would you like to play again? y or n: ";
	cin >> play;
	if ( play == 'y' || 'Y') {
    goto START;
}
	else if ( play == 'n' || 'N') {
	cout << "Thank you for playing ";
	goto END;
}
	default:
	cout << "You made an illegal choice.\n";
	goto START;
	END:
    return 0;
}
}

[Mats]

Take out the goto statements. It's a very bad way to program. This is quite a small program and I'm already finding it f**king annoying to follow the code. Try something like this:

while(play == 'y' || play == 'Y')
{ 
    //Put all the game play code here. 
   std::cout << Do you want to play again y/n?;
   cin >> play;
}

It will make the code much better and easier to read and will also fix your bug.