web-dev-qa-db-ja.com

パラメータをどのように並べ替えますか？

実行しているストアドプロシージャに関するフィードバックを求めることができるかどうか、およびシナリオを処理するためのより効率的な方法があるかどうか疑問に思っています（きっとそうなると思います！）。

基本的に私は単一のSPを呼び出して、1つ以上のステータスとソート順を持つ可能性のあるレコード（ジョブ）のリストを返します（ページングにRowNumを使用しています）。ステータスのバリエーションは常に変化する可能性があるため（ユーザーなどによって）、WITH RECOMPILEを使用している瞬間、何らかのフィルタリングも行われています。

IFステートメントを使用して、基本的に同じコードを実行していますが、変更点は並べ替え順のみです。

私の質問は次のとおりだと思います：これを行うためのより良い方法はありますか？知識がないために複雑すぎますか（かなり可能性があります）SPは実際に問題ありませんが、行数を減らすために微調整が必​​要ですか？）

SPの一部を下に貼り付けました-完全なコードとの唯一の違いは、異なる並べ替え順序の追加のIFステートメントです...

フィードバックをいただければ幸いです。

前もって感謝します！

PROCEDURE [dbo].[sp_Jobs] 

@PageNumber int, 
@PageSize int, 
@FilterExpression varchar(500), 
@OrderBy varchar(50), 
@CustomerID int, 
@ShowNotSet bit, 
@ShowPlaced bit, 
@ShowProofed bit, 
@ShowReProofed bit, 
@ShowApproved bit, 
@ShowOnTime bit, 
@ShowLate bit, 
@ShowProblem bit, 
@ShowCompleted bit, 
@ShowDispatched bit, 
@ShowUnapproved bit, 
@ShowClosed bit, 
@ShowReturned bit, 
@UserID int

WITH RECOMPILE 

AS

--JobNumber DESC 
if @OrderBy='JobNumberDESC' 
BEGIN 

WITH Keys AS (SELECT TOP (@PageNumber * @PageSize) ROW_NUMBER() OVER (ORDER BY JobNumber DESC) as rn,P1.jobNumber,P1.CustID,P1.DateIn,P1.DateDue,P1.DateOut,p1.client,p1.MasterJobStatusID,p1.MasterJobStatusTimestamp,p1.OwnerID 

FROM 
vw_Jobs_List P1 WITH (NOLOCK) 

WHERE 
(@CustomerID = 0 OR CustID = @CustomerID) 
AND (@UserID = 0 OR OwnerID = @UserID) 
AND ((@ShowNotSet = 1 AND MasterJobStatusID=1) OR (@ShowPlaced = 1 AND MasterJobStatusID=2) OR (@ShowProofed = 1 AND MasterJobStatusID=3) OR (@ShowReProofed = 1 AND MasterJobStatusID=4) OR (@ShowApproved = 1 AND MasterJobStatusID=5) OR (@ShowOnTime = 1 AND MasterJobStatusID=6) OR (@ShowLate = 1 AND MasterJobStatusID=7) OR (@ShowProblem = 1 AND MasterJobStatusID=8) OR (@ShowCompleted = 1 AND MasterJobStatusID=9) OR (@ShowDispatched = 1 AND MasterJobStatusID=10) OR (@ShowUnapproved = 1 AND MasterJobStatusID=11) OR (@ShowClosed = 1 AND MasterJobStatusID=12) OR (@ShowReturned = 1 AND MasterJobStatusID=13)) AND (Search LIKE '%'+@FilterExpression+'%')

ORDER BY 
P1.JobNumber DESC ),SelectedKeys AS (
SELECT TOP (@PageSize)SK.rn,SK.JobNumber,SK.CustID,SK.DateIn,SK.DateDue,SK.DateOut 

FROM 
Keys SK 

WHERE 
SK.rn > ((@PageNumber-1) * @PageSize) 

ORDER BY 
SK.JobNumber DESC) 

SELECT SK.rn,J.JobNumber,J.OwnerID,J.Description,J.Client,SK.CustID,OrderNumber, CAST(DateAdd(d, -2, CAST(isnull(SK.DateIn,0) AS DateTime)) AS nvarchar) AS DateIn, CAST(DateAdd(d, -2, CAST(isnull(SK.DateDue,0) AS DateTime)) AS nvarchar) AS DateDue,CAST(DateAdd(d, -2, CAST(isnull(SK.DateOut,0) AS DateTime)) AS nvarchar) AS DateOut, Del_Method,Ticket#, InvoiceEmailed, InvoicePrinted, InvoiceExported, InvoiceComplete, JobStatus,j.MasterJobStatusID,j.MasterJobStatusTimestamp,js.MasterJobStatus 

FROM SelectedKeys SK JOIN vw_Jobs_List J WITH (NOLOCK) ON j.JobNumber=SK.JobNumber JOIN tbl_SYSTEM_MasterJobStatus js WITH (NOLOCK) ON j.MasterJobStatusID=js.MasterJobStatusID 

ORDER BY 
SK.JobNumber DESC 
END

-他の列の並べ替えの場合はELSE IF

sql-server-2008stored-procedures
16
VaticNZ

並べ替えは、CASE式で処理できます。

ORDER BY
    CASE WHEN @SortDirection = 'A' THEN
        CASE 
           WHEN @SortBy = 'JobNumber' THEN JobNumber
           WHEN @SortBy = 'JobId' THEN JobId 
        END
    END ASC
    , CASE WHEN @SortDirection = 'D' THEN
        CASE 
           WHEN @SortBy = 'JobNumber' THEN JobNumber
           WHEN @SortBy = 'JobId' THEN JobId 
        END
    END DESC

ORを使用したwhere条件は、不適切な計画を生成する可能性があるため、再検討することをお勧めします。これ（および代替アプローチ）について説明した最高の記事の1つは、 T-SQLの動的検索条件 です。

編集：パラメータリストをもう一度見ると、プライマリフィルタは@CustomerIdと@UserIdのように見えます。 2つのプロシージャspJobs_SelectByCustomerIdとspJobs_SelectByUserIdを作成することをお勧めします。これらはそれぞれのパラメーターでフィルター処理されるため、 '@ Param = 0またはColumn = @Param'条件を排除できます。次の重要なパラメーターは@ShowCompleted（ジョブが「完了」すると、@ ShowCompleted = 1でない限り表示されないと想定）だと思います。これは、CustomerIdおよびUserIdのインデックスに含めることを検討します。

Edit2：これらの質問が時々あなたの心の奥に沈む方法がおかしい！ :) @ShowCompletedのインデックス作成では、これは 最初に選択性の低いBIT列を使用することが最良の戦略である可能性がある状況の1つです 。 フィルターされたインデックス も考慮する必要があります。

16
Mark Storey-Smith