Очевидно, что хранимая процедура SQL?

Question

У меня есть хранимая процедура, как это:

IF @Active=0 
    select @rowCount=(select count(1) from [table] WHERE (GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active 

IF @Active=1 
    select @rowCount=(select count(1) from [table] WHERE NOT(GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active 

Есть ли способ, чтобы сделать заявление более ясно?

Я делаю пагинация так:

Параметры:

@pageNum AS BIGINT, 
@pageSize AS BIGINT, 
@Active AS INT, 
@RowCount AS INT OUTPUT 

И отдых:

IF @Active=0 
BEGIN 
    WITH K AS 
    ( - SELECT [Id] 
     FROM [table] 
     WHERE not(GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active ORDER BY [Id] 
     OFFSET (@pagenum - 1) * @pagesize ROWS FETCH NEXT @pagesize ROWS ONLY 
    ) 
    SELECT T.[Id],T.[Name] 
    FROM [table] 
    AS T 
     INNER JOIN K 
     ON T.Id = K.Id 

    ORDER BY T.Id; 

    select @rowCount=(select count(1) from [table] WHERE not(GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active 
END 
IF @Active=1 
BEGIN 
    WITH K AS 
    ( 
     SELECT [Id] 
     FROM [table] 
     WHERE (GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active 
     ORDER BY [Id] 
     OFFSET (@pagenum - 1) * @pagesize ROWS FETCH NEXT @pagesize ROWS ONLY 
    ) 
    SELECT T.[Id],T.[Name] 
    FROM [table] 
    AS F 
     INNER JOIN K 
     ON T.Id = K.Id 

    ORDER BY T.Id; 

    select @rowCount=(select count(1) from [table] WHERE (GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active END 

и хранимая процедура выглядит очень долго ..... так Мне интересно, есть ли лучший способ сделать это лучше

Answer 1

Это может быть яснее и включает в себя только один SELECT:

select @rowcount = case @active 
    when 0 then sum(case when GETDATE() BETWEEN StartTime AND EndTime then 1 else 0 end) 
    when 1 then sum(case when GETDATE() BETWEEN StartTime AND EndTime then 0 else 1 end) end 
from [table] 
where Active=@Active 

Или это предложение от @Damien_The_Unbeliever даже pithier (хотя, возможно, менее понятно!):

select @rowcount = sum(case when GETDATE() BETWEEN StartTime AND EndTime then 1-@Active else 0+@active end) 
from [table] 
where Active=@Active 

Answer 2

IF @Active=0 select @rowCount=(SELECT CASE 
    WHEN @Active=0 
     THEN select count(1) from [table] WHERE (GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active 
    WHEN @Active=1 
     THEN select count(1) from [table] WHERE NOT(GETDATE() BETWEEN StartTime AND EndTime) AND Active=@Active 

вы можете сделать свой вопрос более понятно?

Answer 3

Не уверены, если это понятнее, но вот еще один метод, который является немного менее многословным:

SELECT @rowCount = COUNT(1) 
FROM [table] 
WHERE 
    ((GETDATE() BETWEEN StartTime AND EndTime) AND Active = @Active) 
    OR (NOT (GETDATE() BETWEEN StartTime AND EndTime) AND Active = @Active);