Возникают проблемы с использованием .isupper()

Question

Моей программой является силой пароля симулятора и всякий раз, когда я ввожу пароль, то всегда выводит оценку 1.

def upper_case(): 
    points = int(0) 
    limit = 3 
    for each in pword: 
     if each.isupper(): 
      points = points + 1 
     if points > limit: 
      points = limit 
     else: 
      points = points + 0 
     return points 

pword = raw_input("ENTER: ") 
upper_case() 

points = 0 
points += upper_case() 

print points 

Answer 1

Вы возвращающийся points слишком рано, потому что вы с отступом слишком далеко. Удалить отступа:

def upper_case(): 
    points = int(0) 
    limit = 3 
    for each in pword: 
     if each.isupper(): 
      points = points + 1 
     if points > limit: 
      points = limit 
     else: 
      points = points + 0 
    return points 

Вы можете упростить это:

def upper_case(pword): 
    return min((sum(1 for each in pword if each.isupper()), 3)) 

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