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

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

PHP (5703)
C++ (2737)
Куча (2343)
JavaScript (2028)
C# (1928)
Java (1502)
Си (1108)
Pascal (647)
Python (583)
1C (535)
SQL (432)
ActionScript (292)
Objective C (202)
bash (198)
Perl (194)
Assembler (146)
Ruby (145)
VisualBasic (138)
Lua (49)
Go (30)
Swift (27)
Haskell (26)
Kotlin (14)

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

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