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

    • ava guest 28.04.2010 14:57 # +4

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

      я не прав?
      Ответить

    • ava turdman 28.04.2010 15:12 # +1

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

      В любом случае недостаточно контекста
      Ответить

      • ava Bobby 05.05.2010 18:53 # 0

        > return state==States.Active
        Кстати, не говнокод нифига. Это называется domain-specific language.
        Ответить

    • ava 3.14159265 28.04.2010 15:44 # 0

      c шаблонами - это да - ТРУъ эстонское, а тут неясно.
      Оно хоть и попахивает, но вообще может оказатся вполне норм.
      Ответить

    • ava cfdev 28.04.2010 19:29 # +1

      Минусую за недостатком контекста.
      Ответить

      • ava guest 28.04.2010 20:59 # −8

        показать все, что скрытоМожет быть ты ничего не смыслишь в С++?
        Ответить

        • ava cfdev 28.04.2010 22:33 # 0

          может быть, в isKilling проходят какие-то дополнительные проверки, взаимодействие с "внешним миром" (за пределами приватных полей)? может быть, недостаточно простого state.
          Ответить

          • ava guest 05.05.2010 19:09 # 0

            Неговори это этому оболтусу. Он всё равно нихуя непоймёт.
            Ответить

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