Tic Tac Toe Winner Checker Проблема - PullRequest
0 голосов
/ 11 ноября 2019

Я относительно новичок в программировании и Java, и для моего класса CS-173 мне было поручено создать игру Tic Tac Toe. Однако, когда дело дошло до создания метода определения победителя, всякий раз, когда я добивался «победы», код никогда не запускался, говоря, что я выиграл. У меня есть код для проверки каждого способа выигрыша, однако я вытащил его из кода, чтобы устранить некоторые личные проблемы. Кроме того, мои извинения за плохой код.

     public static void playGame(char[][] board, int size){
    Scanner input = new Scanner(System.in);

    int turn = 0;
    int spaces = board.length * board.length;
    boolean valid = false;
    boolean winner = false;

    for (int i = 0; i<spaces; i++){
      int startchecker = 3;
      int xcord = 0;
      int ycord = 0;

      do{      
        do{

          System.out.println("Player 1 please type your coordinates with a space");
            xcord = input.nextInt();
            ycord = input.nextInt();


           valid = isValid (board, xcord, ycord);

         if(i >= spaces){

         }   


        }while(!valid);      
          board[xcord][ycord] = 'X';
        printBoard(board);
         winner = isWinner(board);

        do{
            System.out.println("Player 2 please type your coordinates with a space");
            xcord = input.nextInt();
            ycord = input.nextInt();
             valid = isValid (board, xcord, ycord);

            winner = isWinner(board);

          }while(!valid);
         board[xcord][ycord] = 'O';
         printBoard(board);



        if(i >= spaces){
          winner = true;
          System.out.println("It is a tie!");


        } 
      }while(!winner);

      }
    }


    public static boolean isWinner (char[][] board){
     boolean determiner = false;
     int XCounter = 0;
     int OCounter = 0;
     int size = board.length-1;
     int winner = 3;


   //Check Horizontal
    for(int j = 0; j > size; j++){
         for(int i = 0; i > size; i++){
          if(board[i][j]=='X'){
               XCounter++;

              }
          else if(board[i][j]=='O'){
               OCounter++;

           }
             if(XCounter == winner){
                determiner = true;
                 System.out.println("Player 1 Wins!");
             }
              else if(OCounter == winner){
                System.out.println("Player 2 Wins!");
                determiner = true;
              }
          }
    }

    return determiner; 
    } 

1 Ответ

0 голосов
/ 11 ноября 2019

Ваш метод isWinner не проверяет все способы выигрыша.

Я бы порекомендовал использовать 2 цикла for (один для горизонтальных линий и другой для вертикальных) для ясности и 2 if-операторы (вне циклов) для проверки диагоналей.

Например,

for(int i=0; i<size; i++){
    boolean flag = true; // Assume this line is a winning line
    for(int j=0; j<size; j++){ // Check each tile to see if it has a tile
        // Set the flag to false when it is not the tile you're looking for
    }
}
...