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

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

PHP (5698)
C++ (2733)
Куча (2326)
JavaScript (2027)
C# (1925)
Java (1501)
Си (1095)
Pascal (647)
Python (582)
1C (531)
SQL (431)
ActionScript (292)
Objective C (202)
bash (197)
Perl (194)
Assembler (146)
Ruby (142)
VisualBasic (138)
Lua (49)
Go (29)
Swift (27)
Haskell (26)
Kotlin (13)

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

+70
1. 1
```
if (sAttrName.equalsIgnoreCase("10")) {
```
Запостил: someone, 23 Апреля 2012

Tweet
Комментарии (10) RSS
- Steve_Brown 23.04.2012 14:24 # +1
  Ну, если там много строчек с разными значениями, то чего уж страшного...
  
  if (sAttrName.equalsIgnoreCase("ten") || sAttrName.equalsIgnoreCase("X") || sAttrName.equalsIgnoreCase("10"))
  
  Хотя несколько странное имя атрибута.
  Ответить
  - someone 23.04.2012 14:50 # +1
    
    Это самопальный парсер DXF.
    Ответить
- 3.14159265 23.04.2012 14:59 # +6
  
  Любому мало-мальски знакомому с жавой режет глаз порядок:
  "10".equalsIgnoreCase(sAttrName)
  Ответить
  - Steve_Brown 23.04.2012 15:32 # 0
    
    Ах да, вдруг там null.
    Ответить
    - absolut 23.04.2012 15:51 # 0
      
      кстати, можно было бы для невиртуальных вызовов использовать объявленный тип переменной, и не опираться на текущее значение.
      Ответить
      
      Steve_Brown 23.04.2012 16:30 # 0
      
      Как в С++? Ну да.
      В C# то же самое. Там, правда, можно ввести extension method.
      Зато у QString нулевая строка не отличима от пустой кроме как функцией isNull :P
      Ответить
    - someone 23.04.2012 20:42 # 0
      
      Там не может быть null, это строка, прочитанная из файла. Null отсеивается ещё раньше.
      Ответить
- bormand 23.04.2012 16:13 # +1
  
  А почему именно equalsIgnoreCase, а не просто equals? Он же всяко медленнее, да и не нужен для чисел.
  Ответить
  - 3.14159265 23.04.2012 18:58 # +10
    
    Потому что этот сайт - говнокод.ру.
    Ответить
  - Lure Of Chaos 23.04.2012 23:03 # +1
    
    а вдруг оно не строка )
    Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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