while循环变量未更新致玩家轮询重复的问题排查与修复

核心问题 #

你的代码一直停在Player 1回合的原因很明确：

• 控制玩家切换的变量n在while循环里从未更新，每次循环n都是0，所以始终访问player[0+1]也就是Player 1的数据。
• 没有处理回合切换逻辑，比如玩家选择「Stay」后，没有触发切换到下一个玩家的机制。
• 函数参数player被内部重新赋值，外部传入的玩家数据完全没用，属于冗余问题。
• 循环没有终止条件，playing变量永远是True，会无限循环。

修复后的代码示例 #

def display_game_options():
    """
    根据玩家分数是否>=14打印游戏选项，依次轮询玩家回合
    """
    # 初始化玩家数据（如果需要外部传入玩家信息，删除此处赋值，保留函数参数即可）
    player = {
        1: {'name': 'Player 1', 'score': 0, 'stayed': False, 'at_14': False, 'bust': False},
        2: {'name': 'Player 2', 'score': 0, 'stayed': False, 'at_14': False, 'bust': False},
        3: {'name': 'Player 3', 'score': 0, 'stayed': False, 'at_14': False, 'bust': False},
        4: {'name': 'Player 4', 'score': 0, 'stayed': False, 'at_14': False, 'bust': False}
    }
    total_players = len(player)
    current_player_idx = 1  # 直接用玩家编号索引，替代n+1的冗余写法
    playing = True

    while playing:
        current_player = player[current_player_idx]
        print(f"-------- {current_player['name']}'s turn --------")
        print(f"{current_player['name']}'s score: {current_player['score']}")
        print("1. Roll")
        print("2. Stay")
        if current_player['score'] >= 14:
            print("3. Roll one")
        
        # 处理用户输入，修正拼写错误decison→decision
        decision = input("请选择操作：").strip()
        
        # 处理Stay操作：标记玩家已停留，切换到下一个玩家
        if decision == '2':
            current_player['stayed'] = True
            # 循环切换玩家：最后一个玩家结束后回到第一个
            current_player_idx = current_player_idx + 1 if current_player_idx < total_players else 1
            
            # 检查所有玩家是否都已停留，触发游戏结束
            all_stayed = all(p['stayed'] for p in player.values())
            if all_stayed:
                playing = False
                print("所有玩家都已结束回合，游戏结束")
                continue
        
        # 处理Roll/Roll one操作（示例逻辑，可根据实际规则调整）
        if decision in ['1', '3']:
            # 此处添加掷骰子更新分数、判断是否爆牌等逻辑
            # current_player['score'] += ...
            # 切换到下一个玩家
            current_player_idx = current_player_idx + 1 if current_player_idx < total_players else 1

# 调用函数测试
display_game_options()

关键修复点 #

1. 玩家索引动态更新：每次处理完当前玩家操作后，更新current_player_idx，实现循环切换玩家，最后一个玩家结束后回到第一个。
2. 添加循环终止条件：增加「所有玩家都停留则结束游戏」的逻辑，避免无限循环。
3. 消除参数冗余：如果不需要外部传入玩家数据，直接删除函数参数player；如果需要外部传，就去掉内部的player初始化代码。
4. 代码可读性优化：用玩家编号直接索引，替代n+1的冗余写法；用f-string简化打印语句；修正拼写错误decison→decision。

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