1. Java / Говнокод #3123

    +67

    1. 1
    2. 2
    3. 3
    public boolean isActive() {
            return !isKilling() && !isKilled() && !isKillPending();
        }

    Made in Estonia (part 2)

    Запостил: bot225, 28 Апреля 2010

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

    • мммм... а в чем говнокод, собственно?
      судя по именам функций, у системы может быть три состояния:
      - в процессе "убивания" (мб это долгий процесс);
      - "убито" (здесь понятно);
      - в ожидании "убийства" (мб там очередь долгих процессов "убийства");

      я не прав?
      Ответить
    • Этот код может быть как абсолютно нормальным (активность действительно определяется именно так), так и говном (return state==States.Active) или признаком большего говна (состояние определяется хитровыебнутым мотком флагов)

      В любом случае недостаточно контекста
      Ответить
      • > return state==States.Active
        Кстати, не говнокод нифига. Это называется domain-specific language.
        Ответить
    • c шаблонами - это да - ТРУъ эстонское, а тут неясно.
      Оно хоть и попахивает, но вообще может оказатся вполне норм.
      Ответить
    • Минусую за недостатком контекста.
      Ответить
      • показать все, что скрытоМожет быть ты ничего не смыслишь в С++?
        Ответить
        • может быть, в isKilling проходят какие-то дополнительные проверки, взаимодействие с "внешним миром" (за пределами приватных полей)? может быть, недостаточно простого state.
          Ответить
          • Неговори это этому оболтусу. Он всё равно нихуя непоймёт.
            Ответить

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