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

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

PHP (5703)
C++ (2740)
Куча (2337)
JavaScript (2030)
C# (1929)
Java (1503)
Си (1103)
Pascal (649)
Python (585)
1C (536)
SQL (434)
ActionScript (294)
Objective C (204)
bash (200)
Perl (196)
Assembler (148)
Ruby (144)
VisualBasic (140)
Lua (51)
Go (32)
Swift (29)
Haskell (28)
Kotlin (15)

SQL / Говнокод #1878

−863.7
1. 1
2. 2
3. 3
```
Нашёл в проекте
SELECT DISTINCT sale_date, SUM(sale_price), buyer FROM ... 
GROUP BY sale_date, buyer;
```
DISTINCT здесь лишний

Запостил: Oleg_quadro, 21 Сентября 2009

Tweet
Комментарии (12) RSS
- guest 22.09.2009 14:49 # 0
  
  нисмишно
  Ответить
- guest 22.09.2009 21:23 # 0
  
  Даешь запрос целиком.
  Ответить
  - Oleg_quadro 22.09.2009 22:20 # 0
    
    SELECT DISTINCT sale_date, SUM(sale_price), buyer FROM fa_components
    GROUP BY sale_date, buyer;
    Ответить
    - guest 23.09.2009 05:52 # 0
      
      Вы правы, distinct не нужен.
      Ответить
      
      Oleg_quadro 28.09.2009 11:43 # −2
      
      Покажите пример когда он будет нужен при group by
      Ответить
- guest 29.11.2009 01:59 # +2
  
  Ребята, вы не поняли:
  DISTINCT тут явно остался просто как артефакт.
  Он был до GROUP BY при отладке, а потом остался невыпиленным.
  Проблемы в нем нет - т.к. GROUP BY делает дистинкт неявно.
  Ваш К.О.
  Ответить
- guest 12.12.2009 00:49 # 0
  
  Использование Distinct вне оператора count в 90% случаев это охинея. Часто видно как в запросе выводится куча полей из множества связанных таблиц и кодеру лень было подумать и он тупо решил избавиться от размножения записей (обычно из-за неверного условия) применив Distinct.
  Ответить
- guest 24.12.2009 16:20 # 0
  
  Судя по запросу, архитектура проекта - говнокод.
  SUM(sale_price) !
  Ответить
  - Oleg_quadro 29.12.2009 01:25 # 0
    
    основное средство состоит из нескольких компонентов, стоимость, которых в сумме составляет стоимость основного средства.
    Ответить
- wuhu 14.05.2010 21:53 # 0
  
  ну и чо что distinct остался как артефакт? в чем гавнакод то?
  Ответить
  - guest8 02.06.2019 23:21 # −999
    
    показать все, что скрытоvanished
    Ответить
  - PA3yMHblu_nemyx 03.06.2019 00:16 # 0
    
    SELECT DISTINCT 2*2 AS `four` FROM `petrosyan` WHERE `four` = 2*2 ORDER BY `four` LIMIT 1
    
    Так смешнее?
    Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Помни, guest, за тобой могут следить!

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

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

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

SQL / Говнокод #1878

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

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