CS50x实验6 Python世界杯模拟代码跳过simulate_tournament无输出调试咨询

问题现象根因 #

你观测到的“程序跳过simulate_tournament、无输出”的现象，并非函数未被调用，而是两处代码错误叠加导致程序卡在死循环、无法走到最终打印步骤：

• 循环计数器未自增：写模拟总次数的while循环时，仅初始化了计数器i = 0，但循环体全程没有补上i += 1的自增逻辑，循环判断i < N永远成立，程序会在模拟逻辑里无限循环，根本跳不到最后打印胜率的代码块。
• 计数字典键值硬编码错误：判断获胜队是否在计数字典中时，用了真实的获胜队名做判断，但赋值时没有用拿到的winner变量做键，反而硬编码了字符串"winner"当字典键，哪怕手动跳出循环，所有胜场计数都会堆在一个叫"winner"的无效键上，完全无法输出各队真实的胜率。
• 残留冗余代码：读取csv的循环中单独写了一行row["team"]，这行没有任何实际操作的代码是写题时的草稿残留，不影响运行但属于无效代码。

对应修复步骤 #

1. 补全循环的计数器自增逻辑，修正字典键的引用错误：
   原错误代码块：

   i = 0
   while i < N:
       winner = simulate_tournament(teams)
       
       if winner in counts:
           counts["winner"] += 1
       else:
           counts["winner"] = 1
   

   修正后代码块：

   i = 0
   while i < N:
       winner = simulate_tournament(teams)
       
       if winner in counts:
           # 用实际获胜队名做字典键
           counts[winner] += 1
       else:
           counts[winner] = 1
       # 补上计数器自增，避免死循环
       i += 1
   

2. 删掉读取csv环节的冗余行row["team"]，清理无效代码。
3. 可选优化：可以把while循环替换为for循环，从语法层面避免忘写自增的同类问题，写法更简洁：

   for _ in range(N):
       winner = simulate_tournament(teams)
       if winner in counts:
           counts[winner] += 1
       else:
           counts[winner] = 1
   

调试验证技巧 #

• 测试阶段可以临时把全局常量N的值改成1，跑通最小流程后再逐步调大模拟次数，避免一开始就跑1000次时，因为死循环卡很久看不到进度。
• 可以临时在模拟循环里加一行打印语句，比如拿到获胜队后打印一次队名，观察输出频率，快速定位是不是存在死循环的问题。

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