如何优化可返回1（白方）/0（黑方）的国际象棋玩家回合系统？

嘿，我来帮你看看这段代码的优化方向！首先咱们先梳理下原代码里的几个小问题，然后给出更简洁高效的实现方案：

原代码的核心问题 #

• 冗余的状态变量：WhitePlayer和BlackPlayer是完全互斥的，用两个变量既浪费空间，还可能出现逻辑不一致的情况（比如不小心把两个都设为True）
• 赋值操作错误：你写的BlackPlayer == False是比较操作，不是赋值，应该用单等号=才会修改变量值
• 无终止的死循环：while True没有退出条件，这个方法会永远运行下去，没办法结束
• 缺少返回值：原方法没有返回1或0，不符合你“返回1代表白方，0代表黑方”的需求

优化方案1：单变量跟踪回合 + 带终止条件的循环 #

如果你的需求是持续切换回合直到游戏结束，可以用单个布尔变量来维护状态，同时添加合理的终止逻辑：

def PlayerTurn(self):
    is_white_turn = True
    # 循环直到满足终止条件（比如游戏结束）
    while True:
        # 返回对应回合值：1=白方，0=黑方
        current_turn = 1 if is_white_turn else 0
        print("White Player's Turn!" if is_white_turn else "Black Player's Turn!")
        
        # 这里可以添加玩家操作逻辑，比如等待输入走法
        # 示例：等待用户输入，输入quit则退出循环
        user_action = input("Enter your move (or 'quit' to end): ")
        if user_action.lower() == 'quit':
            break
            
        # 操作完成后切换回合
        is_white_turn = not is_white_turn

优化方案2：用实例变量维护回合状态（更符合游戏逻辑） #

如果这是游戏类的一部分，更合理的方式是把回合状态作为类的实例变量，每次调用方法时返回当前回合并自动切换：

class ChessGame:
    def __init__(self):
        # 初始化时默认白方先下
        self.is_white_turn = True
    
    def get_player_turn(self):
        # 先记录当前回合值
        current_turn = 1 if self.is_white_turn else 0
        print("White Player's Turn!" if self.is_white_turn else "Black Player's Turn!")
        # 切换到下一个回合
        self.is_white_turn = not self.is_white_turn
        # 返回回合标识
        return current_turn

# 使用示例
game = ChessGame()
print(game.get_player_turn())  # 输出1，打印"White Player's Turn!"
print(game.get_player_turn())  # 输出0，打印"Black Player's Turn!"
print(game.get_player_turn())  # 输出1，再次切换回白方

优化后的核心优势 #

• 简化状态管理：用单个变量替代两个互斥变量，逻辑更清晰，避免出错
• 修复赋值错误：用=正确修改变量状态
• 满足返回需求：明确返回1或0，符合你的功能要求
• 更贴近实际游戏场景：实例变量方案能持续维护游戏的回合状态，适合完整的游戏逻辑开发

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