optimisation SQL SP

Question

Je veux optimiser ce SP, tout le monde a une idée de comment je peux faire cela? Merci d'avance.

SET ANSI_NULLS ON 
GO 

SET QUOTED_IDENTIFIER ON 
GO 

ALTER PROCEDURE [dbo].[Members] ( 
    @StartTime datetime = null 
    , @EndTime datetime = null 
    , @CustomerEmail nvarchar(255) = null 
    , @CustomerName nvarchar(255) = null 
    , @ShippingMethod nvarchar(255) = null 
    , @MemberOrderStatusPending int = null 
    , @MemberOrderProcessing int = null 
    , @MemberOrderComplete int = null 
    , @MemberOrderStatusCancelled int = null 
    , @MemberOrderStatusCancelledDiscontinued int = null 
    , @MemberOrderStatusCancelledCustomerRequest int = null 
    , @MemberOrderStatusCancelledPendingNeverPaid int = null 
) 
AS 
BEGIN 

    SET NOCOUNT ON 

    SELECT DISTINCT o.OrderID 
      , o.OrderTotal 
      , o.BillingFirstName + ' ' + o.BillingLastName AS CustomerName 
      , o.CreatedOn AS CreatedOn 
    FROM Order o 
    WHERE (o.CreatedOn > @StartTime OR @StartTime IS NULL) 
      AND (o.CreatedOn < @EndTime OR @EndTime IS NULL) 
      AND (o.ShippingEmail = @CustomerEmail OR @CustomerEmail IS NULL) 
      AND (o.BillingFirstName + ' ' + o.BillingLastName = @CustomerName OR @CustomerName IS NULL) 
      AND (o.ShippingFirstName + ' ' + o.ShippingLastName = @CustomerName OR @CustomerName IS NULL) 
      AND (o.ShippingMethod = @ShippingMethod OR @ShippingMethod IS NULL) 
      AND (o.OrderStatusID = @MemberOrderProcessing 
       OR o.OrderStatusID = @MemberOrderProcessing 
       OR o.OrderStatusID = @MemberOrderComplete 
       OR o.OrderStatusID = @MemberOrderStatusCancelled 
       OR o.OrderStatusID = @MemberOrderStatusCancelledDiscontinued 
       OR o.OrderStatusID = @MemberOrderStatusCancelledCustomerRequest 
       OR o.OrderStatusID = @MemberOrderStatusCancelledPendingNeverPaid 
       OR @MemberOrderProcessing IS NULL 
       OR @MemberOrderProcessing IS NULL 
       OR @MemberOrderComplete IS NULL 
       OR @MemberOrderStatusCancelled IS NULL 
       OR @MemberOrderStatusCancelledDiscontinued IS NULL 
       OR @MemberOrderStatusCancelledCustomerRequest IS NULL 
       OR @MemberOrderStatusCancelledPendingNeverPaid IS NULL) 
    ORDER BY 
      o.OrderID 

END 

Answer 1

Lorsque vous avez trop de conditions OR, les performances risquent de s'en ressentir (sans compter que cela entraînerait un reniflage des paramètres). Je recommande fortement d'utiliser Dynamic SQL ici. Quelque chose comme ça,

DECLARE @query VARCHAR(MAX) 

SET @query = 
'SELECT DISTINCT o.OrderID, o.OrderTotal, o.BillingFirstName + ' ' + o.BillingLastName AS CustomerName, o.CreatedOn AS CreatedOn FROM Order o 
WHERE 1=1 ' 

IF @StartTime IS NOT NULL 
SET @query = @query + ' AND o.CreatedOn > @StartTime' 

IF @EndTime IS NOT NULL 
SET @query = @query + ' AND o.CreatedOn < @EndTime' 

IF @CustomerEmail IS NOT NULL 
SET @query = @query + ' AND o.ShippingEmail = @CustomerEmail' 
...... 
...... 

exec sp_executesql @query, 
    N'@StartTime DATETIME, 
    @EndTime DATETIME, 
    ...<other param definitions>', 
    @StartTime, 
    @EndTime, 
    .. <other param values> 

Answer 2

SI le OrderStatusID est un champ de bits, suivant pourrait travailler

SELECT DISTINCT o.OrderID 
      , o.OrderTotal 
      , o.BillingFirstName + ' ' + o.BillingLastName AS CustomerName 
      , o.CreatedOn AS CreatedOn 
    FROM Order o 
    WHERE (o.CreatedOn > COALESCE(@StartTime, '01-01-1899')) 
      AND (o.CreatedOn < COALESCE(@EndTime, '01-01-2099')) 
      AND (o.BillingFirstName + ' ' + o.BillingLastName = COALESCE(@CustomerName, o.BillingFirstName + ' ' + o.BillingLastName)) 
      AND (o.ShippingFirstName + ' ' + o.ShippingLastName = COALESCE (@CustomerName, o.ShippingFirstName + ' ' + o.ShippingLastName))   
      AND (o.ShippingEmail = COALESCE(@CustomerEmail, o.ShippingEmail) 
      AND (o.ShippingMethod = COALESCE (@ShippingMethod, o.ShippingMethod) 
      AND (o.OrderStatusID & ( 
        COALESCE (@MemberOrderProcessing, 1) 
        | COALESCE (@MemberOrderComplete, 2) 
        | COALESCE (@MemberOrderStatusCancelled , 4) 
        | COALESCE (@MemberOrderStatusCancelledDiscontinued , 8) 
        | COALESCE (@MemberOrderStatusCancelledCustomerRequest , 16) 
        | COALESCE (@MemberOrderStatusCancelledPendingNeverPaid , 32) 
       ) >= 1 
      ) 
    ORDER BY 
      o.OrderID 

Answer 3

meilleure source pour les conditions de recherche dynamiques:

Dynamic Search Conditions in T-SQL by Erland Sommarskog

il y a beaucoup de implications subtiles sur la façon dont vous faites cela pour savoir si un index peut être utilisé ou non. Si vous êtes sur la bonne version de SQL Server 2008, vous pouvez simplement ajouter OPTION (RECOMPILE) à la requête et la valeur de la variable locale au moment de l'exécution est utilisée pour les optimisations.

Considérez ceci, OPTION (RECOMPILE) prendra ce code (où aucun indice peut être utilisé avec ce gâchis de OR s):

WHERE 
    (@search1 IS NULL or Column1=@Search1) 
    AND (@search2 IS NULL or Column2=@Search2) 
    AND (@search3 IS NULL or Column3=@Search3) 

et optimiser au moment de l'exécution à (à condition que seul @ Search2 a été adoptée avec une valeur):

WHERE 
    Column2=@Search2 

et un index peut être utilisé (si vous en avez défini sur Colonne2)

si vous n'êtes pas sur la version nécessaire de SQL Server 2008, l'article lié propose de nombreuses méthodes avec des avantages et des inconvénients pour chacun. Par exemple, si vous pouvez déterminer une plage minimale et maximale pour votre colonne de recherche et que la colonne de recherche n'est PAS NULLE, alors vous pouvez faire mieux que le (@Search IS NULL ou Col = @ Search), see this area of the above linked article. Cependant vous devriez lire l'article entier, il y a tellement de variations qui dépendent de votre situation, vous avez vraiment besoin d'apprendre de multiples approches et quand les utiliser.