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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1926)
6. Java (1501)
7. Си (1097)
8. Pascal (647)
9. Python (582)
10. 1C (533)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   +146

   1. 1

   $news = new news($_GET["news"]);

   Обьект класса news. Выводит нужную новость в соответствии с get запросом.

   Запостил: Somov, 09 Ноября 2010

   Tweet

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

   • andremacareno 09.11.2010 20:11 # +1

     ...new news... вызывает улыбку

     Ответить

   • Анонимус 09.11.2010 20:45 # −4

     ООП в мире PHP напоминает мне басню "мартышка и очки"

     Ответить

   • telnet 09.11.2010 21:00 # +4

     > Обьект
     > Выводит
     > новость
     > в соответствии с get запросом
     Мечты о большой, чистой и светлой MVC убиты на корню.

     Ответить

     • istem 09.11.2010 23:17 # +1

       M > Обьект
       V > Выводит новость
       C > в соответствии с get запросом
       А чито ви хочете?..

       Ответить

       • telnet 10.11.2010 07:09 # 0

         Щитоби ано нилижяла в адной куче, да

         Ответить

         • istem 10.11.2010 14:18 # +3

           Вай вай, красафчек ваабще

           Ответить

   • Lure Of Chaos 09.11.2010 21:22 # +1

     > new news
     > Выводит нужную новость
     
     а в соответствии с лучшими практиками ООП программирования, в конструкторе НЕ должно быть кода, который делает что-либок, помимо инициализации. Сколько писали в каждой книжке....

     Ответить

   • AxisPod 10.11.2010 07:39 # −3

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

     Ответить

   • scalar4eblo4no 10.11.2010 18:11 # 0

     Что-то мне подсказывает что ООП здесь только для того что бы хоть где-нибудь его применить, и класс news используется один раз и в одном экземпляре.

     Ответить

   • japan007 20.11.2010 03:42 # −1

     и?

     Ответить

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

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

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

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

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

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

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