Почему не работает код?

Question

[Evilitself]

Написал такой код для генерации пароля, но не понимаю почему выдает ошибку.

import random

DIGITS = '0123456789'
LOWERCASE_LETTERS = 'abcdefghijklmnopqrstuvwxyz'
UPPERCASE_LETTERS = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ'
PUNCTUATION = '!#$%&*+-=?@^_'

chars = ''

len_pass_all = int(input('Укажите количество паролей для генерации: '))
len_pass_one = int(input('Укажите длину одного пароля: '))
len_pass_numbers = input('Включать ли цифры 0123456789? ')
len_pass_byk = input('Включить ли прописные буквы ABCDEFGHIJKLMNOPQRSTUVWXYZ? ')
len_pass_byk2 = input('Включать ли строчные буквы abcdefghijklmnopqrstuvwxyz? ')
len_pass_symbols = input('Включать ли символы !#$%&*+-=?@^_? ')
len_pass_no_symbols = input('Исключать ли неоднозначные символы il1Lo0O? ')

if len_pass_numbers.lower() == 'Да':
    chars += DIGITS
if len_pass_byk.lower() == 'Да':
    chars += UPPERCASE_LETTERS
if len_pass_byk2.lower() == 'Да':
    chars += LOWERCASE_LETTERS
if len_pass_symbols.lower() == 'Да':
    chars += PUNCTUATION
if len_pass_no_symbols.lower() == 'Да':
    for c in 'il1Lo0O':
        chars = chars.replace(c, '')

def generate_password(len_pass_one, chars):
    password = ''
    for j in range(len_pass_one):
        password += random.choice(chars)
    return password

for _ in range(len_pass_all):
    print(generate_password(len_pass_one, chars))

Comments

[SunTechnik]

А какую ошибку выдаёт, мы должны догадаться?

Answer 1

[dim5x]

Потому что len_pass_numbers.lower() (и прочие) возвращает вам строку 'да', а вы сравниваете с 'Да'.
Соответственно ваш "алфавит" - chars - пустой.

Ну и, вероятно, удобнее пользоваться встроенными коллекциями, чем вручную набивать:

from string import digits, ascii_lowercase, ascii_uppercase, punctuation

Comments

[Evilitself]

Спасибо большое, буду знать на будущее(почитаю еще об этом). Проблему в итоге самостоятельно смог решить). Но вопрос следующий, если получится ответить, Вам, на него, буду благодарен. Я написал строку так:

if len_pass_numbers.lower() == 'д' or len_pass_numbers.lower() == 'Да' or len_pass_numbers.lower() == 'Lf' or len_pass_numbers.lower() == 'lf' or len_pass_numbers.lower() == 'да':

При этом код весь работает, но я так же сравнивал с "Да".

[dim5x]

Evilitself, потому что вы использовали оператор or.
Это выглядит / читается так: "условие выполнено?" или "условие выполнено?" или ...
Какое-то из условий выполняется (возвращает True, те что не выполнены - False): значит строчка примет вид:if False or True or False or False or False: что будет в итоге соответствовать => if True:
Почему? Потому что оператор orвернёт True, если хотя бы один его операндов имеет значение True.
Равносильно записи: any((False, False, True, False)) => True.

Чтобы не писать такие длинные конструкции проще делать так:

if len_pass_numbers in ('д', 'Да', 'Lf', 'lf', 'да'):

[Evilitself]

dim5x, спасибо огромное. Буду знать теперь менее простую конструкцию для таких примеров). Логика присутствует в вашем рассказе. Можно было бы и самому понять))))