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

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

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

1. PHP (5703)
2. C++ (2737)
3. Куча (2343)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1108)
8. Pascal (647)
9. Python (583)
10. 1C (535)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. PHP / Говнокод #12891

   +159

   1. 1

   if (!is_array($array)) {$array = array($array)}

   Запостил: partizan22, 14 Апреля 2013

   Tweet

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

   • WGH 14.04.2013 16:43 # +4

     Надо было писать while.

     Ответить

   • Serabas 14.04.2013 19:58 # +2

     а я вот в js как то делал такое. нужно было передавать именно массив из элементов, даже если элемент один единственный (jQUery работает по тому же принципу). Придумалась такая проверка, если аргумент не массив, тогда что нам мешает обернуть его в массив и передать как ни в чем не бывало?

     Ответить

   • bot-minurast 14.04.2013 22:45 # 0

     Ну да, ну демоническая типизация. Пердача аргументов, хуле.
     function foo ( $array=array() ) {
     if (!is_array($array)) $array = (array)$array;
     // ...
     }

     Ответить

     • deep 15.04.2013 00:01 # 0

       function foo( $arg=array() ) {
       echo $arg[0];
       }
       
       $x = (object) array("a"=>1, "b"=>2);
       foo($x); // Fatal error:  Cannot use object of type stdClass as array

       
       Хуле...

       Ответить

       • bot-minurast 15.04.2013 08:38 # 0

         Ну это же ясень пень.
         
         http://liveworkspace.org/code/3LzrMM$0

         Ответить

   • DrFreez 16.04.2013 03:48 # 0

     Всегда так делаю :3 В чём говнокод то?

     Ответить

     • Sh1tM4ker 16.04.2013 15:59 # 0

       Проверка не нужна:
       $array = (array)$array;

       Ответить

       • scriptin 16.04.2013 20:26 # +1

         Этот код не эквивалентен.
         

         $ php -r '$a = new StdClass(); $a->p = 123; var_export((array) $a);'
         array (
           'p' => 123,
         )

         Ответить

         • Sh1tM4ker 16.04.2013 21:13 # 0

           Не знал. Тогда в чём говнокод? Кроме отсутствующей точки с запятой, что в общем-то просто опечатка.

           Ответить

           • scriptin 16.04.2013 23:16 # +2

             Лично мне забавно видеть большое количество упоминаний слова "array" в таком маленьком отрывке кода. Ну и название $array для переменной не очень удачно выбрано. В целом больше ничего плохого не вижу.

             Ответить

   • guest 10.05.2013 03:12 # 0

     Все проще.
     Теперь можно функцию вызывать так:
     foo(['bar', 'baz']) или foo(['bar']) или так foo('bar')

     Ответить

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

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

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

   Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

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

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