как объединить строки многострочного подзапроса с условным оператором, который будет использоваться в In в T-sql - PullRequest
0 голосов
/ 08 ноября 2018

Я пытаюсь получить записи родительского отдела, если таковые существуют, из представления PayrollView, а также дочерние отделы предоставили DepartmentSecurityMode=2. Строки фильтруются по родительскому идентификатору отдела. Это то, что я пытался, но это показывает мне ошибку

"Подзапрос возвратил более 1 значения. Это недопустимо, если подзапрос следует =,! =, <, <=,>,> = Или когда подзапрос используется в качестве выражения."

DECLARE @LoggedInUserId   UNIQUEIDENTIFIER
                         = 'ff333441-bc4f-4f3d-b992-aad17baa9ebc'
DECLARE @LoggedInRoleId   UNIQUEIDENTIFIER
                         = '2cb31771-7cbc-4f38-b558-804308d908c7'
DECLARE @DepartmentId   UNIQUEIDENTIFIER
                       = '989fd98a-33ae-4ce1-8bbd-ed51ac46137d'
DECLARE @PeriodId   UNIQUEIDENTIFIER = 'e7e52d21-90ad-4f2d-adc2-3f5399a032dd'

SELECT *
FROM PayrollView pv
INNER JOIN Departments d
ON pv.DepartmentId = d.SysId
CROSS JOIN AppRoles r
LEFT JOIN UserDepartmentView rd
ON d.SysId = rd.DepartmentId
AND rd.RoleId = @LoggedInRoleId
AND rd.UserId = @LoggedInUserId
WHERE PeriodId = @PeriodId
AND r.SysId = @LoggedInRoleId
AND 
(
    r.DepartmentSecurityMode = 2
    AND D.SysId IN
    ( 
        (
            SELECT SysId
            FROM Departments ds
            WHERE ds.ParentHierarchy LIKE
             '%' + cast (@DepartmentId AS VARCHAR (50)) + '%'
        )
        ,
        --This is where I want to combine both the child as well the parent.
        (
            CASE
                WHEN @DepartmentId = '00000000-0000-0000-0000-000000000000' THEN D.SysId
                ELSE @DepartmentId
            END
        )
    )
)

Любые предложения приветствуются.

Ответы [ 2 ]

0 голосов
/ 08 ноября 2018

Я не совсем понимаю ваше требование, но, как указывает @ Johey в своем ответе, в вашем заявлении IN есть проблема. Попробуйте заменить логику для D.SysId in этим кодом / дайте нам знать, если это делает то, на что вы надеялись ...

AND D.SysId IN
( 
    SELECT SysId
    FROM Departments ds
    WHERE ds.ParentHierarchy LIKE '%' + cast (@DepartmentId AS VARCHAR (50)) + '%'

    UNION

    SELECT CASE
        WHEN @DepartmentId = '00000000-0000-0000-0000-000000000000' THEN D.SysId
        ELSE @DepartmentId
    END
)
0 голосов
/ 08 ноября 2018

Вы пытаетесь сделать что-то странное: с условием <expression> IN (...) вы можете фильтровать записи, которые существуют в списке элементов. Этот список может быть значениями, разделенными запятыми ИЛИ подзапросом. Вы комбинируете оба, что (по моим сведениям) невозможно.

Итак, вы должны разделить эти две техники ...

Обновление:

@ Ответ JohnLBevan - вариант, но я думаю, что вы могли бы упростить логику, сделав так:

AND (
      -- don't filter if no department provided 
      @DepartmentId = '00000000-0000-0000-0000-000000000000'

      OR 

      -- filter on the given department
      D.SysId IN ( 
                    SELECT SysId
                    FROM Departments ds
                    WHERE ds.ParentHierarchy LIKE '%' + cast (@DepartmentId AS 
VARCHAR (50)) + '%'
                  )
)
...