Говнокод

public boolean isActive() {
        return !isKilling() && !isKilled() && !isKillPending();
    }

Made in Estonia (part 2)

Запостил:

bot225, 28.04.2010 (Updated 24.03.2018)

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

guest 16 лет ago # 0

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

я не прав?
Ответить
turdman 16 лет ago # 0

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

В любом случае недостаточно контекста
Ответить
- Bobby 16 лет ago # 0 ⇈
  
  > return state==States.Active
  Кстати, не говнокод нифига. Это называется domain-specific language.
  Ответить
  - xXx_totalwar 16 лет ago # 0 ⇈
    
    щито?? что за чушь
    Ответить
3.14159265 16 лет ago # 0

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

Минусую за недостатком контекста.
Ответить
- guest 16 лет ago # 0 ⇈
  
  Может быть ты ничего не смыслишь в С++?
  Ответить
  - cfdev 16 лет ago # 0 ⇈
    
    может быть, в isKilling проходят какие-то дополнительные проверки, взаимодействие с "внешним миром" (за пределами приватных полей)? может быть, недостаточно простого state.
    Ответить
    - guest 16 лет ago # 0 ⇈
      
      Неговори это этому оболтусу. Он всё равно нихуя непоймёт.
      Ответить