SQL Server 쿼리 향상

Question

이 쿼리를 향상시켜야합니다.

DECLARE  @timTimeout int, 
     @iniDate varchar(20), 
     @endDate varchar(20)    
SET   @iniDate = '2014-07-20 00:00:00'    
SET   @endDate = '2014-11-24 23:59:59'   
SET   @timTimeout = 4000      

SET ANSI_WARNINGS OFF 

SELECT 
      'Approved (0200)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0200' THEN 1 END), 0), 
      'Approved Off (0220)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0220' THEN 1 END), 0), 
      'Cancel (0400)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0400' THEN 1 END), 0), 
      'Regret (0420)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0420' THEN 1 END), 0), 
      'TOTAL' = COUNT(*), 
      'Time-outs' = ISNULL(SUM(CASE WHEN DATEDIFF(ms, DateMsgIncome, DateMsgSent) > @timTimeout THEN 1 END), 0), 
      'Disponibility (%)' = (1 - CAST(ISNULL(SUM(CASE WHEN DATEDIFF(ms, DateMsgIncome, DateMsgSent) > @timTimeout THEN 1 END), 0) as money)/COUNT(*)) * 100 
FROM Message (NOLOCK) 
    WHERE DateMsgIncome BETWEEN @iniDate AND @endDate 
      AND CodMsgIncome IN ('0200', '0220', '0400', '0420', '0800', '0900', '9080', '9085') 
      AND DescMsgIncome <> '0220' 

이제 전체 데이터를 월별로 정리하여 보고서를 준비해야합니다.

출력 disered은 다음과 같다 :

 Approved (0200) | Approved Off (0220) | Cancel | Total | Time-outs | Disponibility (%) 
July | 35     15     12  62  0   100.00 
. 
. 
. 

편집 : 그것은 내 쿼리에 하나 개의 테이블입니다.

Table Message: 
DateMsgIncome date, 
DateMsgSent date, 
CodMsgIncome varchar(4), 
DescMsgIncome varchar(4), 
CodMsgAnswer int. 

아무쪼록 알려주세요. 미리 감사드립니다.

Answer 1

코드 정리 도구를 사용하여 쿼리를 실행하여 정리를 도와줍니다. 또한 내가 말한 것을 이해하지 못했기 때문에 변수 선언을 변경합니다. 기록을 위해, 코딩 한 방식으로 지난 수 밀리 초 동안 행을 놓친 것일 수 있습니다.

DATEDIFF 함수에서 datepart 이름의 철자가 잘못 사용 되었기 때문에 DATEDIFF 함수를 변경했습니다. 잘못된 약어를 ​​사용하기가 너무 쉽기 때문입니다. 또한 마지막 열의 계산을 단순화했습니다. 1에서 1.0으로 변경하면 돈으로 변환 할 필요가 없습니다. 개체 이름에 예약어를 사용하지 말고 열 이름에 공백을 두지 마십시오. 프론트 엔드가 이런 종류의 형식화를하게하십시오.

테이블 힌트를 사용할 때 WITH 키워드가 곧 추가되었습니다. (나는 그것을 사용하기 전에 NOLOCK이 실제로 무엇을 의미하는지 이해할 것을 권합니다).

DECLARE @timTimeout int 
    , @iniDate date 
    , @endDate date 

SET @iniDate = '2014-07-20'    
SET @endDate = '2014-11-25'   
SET @timTimeout = 4000      

SELECT MONTH(DateMsgIncome) as MyMonthColumn 
    , 'Approved (0200)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0200' THEN 1 END), 0) 
    , 'Approved Off (0220)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0220' THEN 1 END), 0) 
    , 'Cancel (0400)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0400' THEN 1 END), 0) 
    , 'Regret (0420)' = ISNULL(SUM(CASE CodMsgIncome WHEN '0420' THEN 1 END), 0) 
    , 'TOTAL' = COUNT(*) 
    , 'Time-outs' = ISNULL(SUM(CASE WHEN DATEDIFF(MILLISECOND, DateMsgIncome, DateMsgSent) > @timTimeout THEN 1 END), 0) 
    , 'Disponibility (%)' = (1.0 - ISNULL(SUM(CASE WHEN DATEDIFF(MILLISECOND, DateMsgIncome, DateMsgSent) > @timTimeout THEN 1 END), 0)/COUNT(*)) * 100 
FROM [Message] WITH (NOLOCK) --Ack!!! I wouldn't let this fly on my system due to inconsistencies with this hint unless accuracy is not important (like 

WHERE DateMsgIncome >= @iniDate 
    AND DateMsgIncome < @endDate 
    AND CodMsgIncome IN 
    (
     '0200' 
     , '0220' 
     , '0400' 
     , '0420' 
     , '0800' 
     , '0900' 
     , '9080' 
     , '9085' 
    ) 
    AND DescMsgIncome <> '0220' 
GROUP BY MONTH(DateMsgIncome)