Д
Зайдите в войс чат и спорьте там
Size: a a a
2021 January 03
Д
Надо было -4!!
предпочитаю "с запасом"
Ю
Почиьай про
__del__. Его вызов не гарантируетсяСпасибо, почитаю.
K
Спасибо, почитаю.
Вы увеличили карму Павел Дурманов до 10.54 (+9.54)
ПД
Ну я не прошу тебя прям серьезно браться, так, что в глаза бросается
А в глаза бросается форматирование, см. PEP8
𝕬
Смысл сообщения не про это
Большинство "хейтеров" ХХ в этом чате(или любой другой темы в жизни) следуют бездумно мнению большинства, не умея связать пары слов в категорию "аргументы". Ладно это, но когда они пытаются в "шутки" - нереально бесит от уровня интеллекта. Реально аргументировать в этом чате своё мнение может пара десятков. Аргументировать адекватно - единицы.
ПД
А в глаза бросается форматирование, см. PEP8
Но это после del”a, так как оно может сломать чтото а отличии от форматирования
Ю
Но это после del”a, так как оно может сломать чтото а отличии от форматирования
Ну del() я прям так и вызываю, явно.
А
Кому нечем заняться, поревьюйте плз. Буду признателен за любую конструктивную критику...
https://pastebin.com/WBPethXg
https://pastebin.com/WBPethXg
С ходу: глобалы можно выкинуть
150 строка:
Заменяется на
Почитай как нужно сравнивать с True, False и None
Лень ещё вчитываться
150 строка:
if … and (btn_move.st, btn_rotate.st, btn_copy.st) == (False, False, False):Заменяется на
if … and not (btn_move.st or btn_rotate.st or btn_copy.st):Почитай как нужно сравнивать с True, False и None
Лень ещё вчитываться
ПД
Ну del() я прям так и вызываю, явно.
Все еще
БГ
Кому нечем заняться, поревьюйте плз. Буду признателен за любую конструктивную критику...
https://pastebin.com/WBPethXg
https://pastebin.com/WBPethXg
1650 строк в одном файле 0_0
Ю
1650 строк в одном файле 0_0
Да, я думаю уже как это разделить))
КБ
Да, я думаю уже как это разделить))
Наверное самое простое - разнести классы по файлам
𝕬
Кому нечем заняться, поревьюйте плз. Буду признателен за любую конструктивную критику...
https://pastebin.com/WBPethXg
https://pastebin.com/WBPethXg
- PEP8
- тайпинг (опционально, но крайне желательно)
- чуть больше докстрингов (в плане ясности и описания)
- стоило бы разбить работу на файлы, попробуй хоть немного в архитектуру
- более осмысленные названия переменных (не однобуквенные - уже спасибо)
- в код не вчитывался, но мне кажется, что его кол-во можно уменьшить за счёт тех же функций и тд
- тайпинг (опционально, но крайне желательно)
- чуть больше докстрингов (в плане ясности и описания)
- стоило бы разбить работу на файлы, попробуй хоть немного в архитектуру
- более осмысленные названия переменных (не однобуквенные - уже спасибо)
- в код не вчитывался, но мне кажется, что его кол-во можно уменьшить за счёт тех же функций и тд
ᅠ
Большинство "хейтеров" ХХ в этом чате(или любой другой темы в жизни) следуют бездумно мнению большинства, не умея связать пары слов в категорию "аргументы". Ладно это, но когда они пытаются в "шутки" - нереально бесит от уровня интеллекта. Реально аргументировать в этом чате своё мнение может пара десятков. Аргументировать адекватно - единицы.
Достаточно глянуть видео про питон
ПД
- PEP8
- тайпинг (опционально, но крайне желательно)
- чуть больше докстрингов (в плане ясности и описания)
- стоило бы разбить работу на файлы, попробуй хоть немного в архитектуру
- более осмысленные названия переменных (не однобуквенные - уже спасибо)
- в код не вчитывался, но мне кажется, что его кол-во можно уменьшить за счёт тех же функций и тд
- тайпинг (опционально, но крайне желательно)
- чуть больше докстрингов (в плане ясности и описания)
- стоило бы разбить работу на файлы, попробуй хоть немного в архитектуру
- более осмысленные названия переменных (не однобуквенные - уже спасибо)
- в код не вчитывался, но мне кажется, что его кол-во можно уменьшить за счёт тех же функций и тд
Нахуя ему тайпинг
Ю
- PEP8
- тайпинг (опционально, но крайне желательно)
- чуть больше докстрингов (в плане ясности и описания)
- стоило бы разбить работу на файлы, попробуй хоть немного в архитектуру
- более осмысленные названия переменных (не однобуквенные - уже спасибо)
- в код не вчитывался, но мне кажется, что его кол-во можно уменьшить за счёт тех же функций и тд
- тайпинг (опционально, но крайне желательно)
- чуть больше докстрингов (в плане ясности и описания)
- стоило бы разбить работу на файлы, попробуй хоть немного в архитектуру
- более осмысленные названия переменных (не однобуквенные - уже спасибо)
- в код не вчитывался, но мне кажется, что его кол-во можно уменьшить за счёт тех же функций и тд
Спасибо. Там есть функции))
K
Спасибо. Там есть функции))
Вы увеличили карму 𝕬𝖗𝖙𝖊𝖒 до 739.79 (+9.54)
К
Большинство "хейтеров" ХХ в этом чате(или любой другой темы в жизни) следуют бездумно мнению большинства, не умея связать пары слов в категорию "аргументы". Ладно это, но когда они пытаются в "шутки" - нереально бесит от уровня интеллекта. Реально аргументировать в этом чате своё мнение может пара десятков. Аргументировать адекватно - единицы.
Да их везде
𝕬
Нахуя ему тайпинг
у меня после одного проекта с говнокодом на 13к строк на джанге и 8к+ на вью - аллергия на код без прописанных типов и докстрингов в тех или иных местах. И я же написал "опционально"