Почему не работает GROUP BY?

Question

[Light 777]

Есть такой вот селект. Прописываю в конце group by rnn, но он не работает.
Почему не работает или на что его можно заменить? Хочу избавиться от дубликатов в селекте

SELECT  
           TO_DATE(CT.DOG_DATE, 'dd.mm.rrrr') DOG_DATE,
           case when ct.dog_date>='01.01.2013' then cl.iin
           else nvl(cl.iin,cl.rnn) end rnn,
           UPPER(TRIM(CL.CLIENT_NAME)) CLI_NAME
           
      FROM 
           CONTRACTS      CT,
           CLIENT         CL

     WHERE
       CT.CLIENT = CL.CLIENT
       AND CL.CLIENT_CATEGORY = 1
       
       and rownum < 500 group by rnn;

Answer 1

[Сергей Сенкевич]

Чтобы избавиться от дублей, есть DISTINCT. А если хотите именно через GROUP BY, то вам там нужно указать все три поля, присутствующие в SELECT, а не только rnn.

Comments

[Light 777]

а как можно DISTINCT использовать для rnn? Я привел упрошенный селект, по идее здесь где то 30 полей)

[Сергей Сенкевич]

Light 777: Я так понял, что вам нужны просто уникальные записи. А если нужен только уникальный rnn, то тогда без GROUP BY не обойтись. Но в этом случае, ко всем остальным 29-ти полям в SELECT должны применяться какие-то агрегатные функции (MIN, MAX, AVG, SUM, COUNT и т.д.). Т.е. чтобы GROUP BY работал, все поля из SELECT должны быть либо обернуты в одну из агрегатных функций, либо должны присутствовать в GROUP BY.

[Light 777]

Сергей Сенкевич: а больше никак нельзя ? :D Приложил оригинал кода - freetexthost.com/nn6tco140z

[Сергей Сенкевич]

Light 777: Нехилый запрос)). Только остается непонятным - что именно вы хотите получить на выходе? Какие-то статистические данные сгруппированные по rnn, либо просто, чтобы в результате не было одинаковых строк? Если второе (что скорее всего, т.к. в вашем запросе нет ни одной агрегатной функции), то вам нужен DISTINCT.

[Light 777]

Сергей Сенкевич: Просто хочу чтобы отображались записи с уникальным полем rnn, чтобы значения rnn не повторялись ))) А куда мне вставить DISTINCT?

[Сергей Сенкевич]

Light 777: Если нужно, чтобы rnn не повторялось, то к остальным полям должны быть применены агрегатные функции. Попробуйте выполнить запрос без DISTINCT и GROUP BY и посмотрите, какие поля не меняются при одинаковом значении rnn, а какие меняются. После этого, те поля, что не меняются, перечислите в GROUP BY, а к тем, что меняются, примените ту или иную агрегатную функцию. Допустим, имя клиента не меняется, значит его в GROUP BY. А, например, суммы платежей, будут меняться (если речь идет о списке платежей по заданным контракту и клиенту) - к ним применяем функцию SUM (или какую другую). А DISTINCT ставится сразу после SELECT (SELECT DISTINCT ...).

Answer 2

[Вячеслав Успенский]

почитал ветку комментов под предыдущим ответом, к сожалению не понимая что именно вы делаете трудно советовать, но: выберите подзапросом DISTINCT rnn а затем к нему джоином всю остальную выборку.

Comments

[Light 777]

а вы можете примерно примерно шаблон кода написать? Что то у меня не очень получается это реализовать :S

[Вячеслав Успенский]

Можно, а напишите пожалуйста смысл вот этого:

case when ct.dog_date>='01.01.2013' then cl.iin
           else nvl(cl.iin,cl.rnn) end rnn

Я ведь правильно понимаю? Вы просто выводите список клиентов с которыми заключен договор? Зачем тут это условие?

[Light 777]

Вячеслав Успенский: если номер договора больше чем эта дата то в поле rnn вводится значение cl.iin, else если значение cl.iin пусто то вводится значение cl.rnn в поле rnn

[Вячеслав Успенский]

Light 777: это я вижу) к тому же не номер договора а дата, я спросил не как а зачем)
И еще разок: вы просто выводите данные клиентов с которыми заключен договор?
По идее:

select ct.client,
    upper(trim(cl.client_name)) as cli_name,
    case when ct.mdog_date>='01.01.2013' then cl.iin
           else nvl(cl.iin,cl.rnn) end rnn
  from (select client,max(dog_date) as mdog_date from contracts group by client) as ct
  left join client as cl
    on ct.client=cl.client

Это выборка всех клиентов, с которыми заключен договор, при этом имеем последнюю дату заключения договора. Если вам нужно нечто другое то нужно все же не абстрактно описывать)

[Вячеслав Успенский]

Light 777: и не забудьте повесить индекс как минимум на поле client в таблице договоров

[Вячеслав Успенский]

Light 777: хотя у вас договор строкой - крайне плохая идея, лучше date, тогда если нужны будут периоды - потмо не возникнет проблем ну и сортировка будет нормальная

[Вячеслав Успенский]

Light 777: ой блин, этож оракл, а я для MySQL написла, на тег глянул. Только сейчас заметил что вы ссылку на запрос кидали. В общем коли вам требе помосч - обозначьте цель

Answer 3

[Георгий Пичугин]

так?
SELECT nvl(decode(ct.dog_date, date '2013-01-01', cl.iin, null), cl.rnn) rnn,
TO_DATE(CT.DOG_DATE, 'dd.mm.rrrr') DOG_DATE,
UPPER(TRIM(CL.CLIENT_NAME)) CLI_NAME,
count(*)
FROM CONTRACTS CT, CLIENT CL
WHERE CT.CLIENT = CL.CLIENT
AND CL.CLIENT_CATEGORY = 1
and rownum < 500
group by nvl(decode(ct.dog_date, date '2013-01-01', cl.iin, null), cl.rnn),
TO_DATE(CT.DOG_DATE, 'dd.mm.rrrr'),
UPPER(TRIM(CL.CLIENT_NAME));