### GAME PROBLEM - TIC TAC TOE

Game doesn't exit after the user chooses to quit. Loop for the game isn't working. Board does not reset when user wishes to try another game.

 ``123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167`` `````` #include #include #include using namespace std; char square[10] = {'o','1','2','3','4','5','6','7','8','9'}; int checkwin(); void board(); void resetBoard(); int main() { int menu; int player = 1,i,choice; char tralse_A, tralse_B, mark; do{ system ("cls"); cout << "\n\t\t--- TIC TAC TOE GAME ---"; cout << "\n\t\t --- M E N U ---"; cout << "\n\t\t [1] Start Game"; cout << "\n\t\t [2] Quit"; cout << "\n\t\t Enter choice: "; cin >> menu; switch(menu) { case 1 : do { board(); player=(player%2)?1:2; cout << "\t\t\tPlayer " << player << ", Enter a number: "; cin >> choice; mark=(player == 1) ? 'X' : 'O'; if (choice == 1 && square[1] == '1') square[1] = mark; else if (choice == 2 && square[2] == '2') square[2] = mark; else if (choice == 3 && square[3] == '3') square[3] = mark; else if (choice == 4 && square[4] == '4') square[4] = mark; else if (choice == 5 && square[5] == '5') square[5] = mark; else if (choice == 6 && square[6] == '6') square[6] = mark; else if (choice == 7 && square[7] == '7') square[7] = mark; else if (choice == 8 && square[8] == '8') square[8] = mark; else if (choice == 9 && square[9] == '9') square[9] = mark; else { cout<<"Invalid move "; player--; getch(); } i=checkwin(); player++; }while(i==-1); board(); do { if(i == 1) { cout<<"\t\t==> \aPlayer "<<--player<<" win "; } else { cout<<"\t\t==> \aGame draw"; getch(); } cout << "\n\t\t Do you want to play again? [y/n]: "; cin >> tralse_A; }while (tolower(tralse_A)=='y'); resetBoard(); case 2 : tralse_A = 0; tralse_B = 0; cout << "\n\t\t Thank you for playing! Goodbye... ^^"; } }while (tolower(tralse_A)!='n'); cin.ignore(); cin.get(); return 0; } int checkwin() { if (square[1] == square[2] && square[2] == square[3]) return 1; else if (square[4] == square[5] && square[5] == square[6]) return 1; else if (square[7] == square[8] && square[8] == square[9]) return 1; else if (square[1] == square[4] && square[4] == square[7]) return 1; else if (square[2] == square[5] && square[5] == square[8]) return 1; else if (square[3] == square[6] && square[6] == square[9]) return 1; else if (square[1] == square[5] && square[5] == square[9]) return 1; else if (square[3] == square[5] && square[5] == square[7]) return 1; else if (square[1] != '1' && square[2] != '2' && square[3] != '3' && square[4] != '4' && square[5] != '5' && square[6] != '6' && square[7] != '7' && square[8] != '8' && square[9] != '9') return 0; else return -1; } void board() { system("cls"); cout << "\n\n\t\t\t\t Tic Tac Toe\n\n"; cout << "\t\t\tPlayer 1 (X) - Player 2 (O)" << endl << endl; cout << endl; cout << "\t\t\t | | " << endl; cout << "\t\t\t " << square[1] << " | " << square[2] << " | " << square[3] << endl; cout << "\t\t\t _____|_____|_____" << endl; cout << "\t\t\t | | " << endl; cout << "\t\t\t " << square[4] << " | " << square[5] << " | " << square[6] << endl; cout << "\t\t\t _____|_____|_____" << endl; cout << "\t\t\t | | " << endl; cout << "\t\t\t " << square[7] << " | " << square[8] << " | " << square[9] << endl; cout << "\t\t\t | | " << endl << endl; } void resetBoard() { for (int i(0); i < 10; i++) square[i] = '0'+i; }``````
Your code would be so much easier to read if you would use proper indentation. Can you edit your code to use proper indentation? Looking at this is a nightmare...

 ``1234`` ``````case 2 : tralse_A = 'n'; // Set value to 'n' so while loop check return false //tralse_B = 0; // Remove this because you don't use it cout << "\n\t\t Thank you for playing! Goodbye... ^^";``````
Topic archived. No new replies allowed.