Python中的猜谜游戏循环

Question

作为一个刚入门的程序员，我之前看过一些代码评审的内容，所以我尝试自己做一次代码评审。但是不知怎么的，我让程序出现了问题，不能按预期工作。原本应该在玩了5局后停止循环，但有时候却没有这样做。

import random
import math
import console

console.set_font('Arial-BoldMT',15)

print'Welcome to the guessing game!\nA number will be randomly chosen from 0 to 1000.\nThe player will make a guess, and then the computer will guess. Who ever is closest wins that round!\nFirst to 5 wins!'

rounds = 0
run_again = 'y'
player_wins = 0
computer_wins = 0
draws = 0

while run_again == 'y':
    number = random.randint(0,1000)
    player_number = input('\nPlayer: ')
    computer_number = random.randint(0,1000)

    print 'Computer:', computer_number
    print 'Number:', number

    player_score=math.fabs(player_number-number)
    computer_score=math.fabs(computer_number-number)

    if player_score>=computer_score:
        computer_wins+=1
        rounds +=1
        console.set_color(1.00, 0.00, 0.00)
        print 'You lost that round'
        console.set_color(0.00, 0.00, 0.00)

    if player_score<=computer_score:
        player_wins+=1
        rounds +=1
        console.set_color(0.00, 0.00, 1.00)
        print 'You won that round'
        console.set_color(0.00, 0.00, 0.00)

    if player_score==computer_score:    
        draws+=1
        rounds +=1
        console.set_color(0.00, 1.00, 0.00)
        print 'That round was a tie'
        console.set_color(0.00, 0.00, 0.00)


    if rounds == 5:
        if player_wins == 4:
            console.set_color(0.00, 0.00, 1.00)
            print '\nYOU WON THE GAME'
            console.set_color(0.00, 0.00, 0.00)
            break
        if computer_wins == 4:
            console.set_color(1.00, 0.00, 0.00)
            print '\nYOU LOST THE GAME'
            console.set_color(0.00, 0.00, 0.00)
            break

Answer 1

你只有在某个玩家赢了4局的时候才会结束比赛。用一个for循环来限制回合数会简单很多：

for _ in range(5):

Answer 2

我觉得如果你正确地考虑结果的话，应该没问题：

if rounds == 5:
    if playr_wins > computer_wins:
        console.set_color(0.00, 0.00, 1.00)
        print '\nYOU WON THE GAME'
        console.set_color(0.00, 0.00, 0.00)
        break
    elif computer_wins > playr_wins:
        console.set_color(1.00, 0.00, 0.00)
        print '\nYOU LOST THE GAME'
        console.set_color(0.00, 0.00, 0.00)
        break
    else:
       print "Wrong counts"