- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
class ProjectIssue(
UserAgentDetailMixin,
SubscribableMixin,
TodoMixin,
TimeTrackingMixin,
ParticipantsMixin,
SaveMixin,
ObjectDeleteMixin,
RESTObject,
):-
Python / Говнокод #28072
+4
Запостил:
ISO,
05 Марта 2022
Follow us!
в репе нет информации о зависимостях
в репе нет информации о версии python, поддерживаемой скриптом
в репе нет описания функций
в репе нет тестов
код скрипта не отформатирован согласно стандарту PEP8
в репе не указано как конфигурировать скрипт под свой токен
переменные в коде имеет неиформативные имена - kb, c, dc и прочее
в коде по поводу и без стоят комменты (если требуется куча комментов то значит функция слишком сложна и ее стоит разделить на несколько)
в коде несколько раз подряд стоят send_message(), хотя можно отправить сообщение за один раз
в коде есть строка
alphabet = ['a'# любой алфавит]
и мне непонятно что это вообще такое и как может работать
отступы в коде не соответствуют 4 пробелам
по моему мнению в коде местами через чур высокая вложенность инструкций
try-блок судя по отлавливаемой ошибке должен быть в разы меньше
в функции f_code и ff_decode есть участок с повторением одного и того же кода
если скрипт упадет (например при проблеме с сетью) он сам не поднимется заново и будет лежать пока его не перезапустить
И это я даже не пытался запустить
Если бы вы на собеседование на должность джуна это мне принесли, то никаких шансов устроиться бы у вас не было
Какой undead )))
Вообще, этим внешний супервизор должен заниматься, а не сам скрипт.
Шифровать сообщения через бота в телеге -- это сильно. Я бы до такого не додумалась.
мне тоже непонятно, что это такое
Наверное, "И это я даже не пытался запустить" - "Я тоже"
У меня в Си нет никакого "стандартного форматирования".
Сделаем ООП почище!
Бьётся транслятор, в экстазе пылая.
Бушуют питонисты!