含Case When与DATEDIFF的SQL查询语法错误求助

需求说明 #

当R.DateofRepairInvoice不为空且[Last Repairs Rcvd]为空时，计算动态基准日期与当前日期的差值：

• 若R.DateofRepairInvoice的日部分≤24，基准日期取当月月末
• 否则基准日期取下月15日
  最终将差值转为varchar格式并处理空值。

原错误代码 #

(case 
when R.DateofRepairInvoice is not null and [Last Repairs Rcvd] is null 
        then isnull(convert(varchar,(DATEDIFF((Case when Format(R.DateofRepairInvoice, 'dd') <= '24' 
                then EOMONTH ( R.DateofRepairInvoice ,0 ) 
                else DATEFROMPARTS(YEAR(DATEADD(M,1,R.DateofRepairInvoice))
                    ,MONTH(DATEADD(M,1,R.DateofRepairInvoice)
                    ,15))), getdate())),103),'') end) as [Repair Cash Days]

错误分析 #

1. DATEDIFF函数缺少差值单位参数（比如DAY/MONTH），这是语法必填项
2. DATEFROMPARTS的括号不匹配：MONTH(DATEADD(...))部分少了闭合右括号，导致参数传递错误
3. CONVERT函数参数顺序错误：格式代码103应放在目标类型varchar之后，而非DATEDIFF结果之后
4. 使用FORMAT提取日部分存在性能损耗，且字符串比较不如直接取日期数字可靠

修正后的代码 #

CASE 
    WHEN R.DateofRepairInvoice IS NOT NULL AND [Last Repairs Rcvd] IS NULL 
    THEN ISNULL(
        CONVERT(VARCHAR, 
            DATEDIFF(DAY, -- 指定计算天数差值，可按需替换为MONTH/YEAR等
                CASE 
                    WHEN DAY(R.DateofRepairInvoice) <= 24 -- 直接提取日期的日数字，避免字符串转换
                    THEN EOMONTH(R.DateofRepairInvoice, 0)
                    ELSE DATEFROMPARTS(
                        YEAR(DATEADD(MONTH, 1, R.DateofRepairInvoice)),
                        MONTH(DATEADD(MONTH, 1, R.DateofRepairInvoice)),
                        15
                    )
                END, 
                GETDATE()
            ),
            103 -- 格式参数放在CONVERT的第三个位置，符合函数语法
        ),
        ''
    ) 
END AS [Repair Cash Days]

修正细节说明 #

• 补充DATEDIFF的单位参数DAY，明确计算逻辑
• 修复DATEFROMPARTS的括号结构，确保年、月、日三个参数正确传入
• 调整CONVERT的参数顺序，符合SQL Server函数语法规范
• 用DAY()函数替代FORMAT()提取日部分，提升性能并避免字符串比较的潜在问题

内容的提问来源于stack exchange，提问作者John Maygee