I need assistance with the last bug in my program.

[hhjkkh10]

In my program i am trying to only get input (variable guess) for one letter, and if the user does input more than one letter, it will be ignored. How on earth do i do this as at the moment it just causes the whole iteration of my while loop to be ran! Thanks for the help in advance!

Here is the code:

#include <iostream>
#include <array>
#include <time.h>
#include <cctype>
#include <string>

using namespace std;


int main()
{
	bool restart = true;
	while (restart == true) {
		srand(time(0));

		array <string, 20>listOfWords =
		{
			"phenomenal",
			"insane",
			"haunting",
			"immaculate",
			"duck",
			"read",
			"onomatopoeia",
			"eloquent",
			"ridiculous",
			"jurassic",
			"veni vidi vici",
			"who you gonna call",
			"ghostbusters",
			"tremendous",
			"variable",
			"the game",
			"blue",
			"cuckoo",
			"danny devito",
			"oscar"
		};

		int randomWordNumber = rand() % 20;
		string guessWord = listOfWords[randomWordNumber];

		array <char, 20>guesses;
		guesses.fill('_');

		for (int count = 0; count < guessWord.length(); count++)
		{
			if (guessWord[count] == ' ')
			{
				guesses[count] = ' ';
			}
		}

		for (int count = guessWord.length(); count < 20; count++)
		{
			guesses[count] = ' ';
		}

		bool running = true;
		int lives = 5;

		array <char, 20>lettersGuessed;
		lettersGuessed.fill(' ');

		char guess = 0;															
		
		array <char, 20>guess_history;and gave values to.
		guess_history.fill(' ');

		int y = 0;									
		bool trigger = false;
		int m = 0;
		char randnumber;
		int iteration = 1;
		bool allow = true;
		bool trigger1 = true;
		bool trigger2 = true;
		string restart1;


		cout << "Welcome to hangman!\nThe aim of the game is to guess letters in a word.\nEach underscore represents a letter!" << endl;   // Line of code that explains the game to the user
		for (int x = 0; x <= guessWord.length(); x++)
		{																	// this for loop prints out how many underscores there are for every letter
			cout << guesses[x] << " ";
		}
		cout << endl << "You have 5 lives to start off with!" << endl;      
		cout << "PLEASE ONLY ENTER ONE LETTER AT A TIME!" << endl;	

		while (running == true)
		{

			trigger1 = true;
			allow = true;
			bool test = false;
			trigger = false;
			bool winFlag = false;

			if (iteration == 20) {
				goto label2;
			}


			if (iteration > 1 && isalpha(guess)) {

				guess_history[y] = guess;
				y++;

			}
			for (int x = 0; x < guessWord.length(); x++) {
				if (iteration > 1 && isdigit(guess) || ispunct(guess) && guess_history[x] != guess)
				{



				}
			}

			if (iteration > 1 && lives >= 1 && winFlag == false) {

				cout << "You have guessed the letters: ";

				while (trigger == false)									
				{

					cout << guess_history[m] << ", ";
					m++;
					if (guess_history[m] == ' ')
					{
						trigger = true;

					}

				}

			}


			m = 0;
			cout << "\nPlease enter your guess : \n";
			cin >> guess;

			guess = tolower(guess);	
			cout << endl;

			for (int x = 0; x < 20; x++)
			{
				if (guess == guess_history[x])
				{
					cout << "You have already guessed that! Enter a different letter please!\n";
					cout << "\nPlease press enter to type in your next guess!\n";

					


					while (trigger1 == true) {
						
						cin.get();
						if (cin.get() == '\n') {
							system("cls");
							trigger1 = false;

						}
						else
						{
							cout << "Please press the ENTER key\n";
						}
					}


					goto label;

				}


			}



			if (isalpha(guess))        // only executes if the input data for guess is a letter, i used isalpha as it makes my code more clear and readable compared to an if statement or switch statement
			{
				for (int x = 0; x < guessWord.length(); x++) {
					if (guessWord[x] == guess)
					{
						for (int i = 0; i <= guessWord.length(); i++)
						{

							if (guessWord[i] == guess)
							{

								guesses[i] = guess;											// A piece of code that prints out an underscore that represents each letter but now each letter the user guesses correct replaces the corresponding undersore in its place
							}
						}
						for (int x = 0; x <= guessWord.length() && allow == true; x++)
						{
							cout << guesses[x] << " ";
						}

						allow = false;
						test = true;

					}
				}

				if (allow == false) {
					cout << endl << endl << "Correct!" << endl << endl;       // To let the user know that the letter they guessed is correct
					cout << "You have " << lives << " lives remaining!\n" << endl;  // Code shows how many lives the user has left every guess
				}
				static int counter = 0;

				if (test == false)
				{

					for (int x = 0; x <= guessWord.length(); x++)
					{
						cout << guesses[x] << " ";
					}

					lives--;
					lettersGuessed[counter] = guess;
					counter++;
					cout << "\n\nWrong!" << " You have " << lives << " lives remaining!" << endl;				// Code shows how many lives the user has left after every guess
				}
				else
				{
					lettersGuessed[counter] = guess;
					counter++;

				}

				if (lives < 1)
				{

					cout << "YOU LOSE! The word was '";					// I changed the output to make it more user freindly, it also makes more sense that printing out LOSE
					for (int x = 0; x <= guessWord.length(); x++)       // I made the program print out what the word was as if the user loses they won't know
					{
						cout << guessWord[x];					// This for loop prints out every element in the array next to each other
					}
					cout << "'" << endl;
					cin.get();
					running = false;
					restart = false;
				}

				bool winFlag = true;

				for (int count = 0; count < guessWord.length(); count++)
				{
					if (guesses[count] != guessWord[count])
					{
						winFlag = false;
					}

				}

				if (winFlag == true)
				{
					cout << endl << endl << endl << "CONGRATULATIONS YOU WIN! The word was '"; // I changed what was printed to make it more user friendly.

					for (int x = 0; x <= guessWord.length(); x++)       // I made the program print out what the word was
					{
						cout << guessWord[x];					// This for loop prints out every element in the array next to each other
					}
					cout << "'" << endl;

					cin.get();
					running = false;
					restart = false;

				}
				if (lives >= 1 && winFlag == false)
				{
					cout << "\nPlease press enter to type in your next guess!\n";

					while (trigger1 == true) {
						// I used this function as it only requires the user to input one button instead of two making it easier and quicker to use
						cin.get();
						if (cin.get() == '\n') {
							system("cls");
							trigger1 = false;

						}
						else
						{
							cout << "Please press the ENTER key\n";
						}
					}


				}

			}

			else
			{

				if (lives >= 1 && winFlag == false) {
					cout << "PLEASE ONLY ENTER A LETTER! EXAMPLE: h  " << endl << "Please try again!" << endl;  //Reminds the user of what they have to input
					cout << "\nPlease press enter to type in your next guess!\n";

					while (trigger1 == true) {
						// I used this function as it only requires the user to input one button instead of two making it easier and quicker to use
						cin.get();
						if (cin.get() == '\n') {
							system("cls");
							trigger1 = false;
						}
						else
						{
							cout << "Please press the ENTER key\n";
						}
					}
				}

			}
			label2:
			if (iteration == 20) {
				cout << "You have broken the program! Please type restart!" << endl;
				while (trigger2 == true) {
					cin >> restart1;
					if (restart1 == "restart" || restart1 == "RESTART") {
						running = false;
						restart = true;
						trigger2 = false;
						system("cls");
					}
					else {
						cout << "Please type the word restart!" << endl;
					}
				}
			}


		label:
			iteration++;

		}


	}
		return 1;
	
}

[coder777]

After line 140 you should remove all extra characters from the stream. For this use ignore():

http://www.cplusplus.com/reference/istream/istream/ignore/

The best value tor the number of characters to ignore (see Parameters) would be numeric_limits<streamsize>::max()

[hhjkkh10]

Could you maybe give me a few lines of code i can use as i am quite new to c++ and still don't fully understand

[coder777]

cin >> guess;
cin.ignore(numeric_limits<streamsize>::max(), '\n'); // Note: #include <limits> 

'\n' (new line character) is placed in the stream when the user presses enter.

[hhjkkh10]

Thanks for the solution but now i have another problem. Whenever i ask the user to press enter, they have to enter any kind of input data and then press the enter key?

[gunnerfunner]

if the user does input more than one letter, it will be ignored

Instead of cin>> you can use getline(cin, line) where line is std::string and then check it's size to decide whether or not to accept the input