Программа, застрявшая во время цикла, не печатается

Question

import random 

def usertype(): 
    randletter = random.choice('qwer') 
    userinput = raw_input('Press '+str(randletter)) 
    if userinput == randletter: 
     return 'Correct' 
    else: 
     return 'Incorrect' 

def usertypetest(x,y,result): 
    while x <= 9: 
     result = usertype() 
    if result == 'Correct': 
     x = x+1 
     y = y+5 
    else: 
     x = x+1 
     y = y-2 
    return str(y)+'is your score' 

print usertypetest(0,0,usertype) 

Вот мой код. Я хочу, чтобы он попросил пользователя нажать кнопку, случайно выбранную из набора (Q, W, E, R), затем напечатать либо правильную, либо неправильную, в зависимости от того, какую кнопку они нажимают. Я хочу, чтобы это произошло 10 раз. После десяти попыток он напечатает свой результат: 5 для каждого «Исправить» и -2 для «Неправильно». Вместо этого я получаю это.

Press e(e) 
Press e(e) 
Press w(e) 
Press q(e) 
Press q(e) 
Press q(e) 
Press r(e) 
Press e(e) 
Press w(e) 
Press q(e) 
Press e(e) 
Press e(e) 
Press e(e) 
Press e(e) 
Press q(e) 
Press w(e) 
Press r(e) 
Press w(e) 
Press r(e) 
Press w(e) 
Press r(e) 
Press r(e) 

Независимо от того, что я вхожу, он не возвращает ни «Правильный», ни «Неверный». Он также продолжается уже 10 лет и не показывает их оценку. Очевидно, проблема я не вижу.

Мой ввод находится в скобках.

Для уточнения, это то, что я хочу:

Press q(q) 
Correct 
Press e(q) 
Incorrect 
Press w(w) 
Correct 
Press q(q) 
Correct 
Press e(eq) 
Incorrect 
Press e(e) 
Correct 
Press q(q) 
Correct 
Press q(r) 
Incorrect 
Press w(w) 
Correct 
Press r(r) 
Correct 
29 is your score 

Answer 1

В Python отступы очень важно.

В этом коде x для цикла while никогда не изменяется, как if блок находится на том же уровне отступа как петли while. Таким образом, только петлевой инструкция result = usertype()

while x <= 9: 
    result = usertype() 
if result == 'Correct': 
    x = x+1 
    y = y+5 

Два дополнительных критика:

Вы приращением x в двух местах, когда это нужно только сделать один раз.

while x <= 9: 
    result = usertype() 
    if result == 'Correct': 
     y = y+5 
    else: 
     y = y-2 
    x += 1 

Кроме того, поскольку вы зацикливание фиксированного числа раз, почему бы не игнорировать приращения x и использовать цикл, например, так:

for x in range(10): 
    result = usertype() 
    if result == 'Correct': 
     y = y+5 
    else: 
     y = y-2 

Answer 2

Вы должны поместить if result == 'Correct': блок под петли while x <= 9:, где вы получите пользовательский ввод, так что он получает оценку каждый раз. И вы можете добавить print(result), чтобы получить правильный/неправильный обратной связи, как в вашем примере:

def usertypetest(x,y,result): 
    while x <= 9: 
     result = usertype() 
     if result == 'Correct': 
      x = x+1 
      y = y+5 
     else: 
      x = x+1 
      y = y-2 
     print(result) 
    return str(y)+'is your score' 

Answer 3

Просто поместите if блок под петлей while. Проблема решена.

Попробуйте с этим кодом:

import random 

def usertype(): 
    randletter = random.choice('qwer') 
    userinput = raw_input('Press '+str(randletter)) 
    if userinput == randletter: 
     return 'Correct' 
    else: 
     return 'Incorrect' 

def usertypetest(x,y,result): 
    while x <= 9: 
     result = usertype() 
     if result == 'Correct': 
      x = x+1 
      y = y+5 
     else: 
      x = x+1 
      y = y-2 
    return str(y)+'is your score' 

print usertypetest(0,0,usertype) 

Answer 4

Ваша главная проблема заключалась в том, что вы имели, если/еще блок в неправильном объеме. Вам нужно было находиться под блоком while. Это гарантирует, что он проверяет, вводит ли пользователь правильный ввод при каждом запуске usertype().

import random 

moves = 0 
score = 0 

def usertype(): 
    randletter = random.choice('qwer') 
    userinput = raw_input('Press '+str(randletter)) 
    if userinput == randletter: 
     return True 
    else: 
     return False 

def usertypetest(moves, score): 
    while moves < 10: 
     result = usertype() 
     moves = moves + 1 
     if result: 
      score = score + 5 
     else: 
      score = score - 2 
    return str(score) + ' is your score' 

print usertypetest(moves, score) 

Answer 5

Кроме того, вы не печатаете значение результата переменной.После оценки результатов, добавьте следующее:

печать результат

Answer 6

Есть несколько проблем.

1. Вам необходимо напечатать «правильный» перед возвратом пользователя.
2. Вам не нужно указывать «результат» в usertypetest.
3. Положите, если ... еще внутри цикла в usertypetest.
4. Измените свой последний отпечаток.

Вот правильный код.

import random 

def usertype(): 
    randletter = random.choice('qwer') 
    userinput = raw_input('Press '+str(randletter)) 
    if userinput == randletter: 
     print 'Correct' 
     return 'Correct' 
    else: 
     print 'Incorrect' 
     return 'Incorrect' 

def usertypetest(x,y): 
    while x <= 9: 
     result = usertype() 
     if result == 'Correct': 
      x = x+1 
      y = y+5 
     else: 
      x = x+1 
      y = y-2 
    return str(y)+'is your score' 

print usertypetest(0,0) 

Answer 7

Помимо проблемы с отступом, которую идентифицировали другие люди, код не является особенно идиоматическим Python. usertypetest() функция может быть:

def usertypetest(x,y,result): 
    for x in range(10): 
     if usertype() == 'Correct': 
      y = y + 5 
     else: 
      y = y - 2 
    return '%d is your score' % y 

Там могут быть лучшие способы сделать это, но это немного проще, и немного больше Pythonic.

Я также наблюдаю, что я не вижу круглых скобок вокруг ввода, который, как утверждает пример, видит.

Если вы хотите, чтобы увидеть вердикт по каждой букве, то вам необходимо сохранить отдачу от usertype() после всего:

def usertypetest(x,y,result): 
    for x in range(10): 
     result = usertype() 
     print result 
     if result == 'Correct': 
      y = y + 5 
     else: 
      y = y - 2 
    return '%d is your score' % y