1. пропусков нет 2. except Exception 3. глобалы
глобалы юзаю т.к. переменная исползуется вне класса с данными дефами
except Exception для дебага
for x in widgets: widgets[x] втф
эталонный говнокод (с)
Так это перебор словаря же?
а что, уже запретили перебирать элементы словаря или элементы вместе с ключами?
ну есть dict.values() же
Тогда мы ключ знать не будем
Боль.
а он там ключ не использует же
Действительно
5. Словари вместо объектов
6. Какая-то странная вещь kv_er. Выглядит как генерация текста на каком-то языке разметки и как минимум не вижу экранирования
6. нэйминг От функции с названием гет виджетс лично я бы ожидал , что она будет что-то возвращать, а не принтить
ты случайно не JS-ер?
этого достаточно
есть dict.items()
а код пишешь как JS-еры.
я даже понятия не имею про жс
Тогда были бы замыкания на замыканиях
Ну я имею в виду, если бы кто-то другой, например, увидел бы твой код Увидел бы название функции, но не увидел бы код функции То он бы ожидал, что твоя функция вернет что-то вроде списка виджетов Я к тому, что не очевидный нэйминг функции. Имя функции должно отражать то, что функция делает Плохой нэйминг это тоже признак говнокода
код гуи приложения, врядли кто-то будет юзать данный код
это не повод использовать глобалы
а есть другой вариант?
тогда нафига ты его показываешь нам на ревью, если тебе наше мнение не нужно.
Это не повод делать плохо) Говнокодинг, знаешь ли, впитывается в привычку И да, как уже написали. Зачем тогда наше мнение?)
интересно мнение более развитых чем я😅
Обсуждают сегодня