Как оптимизировать код sql?

Question

[chincharovpc]

Есть две сущности users(id,name,department_id) и departments(id,name,is_private)
Мне нужно выбрать список пользователей для пользователя с id=8
1. Если у пользователя с id=8 департамент приватный(is_private=true), то выбрать пользователей только с этим департаментом
2. Если у пользователя с id=8 нет департамента или департамент открытый(is_private=false), то выбрать пользователей, у которых нет департамента или департамент открытый

Код показан ниже

SELECT users.* 
FROM users
         LEFT JOIN "departments" "Department" ON "users"."department_id" = "Department"."id"
WHERE CASE
            WHEN (SELECT COUNT(users.*)
                  FROM users
                           LEFT JOIN departments on users.department_id = departments.id
                  WHERE users.id = 8 AND (users.department_id IS NULL OR departments.is_private = false)) > 0
                THEN users.department_id IS NULL OR "Department".id IN (SELECT departments.id FROM departments WHERE departments.is_private = false)
            ELSE "Department".id = (SELECT users.department_id FROM users WHERE users.id = 8)
    END

Как мне улучшить код?

Comments

[Akina]

Как мне улучшить код?

Не маяться фигнёй с CASE, а сделать UNION ALL двух отдельных запросов.

[chincharovpc]

Akina, Подскажешь как написать для этого запроса?
Плюс у меня подтягиваются еще 4 таблицы и несколько проверок в where, мне нужно дублировать их в union all?

[Akina]

как написать для этого запроса?

Что значит как? Напиши запрос Если у пользователя с id=8 департамент приватный(is_private=true), то выбрать пользователей только с этим департаментом. Затем напиши запрос Если у пользователя с id=8 нет департамента или департамент открытый(is_private=false), то выбрать пользователей, у которых нет департамента или департамент открытый. Потом объедини их с помощью UNION ALL.

Да. Для проверки вовсе не надо считать COUNT(), для этого существует WHERE [NOT] EXISTS. Который к тому же работает быстрее. COUNT() должен выбрать и посчитать все записи, тогда как WHERE [NOT] EXISTS, найдя первую соответствующую условиям запись в зависимой таблице, успокоится и перейдёт к следующей записи основной таблицы.

Плюс у меня подтягиваются еще 4 таблицы и несколько проверок в where, мне нужно дублировать их в union all?

Да наплевать. Всё равно UNION ALL будет быстрее твоего многоэтажного безобразия.
И, обрати внимание, один из двух подзапросов при любых условиях вернёт пустой набор записей. Так что UNION ALL по сути - простая формальность.

[Игорь]

Akina, а почему такие сложности? Чем наполняемость условий не подходит?

Пример запроса

declare @aaa int =  8

SELECT
	*
FROM Users AS usr
	LEFT JOIN Departments dep
		ON usr.Department_ID = dep.ID 
WHERE
		(usr.user_id = @aaa AND dep.Is_Private = 1)
	OR
		(usr.user_id <> @aaa AND (dep.Is_Private <> 1 OR dep.Is_Private IS NULL))

[chincharovpc]

Игорь Surname, Запрос возвращает всех пользователей. Мне нужно, если у user=8 приватный департамент, то возвращать всех из его департамента, иначе всех пользователей, у которых отсутствует департамент или открытый.
Я пока реализовал так, буду дальше разбираться

SELECT users.*
FROM users
         LEFT JOIN "departments" "Department" ON "users"."department_id" = "Department"."id"
         LEFT JOIN users "User2" ON "User2".id = 8
         LEFT JOIN departments "Department2" ON "User2".department_id = "Department2".id
WHERE CASE
          WHEN "Department2" IS NULL OR "Department2".is_private = false
              THEN "Department" IS NULL OR "Department".is_private = false
          ELSE "Department".id = "Department2".id
          END

[Akina]

SELECT *
FROM users
WHERE is_private = TRUE
  AND EXISTS (
    SELECT NULL
    FROM users
    WHERE id = 8
      AND is_private = TRUE
    )
UNION ALL
SELECT *
FROM users
WHERE is_private <> TRUE
  AND NOT EXISTS (
    SELECT NULL
    FROM users
    WHERE id = 8
      AND is_private = TRUE
    )