Не переходит в цикл, хотя условия выполнены. Где ошибка?

Question

[stepantu]

import random
import sys


def print_mimic(mimic_dict, word):
    bred = []
    x = 0
    while x < 200:
        if x == 0:
            first = word                                                          ### В это части цикла всё хорошо
            dobav = random.choice(mimic_dict[first]) 
            bred.append(dobav)
            x += 1
        else:
            print (dobav in mimic_dict)
            if dobav in mimic_dict == True:                              ### А вот тут проблема. Строчка 
                dobav = random.choice(mimic_dict[dobav])      ### выше выдает True но почему  то в цикл if  
                bred.append(dobav)                                          ### не переходит, а соскакивает 
                x +=1                                                                 ### в else. Как такое 
            else:                                                                ### может быть? Спасибо
                dobav = random.choice(mimic_dict[" "])
                bred.append(dobav)
                x += 1



    print (bred)
    return bred
    
mimic_dict = {" " : ["A", "B" , "C"], 
                      "A" : ["1", "2" "3"],
                      "B" : ["4", "5" "6"], 
                      "C" : ["7", "8" "9"]   }
print_mimic(mimic_dict, " ")

Comments

[Валерий Рябошапко]

Cравнивать с True или False — бесполезная затея, совершенно лишнее действие. Это, видимо, привычка, перешедшая из PHP. В Python это бесполезно, так как перед сравнением проводится приведение типов и

>>> 1 == True
True

Если нужно убедиться, что переменная является именно булевым значением, нужно или проверить тип (isisnstance()) или использовать операцию is (a is True).

Answer 1

[Роман]

Приоритет операций нужно учитывать, видимо:
1 in (1, 2, 3) == True
даст False, поскольку сначала выполнится (1, 2, 3) == True, что вернёт False, и затем получится проверка 1 in False, что есть False.
А вот так правильно:
(1 in (1, 2, 3)) == True
даст True.
А вот так ещё правильнее:

if 1 in (1, 2, 3):
    pass

Answer 2

[Сергей П]

В исходном коде много странных мест. Например, странным и опасным образом опущенны запятые в описании словаря, опасное применение переменной dobav в ветке else... Это всё ведёт к небезопасному коду в плане понимания и будущих модификаций. Очевидна нехватка опыта.

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

def print_mimic(mimic_dict, word):
    bred = []
    x = 0
    dobav = word
    while x < 200:
        if x == 0 or dobav in mimic_dict:
            dobav = random.choice(mimic_dict[dobav])
        else:
            dobav = random.choice(mimic_dict[" "])
        bred.append(dobav)
        x += 1
    print (bred)
    return bred

Я понятия не имею что это за код и для чего он. Предыдущий рефакторинг я сделал оставив код эквивалентным исходному (за исключением исправления очевидных ошибок). Можно предположить, что в коде есть ещё ряд логических ошибок. Возможно поведение по ветке "x == 0" не должно отличаться от поведения по ветке else условия "dobav in mimic_dict", а вместо константы " " следовало тоже поставить word. Тогда код становится гораздо более простым, понятным и элегантным.

def print_mimic(mimic_dict, word):
    bred = []
    dobav = word
    for x in range(200):
        dobav = random.choice(mimic_dict.get(dobav, mimic_dict[" "])) 
        bred.append(dobav)

    print (bred)
    return bred

Comments

[stepantu]

Спасибо за ваши комментарии. Да, опыта не много, вбурлился в программирование полтора месяца назад, до этого был только асемблер в институте. Со многими вашими комментариями согласен, но не понятно, что вы имели ввиду под "странным и опасным образом опущены запятые в описании словаря", буду рад вашим объяснениям.

Answer 3

[Антон Федорян]

if dobav in mimic_dict == True:
>>> False
if (dobav in mimic_dict) == True:
>>> True

Answer 4

[abcd0x00]

if dobav in mimic_dict == True:

if dobav in mimic_dict:
И всё.