Змея / Говнокод #28072 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
class ProjectIssue(
    UserAgentDetailMixin,
    SubscribableMixin,
    TodoMixin,
    TimeTrackingMixin,
    ParticipantsMixin,
    SaveMixin,
    ObjectDeleteMixin,
    RESTObject,
):

Запостил: ISO ISO, (Updated )

Комментарии (19) RSS

  • жирная ООПина какая, прямо сочится вся
    Ответить
  • Хуексины. Поставил плюс.
    Ответить
  • Я поставил минус и объясню почему

    в репе нет информации о зависимостях

    в репе нет информации о версии python, поддерживаемой скриптом

    в репе нет описания функций

    в репе нет тестов

    код скрипта не отформатирован согласно стандарту PEP8

    в репе не указано как конфигурировать скрипт под свой токен

    переменные в коде имеет неиформативные имена - kb, c, dc и прочее

    в коде по поводу и без стоят комменты (если требуется куча комментов то значит функция слишком сложна и ее стоит разделить на несколько)

    в коде несколько раз подряд стоят send_message(), хотя можно отправить сообщение за один раз

    в коде есть строка

    alphabet = ['a'# любой алфавит]

    и мне непонятно что это вообще такое и как может работать

    отступы в коде не соответствуют 4 пробелам

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

    try-блок судя по отлавливаемой ошибке должен быть в разы меньше

    в функции f_code и ff_decode есть участок с повторением одного и того же кода

    если скрипт упадет (например при проблеме с сетью) он сам не поднимется заново и будет лежать пока его не перезапустить

    И это я даже не пытался запустить

    Если бы вы на собеседование на должность джуна это мне принесли, то никаких шансов устроиться бы у вас не было
    Ответить
    • > он сам не поднимется заново и будет лежать пока его не перезапустить

      Какой undead )))

      Вообще, этим внешний супервизор должен заниматься, а не сам скрипт.
      Ответить
    • Лол, там ещё и тема проекта бессмысленная и беспощадная.

      Шифровать сообщения через бота в телеге -- это сильно. Я бы до такого не додумалась.
      Ответить
    • > alphabet = ['a'# любой алфавит]
      мне тоже непонятно, что это такое
      Наверное, "И это я даже не пытался запустить" - "Я тоже"
      Ответить
    • > код скрипта не отформатирован согласно стандарту PEP8

      У меня в Си нет никакого "стандартного форматирования".
      Ответить
  • Миксины, миксины, миксины!
    Сделаем ООП почище!
    Бьётся транслятор, в экстазе пылая.
    Бушуют питонисты!
    Ответить

Добавить комментарий

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?


    8