Хранимая процедура T-SQL с включенной сортировкой и подкачкой не работает должным образом

Question

Привет, ребята, я использую следующий код

ALTER PROCEDURE [dbo].[usp_get_all_groups] 
    -- Add the parameters for the stored procedure here
    @pStartIndex smallint,
    @pPageSize tinyint,
    @pOrderBy varchar
AS
BEGIN

 SELECT 
       *
       FROM
        (SELECT ROW_NUMBER() OVER (ORDER BY 

        CASE WHEN @pOrderBy='GroupId ASC' THEN UserGroups._id + ' ASC'
             WHEN @pOrderBy='GroupId DESC' THEN UserGroups._id + ' DESC'
             WHEN @pOrderBy='GroupCode ASC' THEN UserGroups.GroupCode + ' ASC'
             WHEN @pOrderBy='GroupCode DESC' THEN UserGroups.GroupCode + ' DESC'
        END        
        ) AS Row, * FROM UserGroups)
       AS StudentsWithRowNumbers
         WHERE Row>= @pStartIndex AND Row <= @pStartIndex + @pPageSize
END

Когда я выполняю сохраненный процесс, используя следующую команду

DECLARE @return_value int

EXEC    @return_value = [dbo].[usp_get_all_groups]
        @pStartIndex = 0,
        @pPageSize = 15,
        @pOrderBy = N'GroupCode ASC'

SELECT  'Return Value' = @return_value

Я получаю эти результаты, которые не отсортированы.

Row _id GroupCode   Description Type    IsActive
1   1   CS2009  CS 2009 Batch   S   1
2   2   IT2009  IT 2009 Batch   S   1
3   3   ME2009  ME 2009 Batch   S   1
4   4   EC2009  EC 2009 Batch   S   1
5   5   EE2009  EE 2009 Batch   S   1
6   8   CS_F    CS Faculties    F   1
7   9   IT_F    IT Faculties    F   1
8   10  ME_F    ME Faculties    F   1
9   11  EC_F    EC Faculties    F   1
10  12  EE_F    EE Faculties    F   1
11  13  BSC_F   Basic Science Faculties F   1
12  14  Accounts    Accounts    A   1
13  15  Mgmt    Management  M   1
14  16  Lib Library B   1
15  17  TnP Training & Placement    T   1

Можете ли вы сказать мне, что еще требуется?

---

Я пробовал это, но это также дает самолету несортированный результат

SELECT 
        GroupTable._id,
        GroupTable.GroupCode,
        GroupTable.Type,
        GroupTable.Description
       FROM
        (SELECT ROW_NUMBER() OVER (ORDER BY 

        CASE WHEN @pOrderBy='GroupId ASC' THEN CONVERT(varchar(20), '_id ASC') 
             WHEN @pOrderBy='GroupId DESC' THEN CONVERT(varchar(20), '_id DESC') 
             WHEN @pOrderBy='GroupCode ASC' THEN CONVERT(varchar(20), @pOrderBy) 
             WHEN @pOrderBy='GroupCode DESC' THEN CONVERT(varchar(20), @pOrderBy) 
        END        
        ) AS Row, * FROM UserGroups)
       AS GroupTable
         WHERE Row>= @pStartIndex AND Row <= @pStartIndex + @pPageSize

       Select COUNT(*) as TotalRows from UserGroups where IsActive= 1

Answer 1

Замените вашу процедуру на это:

ALTER PROCEDURE [dbo].[usp_get_all_groups] 
    -- Add the parameters for the stored procedure here
    @pStartIndex smallint,
    @pPageSize tinyint,
    @pOrderBy varchar(15)
AS
BEGIN

 SELECT *
 FROM
  (SELECT ROW_NUMBER() OVER (ORDER BY 
      CASE WHEN @pOrderBy='GroupId ASC' THEN UserGroups._id END ASC,  
      CASE WHEN @pOrderBy='GroupId DESC' THEN UserGroups._id END DESC,             
      CASE WHEN @pOrderBy='GroupCode ASC' THEN UserGroups.GroupCode END ASC,
      CASE WHEN @pOrderBy='GroupCode DESC' THEN UserGroups.GroupCode END DESC) AS Row, 
      * FROM UserGroups) AS StudentsWithRowNumbers
  WHERE Row>= @pStartIndex AND Row <= @pStartIndex + @pPageSize
  ORDER BY Row      
END

Вы не можете динамически назначать asc и desc для нединамического выражения.

Answer 2

Кажется, существует неправильное представление о том, что выражение

ORDER BY UserGroups._id + ' DESC'

заставит SQL Server применить нисходящий порядок. На самом деле он просто добавляет строковый литерал " DESC" к значению столбца, а затем сортирует результат в порядке возрастания.

Вам необходимо динамически создать весь оператор SELECT в процедуре, применив ORDER BY дважды, как показано в ответе marc_s, и выполнить оператор, используя sp_executesql. sp_executesql также позволяет передавать параметры @.

Answer 3

Вам не хватает

@pOrderBy varchar(20) because of this your @pOrderBy has only one char 'G'

также проверьте, что эта ссылка может помочь вам

Answer 4

Может быть, это поможет (если _id и GroupCode одного типа):

DECLARE @pOrderBy VARCHAR(100),
        @pStartIndex smallint,
        @pPageSize tinyint
SET @pOrderBy='GroupId DESC'
SET @pStartIndex=0
SET @pPageSize=15
SELECT 
    GroupTable._id,
    GroupTable.GroupCode,
    GroupTable.Type,
    GroupTable.Description
FROM
(SELECT ROW_NUMBER() OVER (ORDER BY 
    CASE @pOrderBy
        WHEN 'GroupId ASC'
        THEN UserGroups._id
        WHEN 'GroupCode ASC'
        THEN UserGroups.GroupCode
    END ASC,
    CASE @pOrderBy
        WHEN 'GroupId DESC'
        THEN UserGroups._id
        WHEN 'GroupCode DESC'
        THEN UserGroups.GroupCode
    END DESC    
    ) AS Row, * FROM UserGroups)
AS GroupTable
WHERE 
    Row>= @pStartIndex 
    AND Row <= @pStartIndex + @pPageSize

Answer 5

Вы не заказываете свой SELECT утверждение.... ни внутреннее SELECT от UserGroups имеет ORDER BYи внешний SELECT.... вам нужно предоставить ORDER BY на SELECT, тоже! (не только в вашем ROW_NUMBER() функция-х OVER() оговорка)

SELECT 
    *
FROM
   (SELECT 
       ROW_NUMBER() OVER (ORDER BY 
        CASE WHEN @pOrderBy='GroupId ASC' THEN UserGroups._id + ' ASC'
             WHEN @pOrderBy='GroupId DESC' THEN UserGroups._id + ' DESC'
             WHEN @pOrderBy='GroupCode ASC' THEN UserGroups.GroupCode + ' ASC'
             WHEN @pOrderBy='GroupCode DESC' THEN UserGroups.GroupCode + ' DESC'
        END) AS Row, * 
    FROM UserGroups) AS StudentsWithRowNumbers
WHERE 
     Row >= @pStartIndex AND Row <= @pStartIndex + @pPageSize
ORDER BY
    CASE 
       WHEN @pOrderBy='GroupId ASC' THEN UserGroups._id + ' ASC'
       WHEN @pOrderBy='GroupId DESC' THEN UserGroups._id + ' DESC'
       WHEN @pOrderBy='GroupCode ASC' THEN UserGroups.GroupCode + ' ASC'
       WHEN @pOrderBy='GroupCode DESC' THEN UserGroups.GroupCode + ' DESC'
    END     

ORDER BY внутри OVER() предложение используется только для вычисления Row значения - это не порядок результирующего набора данных.