无法突破时间循环

Question

我编写了下面的python代码。它需要一张我在网上找到的英文单词的清单，然后把它们列成一个清单，这样我就可以用它们来做绞刑了。我的问题是，每次我运行这个程序并成功地猜出这个单词时，它都不会跳出while循环。只会继续下去。我不知道为什么我的一生。有谁知道为什么没有把最后的信息打印给获胜者呢？

import random

words = []

lettersGuessed = []

isGuessed = 0



wordFile = open(r'C:\Users\Sarah\PycharmProjects\hangman\words.txt')

for word in wordFile:
    words.append(word.rstrip(wordFile.readline()))


mysteryWord = random.choice(words)

while len(mysteryWord) <= 1:
    mysteryWord = random.choice(words)

for letter in mysteryWord:
    print("?", end = "")
print("\n")

def isWon():
    #win conditions
    count = 0
    for letter in mysteryWord:
        if letter in lettersGuessed:
            count += 1

        if count == len(mysteryWord):
            isGuessed = 1



count = 0

while isGuessed == 0:


    guess = input("Guess a letter \n")

    if guess.upper() or guess.lower() in mysteryWord:
        lettersGuessed.append(guess)
        for letter in mysteryWord:
            if letter in lettersGuessed:
                print(letter, end ='')
            else:
                print("?", end = '')
    print("\n")
    count = 0
    isWon()
    if isGuessed == 1:
        break

print("Congratulations, you correctly guessed ", mysteryWord)

Answer 1

顶层代码中的isGuessed和isWon函数中的isGuessed是两个不同的变量。函数是一个单独的命名空间(否则，一个使用公共名称(如i )的变量的函数会在其他代码中造成严重破坏)。

这可以通过global声明来解决，但这是一个非常糟糕的风格。mysteryWord和lettersGuessed等变量也是如此。

相反，您应该从isWon函数返回值：

def isWon(mysteryWord, lettersGuessed):
   # do your counting...
   return isGuessed

# main code
victory = False
while not victory:
   # ...
   victory = isWon(mysteryWord, lettersGuessed)
   # you don't even need the if ... break statement

顺便说一句，你的支票中所有的字母猜测都可以是一条线：

def isWon(mysteryWord, lettersGuessed):
    return set(lettersGuessed) == set(mysteryWord)

Answer 2

当前的问题是，不管输入是什么，isWon()都不设置isGuessed。如果你用"foo“字串猜

lettersGuessed.append(guess)

将使lettersGuessed成为一个包含一个项(即字符串)的列表。我觉得你想做的是

lettersGuessed.extend(list(guess))

将guess中的每个字母添加到lettersGuessed列表中。

还有两点值得一提：

• 如果您在问题中猜到单词的一个字谜，isWon()将考虑游戏的胜负，例如，如果单词为"foo“，"oof”将被认为是一个正确的解决方案。
• words.append(word.rstrip(wordFile.readline()))读取输入文件的每一行偶数行，并在删除与以下单词相同的字符后将其添加到words列表中。你想要做words.append(word.strip())代替。

Answer 3

都是关于范围的。isGuessed用于isWon()是在本地范围中定义的。如果要影响在全局范围中声明的isGuessed，则必须将其作为参数传递给isWon()，或者在修改isGuessed之前使用global关键字。见下文：

def isWon():
#win conditions
count = 0
for letter in mysteryWord:
    if letter in lettersGuessed:
        count += 1

    if count == len(mysteryWord):
        global isGuessed
        isGuessed = 1    

这一变化的产出如下：

python3 test.py
?????

Guess a letter
1
?????

Guess a letter
2
?????

Guess a letter
3
????3

Guess a letter
4t
????3

Guess a letter
t
t??t3

Guess a letter
e
te?t3

Guess a letter
s
test3

Congratulations, you correctly guessed  test3