Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP (5703)
C++ (2737)
Куча (2343)
JavaScript (2028)
C# (1928)
Java (1502)
Си (1108)
Pascal (647)
Python (583)
1C (535)
SQL (432)
ActionScript (292)
Objective C (202)
bash (198)
Perl (194)
Assembler (146)
Ruby (145)
VisualBasic (138)
Lua (49)
Go (30)
Swift (27)
Haskell (26)
Kotlin (14)

Java / Говнокод #11441

+74
1. 1
2. 2
3. 3
```
//code...
item.setInUse((map.getnStreamActive().equals("1") ? true : false));
//...code
```
Писал тим лид одного из вендоров проекта.
nStreamActive - Integer

Запостил: ingenuus, 19 Июля 2012

Tweet
Комментарии (17) RSS
- ingenuus 19.07.2012 10:50 # 0
  
  Нашел потому что с NullPointerException валилось
  Ответить
  - absolut 19.07.2012 10:55 # 0
    
    кто был null?
    Ответить
    - mrWindMak3r 19.07.2012 10:58 # 0
      
      Вот понараздают званий @"тимлидов", а потом грабельками говешко закидывают и флажками помечают...
      Ответить
    - ingenuus 19.07.2012 11:05 # 0
      
      Очевидно, map.getnStreamActive()
      Ответить
      
      absolut 19.07.2012 11:11 # +2
      
      ну, может быть и map, и item
      Ответить
      
      ingenuus 19.07.2012 11:16 # +3
      
      может, но это было бы не так символично.
      Написал бы "1".equals() никто бы и не заметил :)
      Ответить
- roman-kashitsyn 19.07.2012 11:29 # 0
  Пехепешники всюду
  http://govnokod.ru/11386#comment146358
  Ответить
- roman-kashitsyn 19.07.2012 12:41 # 0
  
  А что, количество активных потоков может быть только 1 или 0?
  Ответить
  - ingenuus 19.07.2012 12:54 # 0
    
    Это класс для маппинга mssql. Активный/неактивный.
    Ответить
    - roman-kashitsyn 19.07.2012 12:57 # 0
      
      тогда почему бы не isActive() ?
      Ответить
      
      ingenuus 19.07.2012 13:01 # +1
      
      Наследство от предыдущих разработчиков. Стараемся не менять без реальной необходимости, то что уже написано. Чревато.
      Ответить
      
      absolut 19.07.2012 14:00 # 0
      
      какое исправление было сделано в этот раз?
      Ответить
      
      roman-kashitsyn 19.07.2012 14:04 # +7
      
      сменить обои на менее скучные
      Ответить
      
      Lure Of Chaos 19.07.2012 19:57 # +5
      
      да вы обои такие скучные
      Ответить
- Lure Of Chaos 19.07.2012 18:21 # 0
  
  тим не обязан уметь прожить
  Ответить
  - bormand 19.07.2012 18:42 # 0
    
    зато тим обязан уметь выжить
    Ответить
    - rat4 19.07.2012 19:41 # 0
      
      тим обязан не мешаться
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Семь раз отмерь — один отрежь, guest!

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

Java / Говнокод #11441

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

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