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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2338)
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. SQL / Говнокод #4239

   −176

   1. 1

   SELECT DISTINCT(sid) FROM

   sid - уникальное значение(PK)

   Запостил: ferry-very-good, 13 Сентября 2010

   Tweet

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

   • da4ever 13.09.2010 11:46 # 0

     ... FROM %TableName% WHERE (что-нибудь уникальное, что мы уже знаем)= %QueryParam%
     так мы получаем идентификатор из таблицы.
     
     и говнокод внезапно объясняется особенностями проектирования системы.
     а DISTINCT чтобы нежданчика не случилось, очевидно же.

     Ответить

     • ferry-very-good 13.09.2010 11:48 # 0

       не, там дальше %TableName% WHERE sid=>300
       и почему 300 не написано

       Ответить

       • da4ever 13.09.2010 11:57 # −4

         ну елси там дальше нет AND - то да это чертов говнокод действительно берет последние LIMIT сидов
         
         результат запроса (если это не подзапрос) случайно не итерируют для получения количества уникальных элементов? )

         Ответить

         • ferry-very-good 13.09.2010 11:58 # 0

           нет, это обычная выборка sid'ов

           Ответить

         • bugmenot 13.09.2010 12:24 # −5

           показать все, что скрытофу, LIMIT...

           Ответить

           • da4ever 13.09.2010 12:29 # −4

             на кухню?

             Ответить

     • Oleg_quadro 15.10.2010 16:20 # 0

       какой нежданчик? когда sid - это уникальное значение

       Ответить

       • da4ever 15.10.2010 16:27 # +2

         сейчас уже не вспомню, но "нежданчик" появслся примерно так:
         http://govnokod.ru/4058#comment43817

         Ответить

   • paranoid 13.09.2010 13:04 # 0

     а джойнов там случайно ли нет?

     Ответить

     • ferry-very-good 13.09.2010 13:14 # 0

       > не, там дальше %TableName% WHERE sid=>300

       Ответить

       • newmindcore 14.09.2010 08:33 # +2

         самый логичный вариант - это то, что раньше были left joins, а потом их не стало.
         а вообще да - часто такое встречается. ещё круче когда вместе с GROUP BY sid )

         Ответить

   • Lure Of Chaos 13.09.2010 14:32 # 0

     наверное, когда-то sid не был PK, а код остался

     Ответить

     • ferry-very-good 13.09.2010 14:33 # 0

       исключено

       Ответить

       • Lure Of Chaos 13.09.2010 14:35 # +1

         тогда налицо просто непонимания sql, другого оправдания сабжу нет

         Ответить

   • japan007 20.11.2010 04:12 # 0

     и? по производительности ничего не потеряли...

     Ответить

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

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

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

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

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

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

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