1. 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

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

    • нисмишно
      Ответить
    • Даешь запрос целиком.
      Ответить
    • Ребята, вы не поняли:
      DISTINCT тут явно остался просто как артефакт.
      Он был до GROUP BY при отладке, а потом остался невыпиленным.
      Проблемы в нем нет - т.к. GROUP BY делает дистинкт неявно.
      Ваш К.О.
      Ответить
    • Использование Distinct вне оператора count в 90% случаев это охинея. Часто видно как в запросе выводится куча полей из множества связанных таблиц и кодеру лень было подумать и он тупо решил избавиться от размножения записей (обычно из-за неверного условия) применив Distinct.
      Ответить
    • Судя по запросу, архитектура проекта - говнокод.
      SUM(sale_price) !
      Ответить
      • основное средство состоит из нескольких компонентов, стоимость, которых в сумме составляет стоимость основного средства.
        Ответить
    • ну и чо что distinct остался как артефакт? в чем гавнакод то?
      Ответить

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