Pygame国际象棋开发：如何禁止国王进入对方国王相邻格子？

你的核心问题在于把「国王移动限制」的逻辑塞进了原本负责「判断是否被将军」的in_check函数，既违反了单一职责原则，也因为逻辑时机不对导致遗漏了部分格子。以下是具体修复思路和代码：

1. 拆分职责，重构逻辑 #

in_check函数只保留「判断移动后是否会被将军」的核心功能，国王不能进入对方国王相邻格子的限制，应该在国王生成候选移动列表的阶段就直接过滤掉，而不是事后修改。

修复后的King类calculate_moves方法 #

class King(Piece):
    def calculate_moves(self, board, row, col, update_ui):
        self.moves = []
        # 国王的8个移动方向
        directions = [(-1,-1), (-1,0), (-1,1),
                      (0,-1),          (0,1),
                      (1,-1),  (1,0), (1,1)]
        enemy_color = "white" if self.color == "black" else "black"
        
        # 先定位敌方国王的位置
        enemy_king_pos = None
        for r in range(ROWS):
            for c in range(COLS):
                piece = board.squares[r][c].piece
                if isinstance(piece, King) and piece.color == enemy_color:
                    enemy_king_pos = (r, c)
                    break
            if enemy_king_pos:
                break

        # 生成所有合法候选移动
        for dr, dc in directions:
            new_row = row + dr
            new_col = col + dc
            # 先判断是否在棋盘范围内
            if 0 <= new_row < ROWS and 0 <= new_col < COLS:
                target_square = board.squares[new_row][new_col]
                # 过滤1：不能进入敌方国王的相邻格子
                if enemy_king_pos:
                    er, ec = enemy_king_pos
                    if abs(new_row - er) <= 1 and abs(new_col - ec) <= 1:
                        continue
                # 过滤2：不能移动到己方棋子所在格子
                if target_square.has_friendly_piece(self.color):
                    continue
                # 添加合法移动
                self.moves.append(Move(board.squares[row][col], target_square))
        
        # 最后过滤：排除移动后会让自己被将军的格子
        valid_moves = []
        for move in self.moves:
            if not board.in_check(self, move):
                valid_moves.append(move)
        self.moves = valid_moves

修复后的in_check函数 #

def in_check(self, piece: Piece, move: Move):
    virtual_piece, virtual_board = deepcopy(piece), deepcopy(self)
    virtual_board.move(virtual_piece, move, True)
    
    # 找到移动后己方国王的位置
    king_pos = None
    king_color = piece.color
    for r in range(ROWS):
        for c in range(COLS):
            p = virtual_board.squares[r][c].piece
            if isinstance(p, King) and p.color == king_color:
                king_pos = (r, c)
                break
        if king_pos:
            break
    
    # 检查所有敌方棋子是否能攻击到己方国王
    for row in range(ROWS):
        for col in range(COLS):
            square = virtual_board.squares[row][col]
            p = square.piece
            if square.has_enemy_piece(king_color):
                virtual_board.calculate_moves(p, row, col, False)
                for m in p.moves:
                    if m.final.row == king_pos[0] and m.final.col == king_pos[1]:
                        if not INSTANT_CHECKMATE:
                            return True
                        return False
    return False

2. 原代码问题说明 #

• 职责混乱：你在in_check里直接修改真实棋子的moves列表，会导致状态污染——如果in_check被多次调用，每次都会删除移动项，最终导致国王的可用移动列表被错误修改。
• 逻辑时机错误：国王的移动限制应该在生成候选移动时就过滤，而不是在判断「是否被将军」的阶段事后删除，这会因为遍历顺序、虚拟棋盘状态等问题导致遗漏。
• 判断逻辑不严谨：原代码仅通过对比敌方国王和己方国王的移动终点来过滤，没有直接判断目标格子是否在敌方国王的相邻范围内，容易出现漏判。

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