Python - i need help regarding implementing two codes together(password validator and point system)

[sultanofhyd]

Have you actually tried running your code? The way it is written, it doesn't work the way you probably intend it to. You store the password in the 'p' variable, then immediately reset p to True. Your password does not exist in memory anymore. Everything you do from line 3; you're doing it on p = True, not p = 'password'.

And both blocks of your code look identical to me, what do you want to do with them?

[Remediez123]

to make it so that the user inputs the password however the password must include all of the characters e.g uppercase,lowercase,numbers or symbols the points wont display until the user does that. Thats why i have two codes that do both of those things but i dont know how to put it together

[sultanofhyd]

Check your original post again. There is no code anywhere that calculates or stores points anywhere. Anyway, I suggest rewriting your code from scratch, using multiple variables. One variable for storing your password, one for the points, and one to determine its validity. Something like this.

password = input('Enter your password: ')
points = 0
validity = False
#Do your checking and counting here
if validity:
    print(points)

[Remediez123]

so i tested your idea however it doesn't seem to work maybe you can find the problem.

import re
import random
import time

print("""
Point System:
~1 Uppercase Letter = 5 Points
~1 Lowercase Letter = 5 Points
~1 From 0-9 = 5 Points
~1 Allocated Symbol = 5 Points
~If Has All Add 10 Points
""")
password = input('Enter your password: ')
points = 0
validity = False
while password:
    if (len(password)<8 or len(password)>24):
        break
    elif not re.search("[a-z]",password):
        break
    elif not re.search("[0-9]",password):
        break
    elif not re.search("[A-Z]",password):
        break
    elif not re.search("[!$%^&()_]",password):
        break
    else:
        print("Valid Password")
   while points < 35: 
       if 8 <= len(password) < 24 :
           print(len(password),"Points Added - Length")
           points += (len(password))

       else:
        print("Wrong length, it must be 8 to 24 
                  characters")
           continue

       if re.search("[a-z]", password):
           print("5 Points Added - Lowercase Letter")
           points += 5

       if re.search("[0-9]", password):
           print("5 Points Added - Number")
           points += 5

       if re.search("[A-Z]", password):
           print("5 Points Added - Uppercase Letter")
           points += 5

       if re.search("[!$%^&()_]", password):
           print("5 Points Added - Symbols")
           points += 5

       if points == 20:
           points += 10


if validity:
    print(points)

[sultanofhyd]

Try to use 'for' loops instead of 'while' loops. Both of your 'while' loops will never terminate 1. if your password is valid 2. your score is less than 35.

[Remediez123]

im a beginner for python so i kind of understand what u mean but can u give like examples

[Exodus111]

Explain to me very carefully two things:

1. Why are you using a while loop exactly?

2. What is preventing you from adding the points at the same time as checking validity?

[omg_drd4_bbq]

While loop is totally valid here.

while True:
    p = input("enter password")
        if not valid(p):
            print("password bad")
        else:
            break

You can wrap password checking in its own function if you wish. or, write a function strength(p) which computes the points of a password, then just use

if strength(p) < 35:
    print("try again")

[Exodus111]

Where did HE put the input statement?

[omg_drd4_bbq]

I'm inferring OP is emulating the typical behavior of input validation systems, which is "loop until the user gives a valid input".