Quel est le problème avec cette procédure stockée?

Question

que je dois faire l'accord sur cette sp:

alter PROCEDURE Gaming.usp_DB_GetGameResultsByDateTime 
(@FromDateTime AS DATETIME2(7) = null , 
@ToDateTime AS DATETIME2(7) = null) 
AS 
    DECLARE @FromDateTime_Actual AS DATETIME2(7) , 
      @ToDateTime_Actual AS DATETIME2(7); 
    SET @FromDateTime_Actual = ISNULL(@FromDateTime, DATEADD (DAY, -1, SYSDATETIME())); 
    SET @ToDateTime_Actual = ISNULL (@ToDateTime , SYSDATETIME()); 

    SELECT Id, GameTypeId, PlayerId, 
      BetAmount, Profit, 
      DateAndTime 
    FROM Gaming.GameResults 
    WHERE DateAndTime >= @FromDateTime_Actual 
    AND DateAndTime < @ToDateTime_Actual 
    ORDER BY DateAndTime ASC; 
GO 

Quel est le problème avec cette procédure stockée? Comment réécrireiez-vous cette procédure stockée?

Answer 1

Basé sur ce que vous avez fourni, le proc est assez simple. Si vous avez des problèmes de performances, assurez-vous d'avoir un index sur DateAndTime car tout votre filtrage est sur ce champ.

Answer 2

Attend marchandises à moi, si je pourrais faire

WHERE DateAndTime between @FromDateTime_Actual AND @ToDateTime_Actual 

mais je m'y attendais pas à affecter les performances

Si vous cherchez à faire des performances plus rapides, vous devriez probablement envisager de regarder vos index par exemple DateAndTime pourrait être un bon endroit pour commencer.

Après cela, vous aurez besoin de regarder des articles Perforamnce Conseils like this one

Comme Quassnoi noté, BETWEEN est inclusif qui change < @FromDateTime_Actual à < = @FromDateTime_Actual. Si vous voulez vraiment qu'il ne soit pas inclusif et que vous voulez vous BETWEEN vous pouvez modifier la façon dont vous définissez @ToDateTime_Actual pour que cela se produise.

Answer 3

Il n'y a rien de mal à cette procédure stockée.

Vous pouvez faire un peu plus facile à lire en se débarrassant des variables:

ALTER PROCEDURE 
     Gaming.usp_DB_GetGameResultsByDateTime 
     (
     @FromDateTime AS DATETIME2(7) = NULL , 
     @ToDateTime AS DATETIME2(7) = NULL 
     ) 
AS 
DECLARE 
     SELECT Id, GameTypeId, PlayerId, BetAmount, Profit, DateAndTime 
     FROM Gaming.GameResults 
     WHERE DateAndTime >= ISNULL (@FromDateTime , DATEADD (DAY , -1 , SYSDATETIME())) 
       AND DateAndTime < ISNULL (@ToDateTime , SYSDATETIME()) 
     ORDER BY 
       DateAndTime ASC 
GO 

et plus efficace en créant un index sur GameResults (DateAndTime)