なぜかエラーがででないのに、うまくいかない

Question

前提

他人のpythonのコードを改造してパスワードジェネレーターを作っています。
記号、数字、アルファベットそれぞれのオンオフ機能を追加したいのですが、特にエラーもでてないのにうまくいきません。

試したこと

random_char = secrets.choice(data)から
data =　〜〜でパスワードの文字を管理してると思ったので、分岐によってdataに記号や数字を渡さないプログラムを書きました。
ちゃんと分岐できたか確かめるprint文を書き、しっかり分岐していることがわかりました。
elseはなにも制限してない分岐ですが、実行の結果からelseのprintが表示されてないので、この分岐は通ってないと思われます。
それなのになぜ、結果からは制限されてないパスワードが生成されるのでしょうか？

実行の結果

-es記号、　-en数字、　 -eaアルファベットをそれぞれ制限する。
$ python3 password_generator.py -es
Hello, symbols
Hello, symbols
PASSWORD: Wx4-N2

$ python3 password_generator.py -en
Hello, numbers
Hello, numbers
PASSWORD: k4$g$K

$ python3 password_generator.py -ea
Hello, alphabets
Hello, alphabets
PASSWORD: G03e)(

該当のソースコード

# functions my want to achieve
# 1.you can selected symbols on/off
# 2.you can selected numbers on/off
# 3.you can selected alphabet on/off
# 4.auto copy , achieved in 11/24
# 5.make a error what use -l more 74 and -ed in 11/24

import argparse  # to work with the arguments
import string
import secrets
import pyperclip                # auto copy moduel


# function to generate the password
def get_password(length: int) -> str:
    # creating data
    symbols = ['!', '@', '#', '$', '%', '^', '&', '*', '(', ')', '-', '+']
    # symbols = ['!', '#', '$', '%', '&', ' (', ')','*', '+', ',', '-', '.', '/', ':', ';', '<', '=', '>', '?', '@', '[', ']', '^', '_', '`', '{', '|', '}', '~']
    numbers = [str(a) for a in range(10)]
    lowercase_characters = string.ascii_lowercase
    uppercase_characters = string.ascii_uppercase

    # variable to hold the password
    password = ''

    # adding one symbol
    password += secrets.choice(symbols)

    # adding one number
    password += secrets.choice(numbers)

    # adding one lowercase character
    password += secrets.choice(lowercase_characters)

    # adding one uppercase character
    password += secrets.choice(uppercase_characters)

    # now run a for loop starting from the current length of the password
    # all the way to the maxium length to fill in the remaining data
    while len(password) < length:
        # adding all the data together into a one list
        characters = lowercase_characters + uppercase_characters
        if args.excludesymbols:
            data = numbers + list(characters)
            print('Hello, symbols')
        elif args.excludenumbers:
            data = symbols + list(characters)
            print('Hello, numbers')
        elif args.excludealphabet:
            data = symbols + numbers
            print('Hello, alphabets')
        else: 
            data = symbols + numbers + list(characters)
            print('Hello.')

        # getting a random character from the list
        random_char = secrets.choice(data)

        # if asked to exclude duplicates
        if args.excludeduplicates:
            # if character not already inside the password
            # then add it into the password
            if random_char not in password:
                password += random_char
        else:  # if not asked to exclude duplicates
            # then just add the character without checking
            password += random_char

    # create a list of the password
    password_list = list(password)
    # shuffle the list into random sequence
    password = shuffle(password_list)

    # returning the password
    return password


# shuffle function based on Fisher–Yates shuffle using secrets.choice()
# as the integer selector
def shuffle(password: list):
    # n used to determine range of loop
    n = len(password)
    for x in range(n - 1, 0, -1):
        # set new variable y to random int within needed index
        y = secrets.choice(range(0, x + 1))
        # swap elements at index x and index y
        password[x], password[y] = password[y], password[x]
    # return concatenated password
    return ''.join(password)

# main method


def main(args: argparse.Namespace) -> None:
    # storing the length in a variable
    length = args.length

    # if the value is out of range then inform the user and exit
    # if length < 6 or length > 20:
    #   print('ERROR: -l/--length should be in the range of 6 - 20')
    #  exit(0)

    # add
    if length > 74 and args.excludeduplicates:
        print("ERROR: its too many length")
        exit(0)

    # this will hold the final password
    password = get_password(length)

    # printing the password to the user
    print(f'PASSWORD: {password}')
    pyperclip.copy(password)


if __name__ == '__main__':
    # creating the argparse object
    parser = argparse.ArgumentParser()

    # adding the length argument
    parser.add_argument(
        '-l', '--length',
        help='Length of the password. [MIN=1], [MAX=74, if you use -l and -ed ], [DEFAULT = 6]',
        metavar='',
        default=6,
        type=int
    )

    # adding the exclude duplicates argument
    parser.add_argument(
        '-ed',
        '--excludeduplicates',
        action='store_true',
        help='Excludes duplicate characters from the password'
    )

    parser.add_argument(
        '-es',
        '--excludesymbols',
        action='store_true',
        help='Excludes symbols characters from the password'
    )

    parser.add_argument(
        '-en',
        '--excludenumbers',
        action='store_true',
        help='excludes numbers characters from the password'
    )

    parser.add_argument(
        '-ea',
        '--excludealphabet',
        action='store_true',
        help='excludes alphabet characters from the password'
    )

    # parsing the argument
    args = parser.parse_args()

    # calling the main method with the args
    main(args)

その他の情報

UbuntuとEmacsで書いてます
python は3.9.6です

Comments

[int32_t]

「何をしたら、どうなるはずが、どうなってしまう」のかを具体的な入力や出力を入れて書きましょう。

おっと、自分が質問をよく読んでませんでした。「実行の結果」のところでだいたい伝わりますね。失礼しました。

[mcbkar]

確かに技術の世界では「できません」とか「うまくいかない」とか思考放棄で丸投げするのはよくないですよね；；

Answer 1

    password += secrets.choice(symbols)
    password += secrets.choice(numbers)
    password += secrets.choice(lowercase_characters)
    password += secrets.choice(uppercase_characters)

このコードで記号1文字・数字1文字・小文字1文字・大文字1文字が必ず入るようになってます。ここでも args.exclude何とか をチェックして除外する必要があるでしょう。

Comments

[mcbkar]

このコードの必要性が全くわからなかったので、消してみたらうまく行きました！ありがとうございます！
(多分小さな文字数でも必ず記号とかいれて複雑なパスワードにするコードだと思いますが、)

というかコメントに一つ入れる的なことかいてありますね。ちゃんと読めば手を煩わせることもなかったのに、ごめんなさい；；