Question how to have invalid slot be staying just in one spot.

[jollyholly11]

For my lab I am trying to have my SINGLE_CHIP, MULTIPLE_CHIPS, and MULTIPLE_CHIPS_ALL just do the string "INVALID SLOT, please try again". However when I run the code, and input a invalid number it says "INVALID SLOT, please try again", but it also goes back to the main menu. I want it to just do invalid slot and have the second option for the slots reappear. Please help!
Here is my code so far.

 #include <iostream>
#include <cmath>
#include <string>
#include <cstdlib>
#include <iomanip>
#include <ctime>
#include <math.h>
#include <stdio.h>

using namespace std;

//Prize Money Function 
double get_prize_money(double location)
{

	int slotnumber = 0;// declare some variables
	int chipnumber = 0;
	double reward = 0;
	const double PRZ0 = 100;
	const double PRZ1 = 500;
	const double PRZ2 = 1000;
	const double PRZ3 = 0;
	const double PRZ4 = 10000;
	const double PRZ5 = 0;
	const double PRZ6 = 1000;
	const double PRZ7 = 500;
	const double PRZ8 = 100;
	const int PRICE_PRECISION = 2;
	const int PATH_PRECISION = 1;
	const int RIGHT_WALL = 8;
	const int LEFT_WALL = 0;
	const int SLOT_1 = 1;
	const int SLOT_2 = 2;
	const int SLOT_3 = 3;
	const int SLOT_4 = 4;
	const int SLOT_5 = 5;
	const int SLOT_6 = 6;
	const int SLOT_7 = 7;


	if (location == LEFT_WALL) { reward = PRZ0; }// if statements to determine winnings
	else if (location == SLOT_1) { reward = PRZ1; }
	else if (location == SLOT_2) { reward = PRZ2; }
	else if (location == SLOT_3) { reward = PRZ3; }
	else if (location == SLOT_4) { reward = PRZ4; }
	else if (location == SLOT_5) { reward = PRZ5; }
	else if (location == SLOT_6) { reward = PRZ6; }
	else if (location == SLOT_7) { reward = PRZ7; }
	else if (location == RIGHT_WALL) { reward = PRZ8; }

	return reward;
}


//Single Chip Function
double single_chip(int slotnumber, int chipnumber)
{
	double location = slotnumber;
	const int PATH_PRECISION = 1;
	const double MOVE = 0.5;
	const int RIGHT_WALL = 8;
	const int LEFT_WALL = 0;
	const int BOARD_SIZE = 12;

	if (slotnumber >= 0 && slotnumber <= 8)
	{
		if (chipnumber == 1)
		{
			cout << fixed << setprecision(PATH_PRECISION) << location;
		}
		for (int i = 0; i < BOARD_SIZE; i++)
		{
			double random = rand() % 2;
			if (random < 1)
			{
				location = location + MOVE;
			}
			else
			{
				location = location - MOVE;
			}
			if (location < LEFT_WALL)
			{
				location = LEFT_WALL + MOVE;
			}
			else if (location > RIGHT_WALL)
			{
				location = RIGHT_WALL - MOVE;
			}
			if (chipnumber == 1)
			{
				cout << " " << location;
			}
		}
		//cout << "]" << endl;


	}
	return location;//return final position
}

double single_chip_no_path(int slotnumber)
{
	double location = slotnumber;
	const double MOVE = 0.5;
	const int RIGHT_WALL = 8;
	const int LEFT_WALL = 0;
	const int BOARD_SIZE = 12;

	if (slotnumber >= LEFT_WALL && slotnumber <= RIGHT_WALL)
	{
		for (int i = 0; i < BOARD_SIZE; i++)
		{
			double random = rand() % 2;
			if (random < 1)
			{
				location = location + MOVE;
			}
			else
			{
				location = location - MOVE;
			}
			if (location < LEFT_WALL)
			{
				location = LEFT_WALL + MOVE; ;
			}
			else if (location > RIGHT_WALL)
			{
				location = RIGHT_WALL - MOVE;
			}
		}
	}
	return get_prize_money(location);//return final position
}


//Mulitple Chip Function
void multiple_chips(int slotnumber, int chipnumber)
{
	double winnings = 0;
	double totalwinnings = 0;
	const int RIGHT_WALL = 8;
	const int LEFT_WALL = 0;


	for (int i = 0; i < chipnumber; i++)
	{
		winnings = single_chip_no_path(slotnumber);
		totalwinnings = totalwinnings + winnings;
	}

	double average = totalwinnings / chipnumber;
	cout << endl << "Slot" << slotnumber << endl;
	cout << "Average winnings per Chip: $" << average << endl;
	cout << "Total winnings: $" << fixed << setprecision(2) << totalwinnings << endl;

}

//Multiple Chip Function in all Slots

void all_slots(int chipnumber)
{
	const int RIGHT_WALL = 8;
	const int LEFT_WALL = 0;

	for (int i = LEFT_WALL; i <= RIGHT_WALL; i++)
	{
		multiple_chips(i, chipnumber);
		//cout << "Slot " << slot_number << endl;
	}
}

//Print Menu function
void print_menu()
{
	cout << "Welcome to the Plinko simulator!" << endl;
	cout << "Please enter one of the following options to continue: \n\tDrop One (1)\n\tDrop Multiple (2)\n\tDrop Multiple into all the slots (3)\n\tQuit (0): ";


}




//Main Function 
const int RIGHT_WALL = 8;
const int LEFT_WALL = 0;
const int PRICE_PRECISION = 2;
const int PATH_PRECISION = 1;
const int QUIT = 0;
const int SINGLE_CHIP = 1;
const int MULTIPLE_CHIPS = 2;
const int MULTIPLE_CHIPS_ALL = 3;
const int BLASTED_ONE_THOUSAND = 1000;

int main()
{
	int menu = 1;
	bool done = false;
	int menuoption = 0;
	int slotnumber = 0;// declare some variables
	int chipnumber = 0;
	const int RIGHT_WALL = 8;
	const int LEFT_WALL = 0;
	double reward = 0;
	double location = 0;
	srand(time(0));


	while (!done)
	{
		//cout << "Please enter one of the following options to continue: \n\tDrop One (1)\n\tDrop Multiple (2)\n\tDrop Multiple into all the slots (3)\n\tQuit (0): ";
		print_menu();
		cin >> menuoption;
		if ( menuoption == SINGLE_CHIP || menuoption == MULTIPLE_CHIPS || menuoption == MULTIPLE_CHIPS_ALL || menuoption == QUIT )
		{
			//cin.clear();
			//cin.ignore(1000, '\n');
			//cout << "INVALID SLOT, please try again. " << endl << endl;
			
			if (menuoption == QUIT) {
				cout << "Goodbye and Good Riddance!" << endl;
				done = true;

			}
			else if (menuoption == SINGLE_CHIP)// single chip
			{
				chipnumber = 1;
				cout << "Select a slot (0-8): ";
				cin >> slotnumber;
				if (slotnumber < LEFT_WALL || slotnumber > RIGHT_WALL || cin.fail())
				{
					cin.clear();
					cin.ignore(BLASTED_ONE_THOUSAND, '\n');
					cout << "INVALID SLOT, please try again. " << endl << endl;
				}

				if (slotnumber >= LEFT_WALL && slotnumber <= RIGHT_WALL) //for loop to simulate random falling of chip
				{
					cout << "[";
					location = single_chip(slotnumber, chipnumber);
					cout << fixed << setprecision(PRICE_PRECISION) << "]" << endl << "Winnings: $" << get_prize_money(location) << endl;
					cout << endl << endl << endl;

				}

			}
			else if (menuoption == MULTIPLE_CHIPS)// multiple chips options
			{
				cout << "Please enter the number of chips to drop: ";//multiple chips input
				cin >> chipnumber;
				if (chipnumber < LEFT_WALL || cin.fail())
				{
					cin.clear();
					cin.ignore(BLASTED_ONE_THOUSAND, '\n');
					cout << "INVALID SLOT, please try again. " << endl << endl;

				}

				if (chipnumber > LEFT_WALL)
				{
					cout << "Select a slot (0-8): ";
					cin >> slotnumber;

					if (slotnumber < LEFT_WALL || slotnumber > RIGHT_WALL || cin.fail())
					{
						cin.clear();
						cin.ignore(BLASTED_ONE_THOUSAND, '\n');
						cout << "INVALID SLOT, please try again. " << endl << endl;

					}
					else if (slotnumber >= LEFT_WALL && slotnumber <= RIGHT_WALL)
					{
						multiple_chips(slotnumber, chipnumber);
						cout << endl << endl << endl;
					}
				}
			}

			else if (menuoption == MULTIPLE_CHIPS_ALL)// multiple chips into all the slots
			{
				cout << "Please enter the number of chips to drop: ";//multiple chips input
				cin >> chipnumber;
			if (cin.fail())
				{
					cin.clear();
					cin.ignore(BLASTED_ONE_THOUSAND, '\n');
					cout << "INVALID SLOT, please try again. " << endl << endl;

				}
				else if (chipnumber < LEFT_WALL)
				{
					cin.clear();
					cin.ignore(BLASTED_ONE_THOUSAND, '\n');
					cout << "INVALID SLOT, please try again. " << endl << endl;

				}
				else all_slots(chipnumber);
				cout << endl;
			}
		}

		else if (menuoption != SINGLE_CHIP && menuoption != MULTIPLE_CHIPS && menuoption != MULTIPLE_CHIPS_ALL && menuoption != QUIT || cin.fail())
		{
			cin.clear();
			cin.ignore(BLASTED_ONE_THOUSAND, '\n');
			cout << "INVALID INPUT, Please enter 0, 1, 2, 3. " << endl << endl;
		}
		
	}
	system("pause");
	return(0);
}

[coder777]

I suggest that you put the processing of each menu option in its own function. That makes it easier to keep the overview. Further more you should try to avoid code repetition with functions.

For your problem: You need an inner loop within the menu option processing like the while (!done) loop just with another flag.

[jollyholly11]

Thank you for your help! It really helped.