Tic Tac Toe (Условный)

Question

По какой-то причине, когда я запрограммировал getWinner(), он работает только в 2 случаях (для последней строки). У меня есть все остальное, что касается диагоналей и столбцов, но строка 2 (ну, три, но массивы, так что 2) работает только с o. x только выигрывают, когда o находятся в 0,0 и 1,1 или 0,2 и 1,1. (По существу, только если Выходов находятся в верхнем углу и в центре.

import java.util.Scanner; 
public class TicTacToeRunner 
{ 
    public static void main(String[] args) 
    { 
     Scanner in = new Scanner(System.in); 
     String player = "x"; 
     TicTacToe game = new TicTacToe(); 
     boolean done = false; 
     while (!done) 
     { 
      System.out.print(game.toString()); 
      System.out.print( 
      "Row for " + player + " (-1 to exit): "); 
      int row = in.nextInt(); 
      if (row < 0) 
      { 
      done = true; 
      } 
      else if (row > 2) 
      System.out.println("Invalid, please enter a number from 0 to 2"); 
      else 
      { 
       System.out.print("Column for " + player + ": "); 
       int temp = in.nextInt(); 
       if (temp>2) 
       System.out.println("Invalid, please enter a number from 0 to 2"); 
       else 
       { 
       int column = temp; 
       game.set(row, column, player); 
       if (player.equals("x")) 
       player = "o"; 
       else 
       player = "x"; 
      } 


      if(game.getWinner().equals("x")) 
      {  
       System.out.println("x is the winner!"); 
       done = true; 
      } 
      if(game.getWinner().equals("o")) 
      {  
       System.out.println("o is the winner!"); 
       done = true; 
      } 
      } 
      } 
    } 

}

public class TicTacToe 
{ 
    private String[][] board; 
    private static final int ROWS = 3; 
    private static final int COLUMNS = 3; 
    public TicTacToe() 
    { 
     board = new String[ROWS][COLUMNS]; 
     for (int i = 0; i < ROWS; i++) 
     for (int j = 0; j < COLUMNS; j++) 
      board[i][j] = " "; 
    } 
    public void set(int i, int j, String player) 
    { 
     if (board[i][j].equals(" ")) 
      board[i][j] = player; 
      //what if the spot is filled??? 
    } 
    public String toString() 
    { 
     String r = ""; 
     for (int i = 0; i < ROWS; i++) 
     { 
      r = r + "|"; 
      for (int j = 0; j < COLUMNS; j++) 

       r = r + board[i][j]; 
       r = r + "|\n"; 
     } 
     return r; 
    } 
    public String getWinner() //which one isn't working? I tried a few attempts at the third col and third row, and both worked. 
    { 
     for (int i = 0; i <= 2; i++) //HORIZONTAL 
      { 
       if (board[i][0].equals(board[i][1]) && board[i][1].equals(board[i][2])) 
        return board[i][0]; 
     } 
     for (int j = 0; j <= 2; j++) //VERTICAL 
      { if (board[0][j].equals(board[1][j]) && board[1][j].equals(board[2][j])) 
      return board[0][j]; 
     } 
     if (board[0][0].equals(board[1][1]) && board[1][1].equals(board[2][2])) 
      return board[0][0]; 
     if (board[0][2].equals(board[1][1]) && board[1][1].equals(board[2][0])) 
      return board[1][1];     
     return " "; 
     } 

} Любые советы?

Answer 1

Если нет места берется в строке или столбце или диагональ, то все они будут " ". Однако это все равно будет удовлетворять условию, например

board[i][0].equals(board[i][1]) && board[i][1].equals(board[i][2]) 

После этого он сразу же вернет " " и не будет проверять другие позиции. " " - победитель! Но вы также используете " ", чтобы указать, что победителя нет.

Добавить дополнительную проверку в ваших условиях, чтобы убедиться, что первая позиция проверять не равна " " перед проверкой остальной части условного выражения:

!" ".equals(board[i][0]) && 
board[i][0].equals(board[i][1]) && 
board[i][1].equals(board[i][2]) 

И вы можете изменить другие условия, так же.