Почему условие выполняется не корректно?

Question

[Emince]

Требуется написать проверку вводимого пароля на количество символов (не менее 8), не менее 3-х цифр, не менее одной заглавной буквы. Слепил то, что ниже. Но условия вместе не выполняются, выполняется каждое по отдельности, как будто and работает как or. Не могу понять, где ошибка.

def number_count(string):
    count = 0

    for ch in string:
        if ch.isdigit():
            count += 1
    return count


while 1:
    password = input('Придумайте пароль: ')
    if len(password) < 8 and number_count(password) < 3 and password.islower() == 1:
        print('Пароль ненадёжный. Попробуйте ещё раз.')
    else:
        print('Пароль надёжный.')
        break

Comments

[Модератор]

Обратите внимание на п.3.4 Регламента.

Answer 1

[Emince]

Задачу решил, всем спасибо!

def number_count(string):
    count = 0

    for ch in string:
        if ch.isdigit():
            count += 1
    return count


alphabet = ''.join([chr(i) for i in range(ord('А'), ord('Я') + 1)] + \
           [chr(i) for i in range(ord('а'), ord('я') + 1)])


while 1:
    ru_count = 0
    password = input('Придумайте пароль: ')
    for chr in password:
        for smb in alphabet:
            if smb == chr:
                ru_count += 1
    if ru_count > 0:
        print('Допускается ввод только латинских букв')
    else:
        if number_count(password) < 3 or password.islower() == 1 or password.isdigit() == 1 or len(password) < 8:
            print('Пароль ненадёжный. Попробуйте ещё раз.')
        else:
            print('Пароль надёжный.')
            break

Answer 2

[Александр Талалаев]

Вы в условии используете оператор and который будет false если хотя бы одно из условий неверно.

Поэтому меняйте его на or, те хотя бы одно из этих условий true, должен туда заходит. А иначе, если хотя бы одно false, то условие уйдёт в пароль надёжный.

if (len(password) < 8) or (number_count(password) < 3) or (password.islower() == 1):

Comments

[Emince]

Спасибо!

[Emince]

Все равно если ввести 8 цифр, выпадает в "пароль надежный", пробовал первое и второе условие объединять в and, результат тот же. Думаю проблема где то в < 3 и < 8...

def number_count(string):
    count = 0

    for ch in string:
        if ch.isdigit():
            count += 1
    return count

alphabet = ''.join([chr(i) for i in range(ord('А'), ord('Я') + 1)] + \
           [chr(i) for i in range(ord('а'), ord('я') + 1)])


while 1:
    password = input('Придумайте пароль: ')
    ru_count = 0
    for chr in password:
        for smb in alphabet:
            if smb == chr:
                ru_count += 1
    if ru_count > 0:
        print('Допускается ввод только латинских букв')
    else:
        if len(password) < 8 or number_count(password) < 3 or password.islower() == 1:
            print('Пароль ненадёжный. Попробуйте ещё раз.')
        else:
            print('Пароль надёжный.')
            break

[Сергей П]

Emince,

если ввести 8 цифр, выпадает в "пароль надежный"

Так у вас условие, что пароль НЕнадёжный, если МЕНЬШЕ 8 сиволов. То есть если 8 или больше символов, то надёжный.
Просто переведите на русский язык это условие и прочитайте. "И" тут неуместно.
"И" будет истино когда одновременно истины и левая и правая его часть.
"ИЛИ" будет истино, если любая хотя бы одна ее часть будет истина.

[Emince]

Я понял, что проблема в len(password) < 8, но как ее сделать позже чем эти проверки?

def number_count(string):
    count = 0

    for ch in string:
        if ch.isdigit():
            count += 1
    return count


alphabet = ''.join([chr(i) for i in range(ord('А'), ord('Я') + 1)] + \
           [chr(i) for i in range(ord('а'), ord('я') + 1)])


while 1:
    ru_count = 0
    password = input('Придумайте пароль: ')
    for chr in password:
        for smb in alphabet:
            if smb == chr:
                ru_count += 1
    if ru_count > 0:
        print('Допускается ввод только латинских букв')
    else:
        if number_count(password) < 3 or password.islower() == 0 or len(password) < 8 or password.isdigit() == 1:
            print('Пароль ненадёжный. Попробуйте ещё раз.')
        else:
            print('Пароль надёжный.')
            break

Answer 3

[Максим Припадчев]

while True:
    password = input('Придумайте пароль: ')
    crit1 = len(password) >= 8
    crit2 = sum(s.isnumeric() for s in password) >=3
    crit3 = sum(s.isupper() for s in password) >=1

    if not (crit1 & crit2 & crit3):
        print('Вы ввели не надежный пароль')
    else:
        print('Пароль надежный')
        break

Comments

[Сергей П]

YНе надо учить новичков плохому.
Это довольно грязный хак, причем с не всегда очевидными побочными эффектами.
Штатные булевы операции (and, or) участвуют в ленивой оптимизации на уровне интерпретации.
То есть, когда результат уже очевиден и дальше можно высисления не проихводить, тогда интерпретатор возвращает уже понятный результат.
Например a() and b()
Если результат a() будет False, то функция b не будет даже вычисляться. Потому что уже очевидно, что результат всего выражения будет False из-за первого аргумента.
В вашем случае "&" не оптимизируется и вы бестолку вычиляете то, что не нужно вычислять.
Нельзя стать программистом не понимая этой особенности большинства языков программирования.

Именно поэтому ваш код учит плохому.

[Максим Припадчев]

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

[Сергей П]

Максим Припадчев,
Если выносить критерии, то лучше уж вот так:

crits = [
    lambda password: len(password) >= 8,
    lambda password: sum(s.isnumeric() for s in password) >=3,
    lambda password: sum(s.isupper() for s in password) >=1,
]

while True:
    password = input('Придумайте пароль: ')
    if not all(crit(password) for crit in crits):
        print('Вы ввели не надежный пароль')
    else:
        print('Пароль надежный')
        break

Все криитерии имеет смысл проверять только в случае, если нужно выдать полный отчет о несоответствиях пользователю:

crits = [
    ((lambda password: len(password) >= 8), '- слишком короткий'),
    ((lambda password: sum(s.isnumeric() for s in password) >=3), '- должно быть >2 цифр'),
    ((lambda password: sum(s.isupper() for s in password) >=1), '- должно быть >1 символа в верхнем регистре'),
]

while True:
    password = input('Придумайте пароль: ')
    problems = [comment for crit, comment in crits if not crit(password)]
    if problems:
        print('Вы ввели не надежный пароль:')
        print('\n'.join(problems))
    else:
        print('Пароль надежный')
        break

[Максим Припадчев]

Сергей Паньков, Смотрите, я вас понял прекрасно, что вы имеете ввиду. Вы мне говорите, что я зря посчитал критерии, потому что в булевом выражение например простейшем (False & .. ) то что справа никогда не будет выполнено, так как python уже поймет, что выражение False, как результат происходят лишние вычисления критериев, в моем коде они то все вычисляются (А если держать их функциями и вставить функции в булевое выражение, питону надо будет вызовет, не надо будет не вызовет). Оптимизация ли это конечно оптимизация.

Повторюсь, для меня это был вопрос дизайна, как минимум в аналитике при декомпозиции, сложный ввод, или какие то выборки из структур данных, разбивают на критерии и выполняют их все, когда возможно конечно, (ваше замечание никто не отменял), потому что в большинстве случаев нам необходимо не только выполнить выражение, но и знать по какому именно критерию у нас False. (Собственно, вы уже предложили, добавить сообщение пользователю, о том в чем причина почему пароль не надежный).

По этому при все уважении, я не согласен, что я учу плохому, мой код плохая практика, грязный хак, как вы выразились. И останусь на своем.

Ну а мнение, всегда ценно (критическое ценно вдвойне) . Спасибо.

[Сергей П]

Максим Припадчев, А почему вы в своём коде не использовали булевы операции? Результат был бы идентичный.
Ваше выражение:
not (crit1 & crit2 & crit3)
по результату не отличается от:
not (crit1 and crit2 and crit3)
А если инвертировать критерии обратно, как было у автора в вопросе, то:
crit1 or crit2 or crit3
И не надо думать для чего использовались эти & и какие проблемы они могут принести.

[Максим Припадчев]

Сергей Паньков, А вот это уже аргумент, вы правы. В моем случае, когда дизайн подразумевает при любом раскладе вычисление всех критериев, странно использовать побитовое AND, надо использовать логическое AND, раз уже все посчитано, так пусть полностью выполняет выражение, а не если слева False то стоп, если True идем дальше. Согласен. Надо было мне в этом моменте быть внимательно, привык в pandas использовать ПОБИТОВЫЕ операторы и влепил. Ну это не оправдание, в этом моменте вы правы.

[Сергей П]

Максим Припадчев, погодите. Как раз логические and и or работают с оптимизацией. А бтовые "&|" вычисляют всё выражение независимо от промежуточныхрезультатов.

[Сергей П]

Надо, кстати, отметить, что такие тупые запретительные критерии - плохая идея в целом.
Гораздо правильнее - это считать энтропию введённого пароля, и, если она превышает пороговую, принимать пароль.
У примеру, что плохого в пароле без верхнего регистра и спецсимволов, если он длиной больше 20 символов?
Что плохого, если в пароле нет цифр, если он длинный, или что нет спец-символов, если итоговая энтропия получается достаточно большая.
Зато такие пароли гораздо легче запоминать, не придётся записывать на бумажках под клавиатурой, не придётся часто восстанавливать, потому что забыл куда поставил восклицательный знак в этот раз, а какое 'o' заменил на нолик, чтобы большой запоминающийся пароль стал удовлетворять тупым требованиям сайта.

[Максим Припадчев]

Сергей Паньков, Отлично, правильно логические на то и логические (что бы тестировать логически) а по битовые что бы тестировать оба операнда. Что же меня ждет явное повторение булевых и побитовых операций в чистом python. Спасибо надо устранять, я так ошибаться не должен.

[Максим Припадчев]

Сергей Паньков, Не ну насчет критериев, понятно надо вводить критерий сложности. Хотя бы что то вроде этого. Энтропия = log_2(Количество возможных символов^длинна пароля). Ну и сравнивать, оптимизимировать, относительно желаемых условий и т.д.

[Максим Припадчев]

Сергей, Почитал , немножко разочарован (с ваших слов я ожидал, что что-то серьезное упустил) побочный эффект как раз таки имеет логический оператор, собственно вы этот эффект и предлагали изначально использовать что бы не производить лишние вычисления. По битовое AND (&) он берет бинарное представление целого числа далее a_i * b_i для бинарного представления числа. По битовое OR (|) например a_i + b_i - (a_i * b_i). Бинарное представление True ->1, False ->0, то любая операция даст на выходе 1 или 0, как и ожидается.

False, True являются подклассом int. 0, 1 соответственно. ,бинарная форма 0 -> 0 , бинарная форма 1 -> 1. Откуда тут взяться побочным эффектам.

[Сергей П]

Максим Припадчев, В данном случае их нет, но они могли бы быть. Просто нужны особенные дополнительные ментальные усилия. чтобы удивиться там, где ожидаешь and\or встретив "&|".
Не берите в голову. Есть хоть одна причина использования битовых операций вместо логических в этом примере? Нет.
Кстати, а побочные эффекты могут возникнуть при ошибках в выражениях, описывающих критерии.

[Максим Припадчев]

Сергей П, Нет, я не то что бы спорить зачем есть логические операторы и их надо использовать в первую очередь, а если есть какие то причины требующие побитовые то только тогда побитовые, я согласен с вами, потенциальный эффект логического оператора ну например False AND some_function() если функция скажем не валидная должно быть исключение а вместо исключения у нас будет False, это негативный сценарий на позитивный указали вы, мы избегаем лишних вызовов функций. Полностью согласен.

Буду честен я использовал побитовые машинально, потому что основной код который пишу это pandas а там вместо логических используются побитовые. Но я не должен машинально использовать, а должен понимать, что делаю, python мой основной язык. По большему счету мне ПОВЕЗЛО, что код не имеет побочных эффектов. Повезло, это не профессионально и не куда не годится.

Я вечерком открыл побитовую арифметику и не нашел потенциальных побочных эффектов. Где у нас могут проблемы (как мне думается) если мы вылетим где то из 0 или 1. например 10 & 9 он вернет 8, почему потому что в результате побитового умножения мы мы получим 8 (сначала в бинарном представлении) ну и он конвертирует в целое число 8. Ну и такие кейсы это не то что мы хотим. Но за счет того что любой критерий это True или False то есть 1 или 0, я не увидел не каких кейсов где это будет иметь непредсказуемое поведение.

В любом случае спасибо за дискуссию, лучше лишний раз что то повторить и разобраться.

[Сергей П]

#@Maksim_64, да, похоже тема исчерпана.
Разве что не соглашусь, что отсутствие исключения при оптимизации - это негативный побочный эффект. На базе такого эффекта можно добиться что эффекта, схожего с монадой Maybe на минималках:
К примеру, у вас есть список аргументов

arg = sys.argv
# Вам нужно получить первый после нулевого элемент, если он есть, 
# а если его нет, то взять значение из настроек, или что-то по умолчанию:
conf = {'cfg': 'config.ini'}
param = arg[1:] and arg[1] or conf and conf.get('cfg') or 'default.ini'

И такая запись вполне прозрачно читается.
На тернарных операторах или if'ах это было бы громоздко и нечитабельно.
В SQL есть удобная функция coalesce, а в питоне её может щаменить цепочка выражений, соединенных через `or`.
Сплошная польза.