Java实现TicTacToe(井字棋)胜负判定方法优化咨询

1、是否应当将checkforWin方法拆分为玩家胜负判定、电脑胜负判定两个独立方法？ #

• 不需要拆分。玩家和电脑的胜负判定逻辑完全一致，仅需要校验的棋子类型不同，拆分为两个独立方法会产生大量重复代码，违反代码复用的开发原则，反而会降低代码可维护性。

2、当前对角线判定逻辑是否合理？ #

• 现有逻辑可以正常运行，但存在明显的冗余问题：
  • 对角线判定中的for(int x = 0; x <= board.length; x ++)属于完全无效的代码，循环逻辑和判断条件没有任何关联，等价于重复执行了4次完全相同的if判断
  • 分别对X和O写两套对角线判断逻辑，产生了不必要的代码重复
  • 先判断中心棋子类型的逻辑本身没问题，但和前面的横、纵向判断的编码风格不统一，可读性较差

你可以通过预定义获胜线坐标的方式大幅简化checkforWin的逻辑，优化后的代码如下：

// 预定义所有获胜线的坐标，每一组为3个落子点的[行, 列]坐标
private static final int[][] WIN_LINES = {
    // 横向获胜线
    {0,0, 0,2, 0,4},
    {1,0, 1,2, 1,4},
    {2,0, 2,2, 2,4},
    // 纵向获胜线
    {0,0, 1,0, 2,0},
    {0,2, 1,2, 2,2},
    {0,4, 1,4, 2,4},
    // 对角线获胜线
    {0,0, 1,2, 2,4},
    {0,4, 1,2, 2,0}
};

/**
 * 判定指定棋子是否获胜
 * @param board 棋盘数组
 * @param targetPiece 要校验的棋子（X/O）
 * @return 是否获胜
 */
public static boolean checkforWin(char[][] board, char targetPiece) {
    for (int[] line : WIN_LINES) {
        // 判断当前获胜线的三个点是否都是目标棋子
        if (board[line[0]][line[1]] == targetPiece
                && board[line[2]][line[3]] == targetPiece
                && board[line[4]][line[5]] == targetPiece) {
            return true;
        }
    }
    return false;
}

调用的时候只需传入对应棋子即可：

• 玩家落子后调用：checkforWin(board, playerPiece)
• 电脑落子后调用：checkforWin(board, computerPiece)
  优化后的优势：
• 代码量大幅缩减，没有任何重复逻辑
• 所有获胜规则统一维护，后续调整规则仅需修改WIN_LINES常量即可，无需修改判定逻辑
• 可读性和可维护性大幅提升，删除了原有的无效循环和冗余判断

你现有代码中的validateMove、playerMove、computerMove中的大量switch分支也可以通过坐标映射表的方式简化，进一步减少重复代码。

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