mysql_real_escape_string / Говнокод #1878 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
Нашёл в проекте
SELECT DISTINCT sale_date, SUM(sale_price), buyer FROM ... 
GROUP BY sale_date, buyer;

DISTINCT здесь лишний

Запостил: Oleg_quadro Oleg_quadro, (Updated )

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

  • ava guest # Ссылка на оригинал 0

    Ребята, вы не поняли:
    DISTINCT тут явно остался просто как артефакт.
    Он был до GROUP BY при отладке, а потом остался невыпиленным.
    Проблемы в нем нет - т.к. GROUP BY делает дистинкт неявно.
    Ваш К.О.
    Ответить

  • ava guest # Ссылка на оригинал 0

    Использование Distinct вне оператора count в 90% случаев это охинея. Часто видно как в запросе выводится куча полей из множества связанных таблиц и кодеру лень было подумать и он тупо решил избавиться от размножения записей (обычно из-за неверного условия) применив Distinct.
    Ответить

  • ava guest # Ссылка на оригинал 0

    Судя по запросу, архитектура проекта - говнокод.
    SUM(sale_price) !
    Ответить

    • ava Oleg_quadro # Ссылка на оригинал 0 ⇈

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

  • ava wuhu # Ссылка на оригинал 0

    ну и чо что distinct остался как артефакт? в чем гавнакод то?
    Ответить

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

Переведи на "PHP", guest!

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

    ↑
    8
    ↓