0

Совсем недавно на этом же сайте мне показали как сделать запрос на выбор данных из 4-х таблиц, взяв это за пример я сделал другой запрос:

        SELECT L.*,
        (SELECT group_concat(distinct client.client_full_name) 
        FROM client 
        JOIN cases ON (client.client_id=cases.client_id) 
        JOIN lawyer ON (lawyer.lawyer_id=cases.lawyer_id) 
        GROUP BY lawyer.lawyer_full_name) clientname,
        (SELECT count(cases.lawyer_id) 
        FROM cases 
        JOIN lawyer ON (lawyer.lawyer_id=cases.lawyer_id) 
        WHERE cases.case_archive=1
        GROUP BY lawyer.lawyer_id) archive
        FROM lawyer L

Выдает мне ошибку, я не понимаю почему и как ее исправить...

Задача запроса получить строки на каждого адвоката (таблица lawyer) все (*) его данные + список его клиентов сейчас (достать из таблицы cases) и сума его архивных дел (тоже из cases, только тут cases.case_archive=1).

Предоставляю так же онлайн модель базы для удобной проверки запросов: http://sqlfiddle.com/#!9/9130c/

И последнее: прошу описать каждый шаг запроса, я очень путаюсь в запросах и вот засоряю форум почти одинаковыми темами...

15
  • в group_concat separator не нужен?
    – Saidolim
    13 дек 2015 в 10:43
  • Выглядит жутковато :) запрос очень странно заканчивается на FROM lawyer L, client CL без условий по которым эти таблицы клеятся WHERE. В итоге перемножаются все записи адвокатов на всех клиентов. сильно сомневаюсь что так задумано
    – Mike
    13 дек 2015 в 10:44
  • И запятая после ) archive явно не в кассу
    – Mike
    13 дек 2015 в 10:47
  • И что такое "сумма архивных дел". сейчас вы делаете sum по id адвокатов в делах. сомневаюсь, что сумма ID кому то интересна
    – Mike
    13 дек 2015 в 10:54
  • @Mike Да, я ту запятую на пробнике заметил после публикации поста и удалил. Условия склеивания вы имеете ввиду USING или GROUP BY?
    – Telion
    13 дек 2015 в 10:54

1 ответ 1

1

Вообще подзапросы в списке выборки - это крайний случай. Да, он удобен. Но обычно он утяжеляет запрос. Его надо использовать тогда, когда обычных средств не хаватает. Например, если основной запрос, произведя склейку нескольких таблиц создает такое кол-во записей, что по ним уже нельзя получить сумму какого то поля, т.к. одно и то же значение этого поля встречается несколько раз.

Примерный план составления такого запроса:

  1. Определяем участвующие таблицы. Во главе у нас lawyer т.к. именно из нее надо получить все записи, вне зависимости от наличия дел и клиентов. Для получения количества дел надо добавить таблицу cases и для получения клиентов таблицу client, причем она клеится через дела.

  2. Составляем базовый запрос:

    SELECT *
      FROM lawyer L
      LEFT JOIN cases  S ON (L.lawyer_id=S.lawyer_id)
      LEFT JOIN client C ON (S.client_id=C.client_id)
    

    Мы используем только LEFT JOIN - таким образом адвокаты будут точно все. Наличие записей в остальных таблицах не требуется.

    Проверяем запрос, смотрим все ли записи, которые нам нужны для последующего суммирования у нас выдались. Да, все. Причем есть имена всех клиентов и есть все дела по адвокату. Хотя нас интересует только кол-во архивных дел, а у нас в запросе все дела (если всех дел не будет, мы получим не всех клиентов). Значит нам надо как-то хитрее посчитать кол-во дел. Применим IF.

  3. Добавляем группировку и групповые функции

    SELECT L.*,
           group_concat(distinct C.client_full_name) clients,
           sum(IF(S.case_archive=1,1,0)) case_arx_sum,
           count(S.case_id) case_all_sum
      FROM lawyer L
      LEFT JOIN cases  S ON (L.lawyer_id=S.lawyer_id) 
      LEFT JOIN client C ON (S.client_id=C.client_id)
     GROUP BY L.lawyer_id
    

    По поводу групповой функции с IF. обычный count(S.case_id) считает все NOT NULL значения и таким образом посчитает кол-во дел и не затронет записи, в которых только адвокат без единого дела. А мы хотим получить все архивные дела, делаем SUM по условию, которое должно выдать для суммирования 1 для архивных дел и 0 во всех остальных случаях, включая отсутствие дел

Предположим, нам надо было бы в этом запросе показать заодно суммарное наказание клиентов данного адвоката. Если мы приклеим таблицу punishment в этот же запрос то некоторые дела буду попадаться несколько раз, т.к. записи просто размножатся, в случаях когда есть несколько наказаний у клиента. Если есть 2 дела и 2 наказания - то мы получим 4 записи, со всеми вариантами дело/наказание. В такой ситуации посчитать кол-во чего либо будет крайне сложно. Поэтому тут нам поможет как раз подзапрос в списке выборки.

Подзапрос должен возвращать одно значение для каждого адвоката. Пишем его сначала как отдельный запрос, который получит нужное значение для ID одного конкретного адвоката. нам нужен только ID адвоката, поэтому таблицу самих адвокатов использовать не обязательно, до нужной нам ID мы можем добраться через таблицу дел (cases). Получаем такой запрос для адвоката ID=1:

SELECT sum(P.punishment_value)
  FROM punishment P, cases C1
 WHERE C1.client_id=P.client_id and C1.lawyer_id=1

осталось только вставить его в основной запрос, использовав в качестве id адвоката соответствующий id из основного запроса (C1.lawyer_id=L.lawyer_id):

SELECT L.*,
           group_concat(distinct C.client_full_name) clients,
           sum(IF(S.case_archive=1,1,0)) case_arx_sum,
           count(S.case_id) case_all_sum,
           (SELECT sum(P.punishment_value)
              FROM punishment P, cases C1
             WHERE C1.client_id=P.client_id and C1.lawyer_id=L.lawyer_id) punish_sum
      FROM lawyer L
      LEFT JOIN cases  S ON (L.lawyer_id=S.lawyer_id) 
      LEFT JOIN client C ON (S.client_id=C.client_id)
     GROUP BY L.lawyer_id
7
  • Идеально. Впрочем как всегда, но у меня есть вопросы. На счет IF: в моей таблице дел cases значения NOT NULL если строка вообще будет. Я имею ввиду что если есть адвокат в ЭТОЙ таблице то будет и дело ибо эта таблица их связывает. Вопрос чисто из интереса, выходит в моем случае можно указывать count без IF?
    – Telion
    13 дек 2015 в 12:33
  • Стоп, я понял, что ничего не понял. Чем отличаются эти 2 строки: sum(IF(S.case_archive=1,1,0)) case_arx_sum,count(S.case_id) case_all_sum ?
    – Telion
    13 дек 2015 в 12:45
  • @Levelleor Я в запросе специально привел case_all_sum - это как раз количество всех дел адвоката и считается оно как раз count() причем по полю case_id, потому как count не считает кол-во NULL значений и не посчитает адвокатов без дел. А case_archive я и не вывожу. я его использую внутри IF. По факту IF выдает 1 которую sum добавляет к сумме. И IF это делает только если совпало его условие т.е. для case_archive=1. NULL это не 0 и не 1 так что условие IF на него никогда не среагирует
    – Mike
    13 дек 2015 в 12:47
  • @Levelleor У меня в запросе в where или on признак архивного дела не проверяется и запрос работает со всеми записями и с архивными и с рабочими. Иначе бы он не выдал гарантированно всех клиентов для адвоката
    – Mike
    13 дек 2015 в 12:49
  • А почему запрос не сработает верно, указав в ON дополнительное условие case_archive=1? Не выведет весь список дел? Я о нем изначально и не думал.
    – Telion
    13 дек 2015 в 12:53

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.