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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
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. PHP / Говнокод #6649

   +159

   1. 1
   2. 2
   3. 3

   while ($row = @mysql_fetch_array($this->_sqlResult, $result_type)):
   	$return[] = $row;
   endwhile;

   Запостил: 1_and_0, 13 Мая 2011

   Tweet

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

   • eth0 13.05.2011 18:15 # 0

     endwhile, значит. Век живи - век учись.
     Это, наверное, пошло из бейсиков с их NEXT i.

     Ответить

     • rO_ot 14.05.2011 01:05 # 0

       Это для вьюшек
       <?while(true):?>
       <div>ololo</div>aaa
       <?endwhile?>

       Ответить

       • eth0 14.05.2011 12:00 # −2

         К счастью, я в этом не разбираюсь, но не будет ли кошернее и проще вариант типа
         <?while(true){?>
         <div>ololo</div>aaa
         <?}?>
         нежели приведённый?

         Ответить

         • rO_ot 14.05.2011 15:30 # −1

           Нет, не очевидно какая дужка к какому оператору относится, вьюшки они могут быть большими.

           Ответить

           • Lure Of Chaos 14.05.2011 18:37 # 0

             идешки умеют подсвечивать парные скобки. Ключевые слова они не подсвечивают

             Ответить

             • rO_ot 14.05.2011 19:17 # −1

               Получаается некрасиво,
               и операторы двойные по-моему нетбинс последний умеет подсвечивать.

               Ответить

           • eth0 15.05.2011 11:56 # 0

             Два вложенных while. Или два while подряд. Тогда уже совсем неочевидно.
             Давайте тогда введём endif, endfor или endswitch.

             Ответить

             • rO_ot 15.05.2011 13:12 # −1

               А если вайл форич иф и еще ченибудь с дужками тогда точно будет путаница, а с if endif, for endfor, foreach endforeach будет понятнее и красивее.
               
               // Ввели уже:)

               Ответить

               • Vindicar 15.05.2011 15:15 # 0

                 Ну да, будет путаница, ведь индентацию кода придумали для лохов. =)

                 Ответить

                 • rO_ot 15.05.2011 15:36 # −2

                   В более менее серьезных проектах во вьюшках используются именно парные операторы, а не дужки. Даже к доке по Zend_View используются они, зачем делать иначе?

                   Ответить

                   • bugmenot 15.05.2011 17:42 # 0

                     monkey see - monkey do

                     Ответить

                     • rO_ot 15.05.2011 18:10 # −2

                       Это называется опыт;)

                       Ответить

                       • bugmenot 15.05.2011 19:16 # +1

                         опыт - это когда попробовал разное и выработал чёткие критерии, почему выбранный вариант перфектен.

                         Ответить

                         • rO_ot 15.05.2011 20:01 # −2

                           Выше же писал, дужечки путаются между собой, чтобы узнать кому какая дужечка принадлежит нужно пометить эту дужечку и искать какой оператор в иде "застветится", а с двойными операторами сразу понятно. Не просто же так в шаблонизаторах (Smarty, etc.) закрывающий тэг именуется "так же" как открывающий.

                           Ответить

                           • bugmenot 15.05.2011 20:17 # 0

                             это не операторы.
                             
                             две вложенные гомогенные конструкции вернут проблему.

                             Ответить

                             • rO_ot 15.05.2011 21:36 # −2

                               Но в 90% случаев конструкции будут разными, в то время как с дужками 100% будут одинаковыми

                               Ответить

                               • bugmenot 15.05.2011 22:23 # +1

                                 ты просто душка

                                 Ответить

                                 • rO_ot 15.05.2011 23:27 # −1

                                   знаю ^_^

                                   Ответить

                               • Lure Of Chaos 16.05.2011 19:56 # 0

                                 > while
                                 >endwhile
                                 одинаковые? еще скажите begin и end

                                 Ответить

                                 • rO_ot 17.05.2011 10:55 # −3

                                   В том то и дело что разные
                                   while ...
                                   ...
                                   foreach
                                   ...
                                   if ...
                                   endif
                                   endforeach
                                   endwhile
                                   
                                   
                                   Против
                                   while {
                                   ...
                                   foreach {
                                   ...
                                   if {
                                   ...
                                   }
                                   }
                                   }
                                   
                                   Добавить сюда кучу хтмл кода и дужечки будут очень путаться

                                   Ответить

                                   • Lure Of Chaos 17.05.2011 13:00 # +1

                                     соблюдаем отступы и разницы нет, разве что ключевые слова очень лень писать

                                     Ответить

                                     • guest 17.05.2011 14:44 # −1

                                       <? } ?>
                                       <? } ?>
                                       <? } ?>
                                       выглядит УГ

                                       Ответить

                                       • Lure Of Chaos 17.05.2011 14:46 # 0

                                         а зачем выключать-включать пхп?

                                         Ответить

                                         • guest 17.05.2011 15:11 # +1

                                           Это php, здесь нельзя поступать разумно.

                                           Ответить

                                       • eth0 17.05.2011 22:07 # 0

                                         > выглядит УГ
                                         Я не Штирлиц, настаивать на своём не буду. Но оба варианта выглядят довольно уныло.

                                         Ответить

   • guest 13.05.2011 18:24 # 0

     begin variable_name
     x
     end variable_name
     begin operator
     =
     end operator
     begin variable_value
     0
     end variable_value
     
     begin call_function
     echo
     end call_function
     begin argument_value
     x
     end argument_value

     Ответить

   • Lure Of Chaos 13.05.2011 18:31 # +4

     смешное в альт. синтаксисе и только?

     Ответить

     • gegMOPO4 14.05.2011 03:00 # 0

       .

       Ответить

   • rO_ot 14.05.2011 01:06 # +1

     PS Собачки - гавно

     Ответить

   • shmaltorhbooks 14.05.2011 15:48 # 0

     чем плохо использование endwhile вместо скобок? ну, кроме того, что выглядит непривычно?

     Ответить

     • guest 14.05.2011 17:23 # 0

       похоже на висуал басиц

       Ответить

     • Lure Of Chaos 14.05.2011 18:38 # 0

       слишком длинно. вьюшка должна быть разметкой таки, а не программой

       Ответить

       • shmaltorhbooks 15.05.2011 11:22 # 0

         причем тут вьюшки? я говорю в целом о языке, а не о вьюшке или каком-то там транспортном слое.
         кроме того, программу делает программой наличие некоей логики, а не тип фигурных скобок, использованных при написании.

         Ответить

   • guest8 08.04.2019 22:12 # −999

     показать все, что скрытоvanished

     Ответить

   • guest8 09.04.2019 10:37 # −999

     показать все, что скрытоvanished

     Ответить

   • guest8 09.04.2019 10:45 # −999

     показать все, что скрытоvanished

     Ответить

   • guest8 09.04.2019 17:55 # −999

     показать все, что скрытоvanished

     Ответить

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

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

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

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

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

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

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