Learn Regular Expressions by Building a Password Generator - Step 61

Tell us what’s happening:

Code will not pass and not sure what is wrong. I have tried multiple attempts but nothing will work. I know how to add an empty list to all() but even if i put that in still no luck.

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}]')            
        ]

        # Check constraints
        count = 0

# User Editable Region

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

# User Editable Region

            break

    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/124.0.0.0 Safari/537.36

Challenge Information:

Learn Regular Expressions by Building a Password Generator - Step 61

For the if condition, use a call to the all() function and pass an empty list as the argument to the function call.

Missing the all() function (and the empty list as well)

if all() [constraint <= len(re.findall(pattern, password)) for constraint,pattern in constraints]:
break

Is this what you mean? This try did not pass

For the if condition, use a call to the all() function and pass an empty list as the argument to the function call.

You have all() :white_check_mark:
you have an empty list :white_check_mark:

What is this?

That is the replacement for the for loop and if statements. I guessing I need to adjusting something in there? Do I need to simplify it more?

Ok I actually just solved it myself. Definitely over thought that one.

1 Like