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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2341)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. JavaScript / Говнокод #10196

   +157

   1. 1

   var c = 'function' === typeof o.pop ? [] : {};

   Запостил: хуита, 05 Мая 2012

   Tweet

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

   • Steve_Brown 05.05.2012 09:16 # −2

     Какой-нибудь костыль для мультибраузерности?

     Ответить

     • guest 05.05.2012 10:46 # −2

       Не, быстрая проверка на то, что o — массив.
       Хотя тут либо var c = o.pop? [] : {} для экономии места, либо полноценную проверку (например, через o instanceof Array).

       Ответить

       • Lure Of Chaos 05.05.2012 11:46 # −1

         а если "массив" - пустой?

         Ответить

       • wvxvw 05.05.2012 13:32 # 0

         Нет, проверка на то, что o.pop - функция, но от того, что такой метод есть у массива не следует, что o массив. Или, если именно так автор проверял, что o - массив, тогда печально.
         
         А, ну и еще три равно для сравнения со строкой... т.е. автор предполагал, что typeof x может вернуть new String("object") или что-то в этом духе. Вобщем, печально.

         Ответить

         • guest 05.05.2012 13:45 # +1

           Так проверка быстрая же :)
           А строгое равенство, видимо, для скорости.
           
           В общем, говнокод, конечно, потому что и в размере выигрыша нет и не универсально.

           Ответить

           • wvxvw 05.05.2012 14:03 # −1

             Для строк строгое равенство и нестрогое - это одна и та же функция. Разница только если пытаться сравнить строку с не-строкой, или если сравнивать с new String(...).

             Ответить

           • wvxvw 05.05.2012 20:55 # −1

             [].pop == Array.prototype.pop

             
             Кстати, если так сравнивать, то работало бы даже почти правильно (за исключением того, что прототип можно динамически "унаследовать")

             Ответить

   • rat4 05.05.2012 10:16 # 0

     >Запостил: хуита
     
     Логичнее смотрелся бы ник "хуиту"

     Ответить

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

   Ошибка компиляции комментария:

   1. Гости могут высказаться только в понедельник, среду, четверг или воскресение

   Где здесь C++, guest?!

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

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

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