이 저장 프로 시저에 어떤 문제가 있습니까?

Question

나는이 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 

이 저장 프로 시저에 문제가 있습니까? 이 저장 프로 시저를 어떻게 다시 작성 하시겠습니까?

Answer 1

제공 한 내용에 따라 proc은 매우 간단합니다. 성능 문제가있는 경우 모든 필터링이 해당 필드에 있으므로 인덱스는 DateAndTime이어야합니다. 당신은 빨리 당신이 아마 당신의 색인을보고 고려해야 성능을 찾고 있다면 내가

WHERE DateAndTime between @FromDateTime_Actual AND @ToDateTime_Actual 

을 할 수도 있지만 나는 그것이 성능

에 영향을 기대하지 않을 것이다 제외하고

Answer 2

나에게 물건을 본다 예를 들어 DateAndTime을 시작하는 것이 좋습니다. 당신이 Quassnoi가 언급 한 바와 같이 Perforamnce 팁 기사 like this one

보고해야합니다 그 후

, BETWEEN는 < = @FromDateTime_Actual에 < @FromDateTime_Actual을 변경하는 포괄적이다. 당신이 정말로 그것을 포괄적이 아니고 당신이 원한다면 BETWEEN 당신은 당신이 @ ToDateTime_Actual을 어떻게 설정 하는지를 수정할 수 있습니다.

Answer 3

이 저장 프로 시저에 아무런 문제가 없습니다.

당신은 변수 치우는하여 조금 더 읽기 쉽게 만들 수 있습니다 :

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 

하고 당신이 그것으로 발생하는 문제는 무엇