Learn Regular Expressions by Building a Password Generator - Step 57

Tell us what’s happening:

What am I missing here?

Check constraints

    for constraint, pattern in constraints:
        len(re.findall(pattern, password))
        constraint <= len(re.findall(pattern, password))

Your code so far

import re
import secrets
import string


def generate_password(length, nums, special_chars, uppercase, lowercase):
    # Define the possible characters for the password
    letters = string.ascii_letters
    digits = string.digits
    symbols = string.punctuation

    # Combine all characters
    all_characters = letters + digits + symbols

    while True:
        password = ''
        # Generate password
        for _ in range(length):
            password += secrets.choice(all_characters)
       
        constraints = [
            (nums, r'\d'),
            (lowercase, r'[a-z]'),
            (uppercase, r'[A-Z]'),            
            (special_chars, fr'[{symbols}]')            
        ]

# User Editable Region

        # Check constraints
        for constraint, pattern in constraints:
            len(re.findall(pattern, password))
            constraint <= len(re.findall(pattern, password))
                    
                 

# User Editable Region

    return password

# new_password = generate_password(8)
# print(new_password)

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36 Edg/120.0.0.0

Challenge Information:

Learn Regular Expressions by Building a Password Generator - Step 57

Do you get an error or hint? Check the console for more information

I get this:

Sorry, your code does not pass. You’re getting there.

You should compare constraint and the length of the list returned by findall using the <= operator inside your for loop.

Console says:

// running tests 1. You should compare

constraint

and the length of the list returned by

findall

using the

<=

operator inside your

for

loop. // tests completed

Well it looks like you’ve done this.

It’s a bit redundant though. Why did you add another line and not just edit the existing len() ? Now you have len(re.findall(pattern, password)) twice

Thank you very much!

1 Like