Java井字棋获胜判断问题：isGameOver方法始终返回true

嘿，我一眼就揪出你isGameOver方法里的致命问题了——这就是为什么它永远返回true的根源！

核心问题分析 #

• 完全错误的比较逻辑：你写的判断条件都是让同一个元素和自身重复比较，比如：

  if (board[0][columnMove] == board[0][columnMove] && board[0][columnMove] == board[0][columnMove]) {
      return true;
  }
  

  任何值和自己比较必然是true，所以这个条件永远成立，不管你下在哪，方法都会直接返回true。
• 缺少空格校验：就算修复了比较逻辑，你也得排除空格的情况——初始棋盘全是空格，总不能刚开局就判定游戏结束吧？

修正后的isGameOver方法 #

下面是修复后的完整方法，包含了行、列、对角线的正确判断，同时排除了空格的干扰，还额外补充了平局的判断逻辑：

public static boolean isGameOver(int rowMove, int columnMove) {
    char currentPlayer = board[rowMove][columnMove];
    // 检查当前落子所在的行是否连成一线
    if (board[rowMove][0] == currentPlayer && 
        board[rowMove][1] == currentPlayer && 
        board[rowMove][2] == currentPlayer) {
        return true;
    }
    // 检查当前落子所在的列是否连成一线
    if (board[0][columnMove] == currentPlayer && 
        board[1][columnMove] == currentPlayer && 
        board[2][columnMove] == currentPlayer) {
        return true;
    }
    // 检查主对角线（左上→右下）：只有落子在对角线上时才需要检查
    if (rowMove == columnMove) {
        if (board[0][0] == currentPlayer && 
            board[1][1] == currentPlayer && 
            board[2][2] == currentPlayer) {
            return true;
        }
    }
    // 检查副对角线（右上→左下）：只有落子在对角线上时才需要检查
    if (rowMove + columnMove == 2) {
        if (board[0][2] == currentPlayer && 
            board[1][1] == currentPlayer && 
            board[2][0] == currentPlayer) {
            return true;
        }
    }
    // 检查是否平局（棋盘已满但无人获胜）
    boolean isBoardFull = true;
    for (int i = 0; i < 3; i++) {
        for (int j = 0; j < 3; j++) {
            if (board[i][j] == ' ') {
                isBoardFull = false;
                break;
            }
        }
        if (!isBoardFull) break;
    }
    if (isBoardFull) {
        System.out.println("平局！");
        return true;
    }
    // 以上条件都不满足，游戏继续
    return false;
}

额外优化建议 #

• 增加落子合法性校验：在play方法里，要判断用户输入的row和column是否在0-2范围内，以及目标位置是否已经被占用，避免数组越界或重复落子的问题。
• 代码可读性优化：我用currentPlayer变量存储当前落子的字符，避免重复调用board[rowMove][columnMove]，让代码更简洁易懂。

内容的提问来源于stack exchange，提问作者santana011