Проверка пароля не возвращается true

Question

Я написал пароль для проверки безопасности пароля. Существует три функции для проверки безопасности и возврата значения, слабый, средний и сильный. Я называю эти функции здесь:

if check_pass_strength_weak(password): 
      print "Weak" 
     elif check_pass_strength_medium(password): 
      print "Medium" 
     elif check_pass_strength_strong(password): 
      print "Strong"" 

, но сильный никогда не возвращается. Вот функция.

def check_pass_strength_strong(password): 
    upper_count = 0 
    lower_count = 0 
    number_count = 0 
    for letter in password: 
     if letter == letter.upper(): 
      upper_count = upper_count + 1 
     elif letter == letter.lower(): 
      lower_count = lower_count + 1 
     elif letter.isalpha(): 
      number_count = number_count + 1 

    if upper_count > 0 and lower_count > 0 and number_count > 0: 
     return True 

Answer 1

Ваша проблема в том, что:

'1' == '1'.upper() 

так что остальная часть вашего elif никогда не запускать и number_count никогда не увеличивается. Отразить его вокруг:

if letter.isdigit(): 
    ... 
elif ... 

Или, лучше, на самом деле использовать в Python str methods, и не забудьте return False:

def check_pass_strength_strong(password): 
    upper_count = 0 
    lower_count = 0 
    number_count = 0 
    for letter in password: 
     if letter.isdigit(): 
      number_count += 1 
     elif letter.isupper(): 
      upper_count += 1 
     elif letter.islower(): 
      lower_count += 1 
    if upper_count > 0 and lower_count > 0 and number_count > 0: 
     return True 
    return False 

---

В самом деле, как вы проверяете несколько уровней, вы можете вынесем Счета:

def count_char_types(password): 
    lower_count = upper_count = number_count = 0 
    for letter in password: 
     if letter.isdigit(): 
      number_count += 1 
     elif letter.isupper(): 
      upper_count += 1 
     elif letter.islower(): 
      lower_count += 1 
    return lower_count, upper_count, number_count 

затем осуществить, например,

def check_pass_strength_strong(password): 
    l, u, n = count_char_types(password) 
    return l > 0 and u > 0 and n > 0 

и рассмотреть возможность добавления symbol_count!

Answer 2

Ваш код нуждается в двух исправлениях. Во-первых, как указано jonrsharpe в his answer, '1' == '1'.upper(), поэтому вы не считаете числа. Вы должны проверить это первый:

if letter.isdigit(): 
    #... 
elif #upper/lower conditions 

И вы, вероятно, хотите отменить свои условия прочности пароля:

if check_pass_strength_strong(password): 
    print "Strong" 
elif check_pass_strength_medium(password): 
    print "Mediu" 
elif check_pass_strength_weak(password): 
    print "Weak" 

Answer 3

Что об использовании регулярных выражений для проверки цифр/верхний регистр/нижний регистр символов?

например:

def check_pass_strength_strong(password): 
    return all((re.findall(r'[A-Z]', password), 
       re.findall(r'[a-z]', password), 
       re.findall(r'[0-9]', password)))