Pythonでのパスワードの確認機能

Question

パスワードの検証機能は、パスワードが3未満で長さが20より大きい場合、および両方のパスワードが一致しない場合はfalseを返します。なぜ私のロジックが失敗しているのか理解してくれて、私にこれをやるより良い方法を教えてくれますか？私はあなたのコードに最小の変更のように思えた簡単な方法があると確信しているものの

def verifyPassword(password, password2): 
    if(len(password) == len(password2): 
     if(len(password) and len(password2) >= 3: 
      if(len(password) and len(password2) < 20: 
       if password == password: 
        return True 
       else:     
        return False 
      else: 
       return False 
     else: 
      return False 
    else: 
     False 
print(verifyPassword("le3ather", "leather2")) 
print(verifyPassword("leather", "leather2")) 
print(verifyPassword("leather", "leather")) 
print(verifyPassword("le", "Le")) 
print(verifyPassword("leatherLeatherleather", "leatherLeatherleather")) 

Answer 1

私はあなたが正確に狙っているか知っているが、ここで私はあなたのために作ったものではありません。 https://gist.github.com/dgsbicak/96517f33e516ca4438c6e252149b8cbc

def verifyPassword(password, password2): 
    count = 0 
    # Rule 1 
    pas = [password, password2] 
    for p in pas: 
     if len(p)<3: 
      print("Password length is shorter than 3") 
      break 
     elif len(p)>19: 
      print("Password length is longer than 19") 
      break 
     else: pass 


    if password==password2: 
     return "Password is correct" 
    else: 
     return "Password is not correct" 

print(verifyPassword("le3ather", "leather2")) 
print(verifyPassword("leather", "leather2")) 
print(verifyPassword("leather", "leather")) 
print(verifyPassword("le", "Le")) 
print(verifyPassword("leatherLeatherleather", "leatherLeatherleather")) 

Answer 2

この

if((len(password) == len(password2)) and (len(password) and len(password2)) >= 3 < 20): 

は、おそらくこの

if((len(password) == len(password2)) and (len(password) and len(password2)) >= 3 and (len(password) and len(password2)) < 20): 

でなければなりません。

Answer 3

MINLEN = 3 
MAXLEN = 20 

def verifypassword(pw1, pw2): 
    if pw1 != pw2: 
     return False 
    if len(pw1) < MINLEN: 
     return False 
    if len(pw1) > MAXLEN: 
     return False 

    return True 


def test(): 
    tests = [ 
      ("le3ather", "leather2"), 
      ("leather", "leather2"), 
      ("leather", "leather"), 
      ("le", "le"), 
      ("thisiseverybittoolong", "thisiseverybittoolong"), 
    ] 
    for p1, p2 in tests: 
     print(p1, p2, verifypassword(p1, p2)) 


if __name__ == '__main__': 
    test()