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

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

PHP (5704)
C++ (2738)
Куча (2348)
JavaScript (2028)
C# (1928)
Java (1502)
Си (1111)
Pascal (647)
Python (583)
1C (536)
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)

PHP / Говнокод #12455

+50
1. 1
```
if(!empty($filter['features']) && !empty($filter['features'])) ...
```
Simpla CMS /api/Products.php

Запостил: Andretti, 21 Января 2013

Tweet
Комментарии (14) RSS
- dev6alexander 21.01.2013 17:40 # 0
  
  Ну о печатался чувак, зачем сразу на говно код?
  Ответить
  - 3.14159265 22.01.2013 18:28 # +2
    
    Метод empty может быть переопределен с побочным эффектом, который меняет переданный по ссылке аргумент.
    Ответить
    - dev6alexander 22.01.2013 18:30 # 0
      
      Можно примерчик?
      Ответить
      
      3.14159265 22.01.2013 18:46 # +2
      
      http://ideone.com/y8Amyf
      Пожалуйста
      Ответить
      
      dev6alexander 22.01.2013 18:52 # 0
      
      Да уж хрень какая то...
      Я даже не думал, что php дает возможность переопределить функции SPL. Причем если в Вашем примере поменять название функции на пример на pi() то метод работать уже не будет
      Ответить
      
      3.14159265 22.01.2013 19:13 # +3
      
      Не воспринимайте серъезно.
      Это была шутка.
      Ответить
      
      inkanus-gray 22.01.2013 19:54 # +2
      
      У меня всё работает: http://ideone.com/y7KH5F
      Ответить
      
      dev6alexander 22.01.2013 19:56 # 0
      
      Остается вопросом зачем разработчики оставили такую возможность
      Ответить
      
      inkanus-gray 22.01.2013 20:00 # 0
      
      Чтобы писать такой код: http://ideone.com/l68uUd
      
      Ваш капитан Очевидность.
      Ответить
      
      dev6alexander 22.01.2013 20:02 # 0
      
      Да не здесь более глубокий смысл или его отсутствие
      Ответить
- Centry 22.01.2013 14:13 # +5
  
  Хорошая проверка - двойная проверка. И программисты на языке Ctrl+C/Ctrl+V это знают.
  Ответить
  - Lure Of Chaos 23.01.2013 00:17 # +1
    
    you can never be sure
    Ответить
- alpex 28.01.2013 03:43 # +1
  
  Там вся cms так написана
  Ответить
- killro0000 02.02.2013 18:28 # 0
  
  Автор - параноик.
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

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

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

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

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

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

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

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

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

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

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

PHP / Говнокод #12455

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

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