[sql] sql vereenvoudigen

[Tinuske]

Onderstaande functie gebruik ik voor paging

code:

parameters toevoegen:
   @statusid int,    
   @Pagenumber int,
   @PageSize int,
   @SortBy varchar(100) 

)
as
begin
   set nocount on
   declare @TotalRows int 
   declare @FirstSelectingRow int 
   declare @LastSelectingRow int

   select @TotalRows = 
(count(bedrijfid) from bedrijven where bedrijfstatus = @statusid )

   select @FirstSelectingRow = (@Pagenumber - 1) * @PageSize + 1

   if (@FirstSelectingRow <= @TotalRows)
   begin
      set rowcount @FirstSelectingRow
      select @FirstSelectingRow = (select * from bedrijven
      where bedrijfstatus = @statusid
      Order By
    CASE WHEN @SortBy='Achternaam' THEN CONVERT(char(50), Achternaam) + CONVERT(char(50), Voornaam)
       WHEN @SortBy='Salaris' THEN CONVERT(varchar(50), Salaris)
       WHEN @SortBy='ID' THEN ID      

      set rowcount @PageNumber
     select * from bedrijven where bedrijfstatus = @statusid) 
      where primarykey >= @FirstSelectingRow  
      Order By
CASE WHEN @SortBy='Achternaam' THEN CONVERT(char(50), Achternaam) + CONVERT(char(50), Voornaam)
       WHEN @SortBy='Salaris' THEN CONVERT(varchar(50), Salaris)
       WHEN @SortBy='ID' THEN ID       
   end
   set nocount off
end

Is er een manier om deze te vereenvoudigen, want zoals je ziet zitten er hierin 3 op gelkaar gelijkende sql's. Aangezien sommige van deze sql's iets van 50 pagina's zijn, wordt het al snel onoverzichtelijk . Hoe is dit overzichtelijker te maken ?

[whoami]

Kan je niets doen met een TOP clausule icm een ORDER BY en een WHERE [sortveld] > laatste_waarde_vorige_page

[Tinuske]

Je krijgt dan zoiets -> (Alsnog 3 x ongeveer dezelfde SQL dus)
en dit is bovendien ook nog trager :

code:

DECLARE @sql NVARCHAR(1024)  
 
    SET @sql = N'SELECT 
            ArtistName, 
            Title 
        FROM  
        (  
            SELECT TOP '+RTRIM(@perpage) 
                +' ArtistName, 
                Title 
            FROM  
            ( 
                SELECT TOP '+RTRIM(@perpage*@pagenum) 
                    +' ArtistName, 
                    Title 
                FROM 
                    SampleCDs WITH (NOLOCK) 
                ORDER BY 
                    ArtistName+''~''+Title 
            ) a  
            ORDER BY 
                ArtistName+''~''+Title DESC 
        ) b  
        ORDER BY 
            ArtistName+''~''+Title' 
 
    EXEC sp_executeSQL @sql

[d00d]

De volgende pseudocode kan misschien gaan werken?

code:

create table #temp (
    id int identity,
    col1 datatype1,
    col2 datatype2,
    col n datatype n)

insert into #temp (columns)
    select top (@pageNumber * @pageSize)
    from table
    order by @sortby

select <columns>
from #temp
where id between
    ( (@pagenumber * @pagesize) - @pagesize 
    AND (@pagenumber * @pagesize) )
order by id