如何在AVG语句中排除DATEDIFF负值？SQL语法错误求助

嘿，刚接触SQL遇到语法问题太正常了，我来帮你把这个语句理顺，顺便给你几个实用的小技巧～

先说说你原语句的核心问题 #

你外层的SELECT Avg_DayDiff case when Avg_DayDiff > 0存在两个关键语法错误：

1. 缺少AVG()聚合函数：你要计算平均值，必须用AVG()包裹目标列，直接写列名只会返回所有行的原始差值，不是平均值。
2. CASE表达式不完整：CASE的正确结构是CASE WHEN 条件 THEN 返回值 ELSE 替代值 END，你既没写THEN的返回内容，也没加END，位置也放错了。

两种修正后的可行方案 #

方案一：子查询里直接过滤负值（推荐，更高效） #

在子查询阶段就把差值为负的行排除掉，外层直接计算剩余行的平均值：

SELECT AVG(Avg_DayDiff) AS Avg_DayDiff
FROM (
    SELECT DATEDIFF(DAY, xx, yy) AS Avg_DayDiff
    FROM Database1.dbo.tbl_1
    WHERE MONTH(datecompleted) = MONTH(DATEADD(MONTH, -1, CURRENT_TIMESTAMP))
      AND YEAR(datecompleted) = YEAR(DATEADD(MONTH, -1, CURRENT_TIMESTAMP))
      AND ApprovalRequiredFrom = 'GRM'
      -- 直接在这里排除差值为负的行
      AND DATEDIFF(DAY, xx, yy) > 0
) temptable

方案二：用CASE将负值转为NULL，AVG自动忽略NULL #

如果需要保留子查询的所有数据（比如后续还要复用），可以在外层用CASE把负值转为NULL——因为AVG()函数会自动忽略NULL值，从而达到排除负值的效果：

SELECT AVG(CASE WHEN Avg_DayDiff > 0 THEN Avg_DayDiff ELSE NULL END) AS Avg_DayDiff
FROM (
    SELECT DATEDIFF(DAY, xx, yy) AS Avg_DayDiff
    FROM Database1.dbo.tbl_1
    WHERE MONTH(datecompleted) = MONTH(DATEADD(MONTH, -1, CURRENT_TIMESTAMP))
      AND YEAR(datecompleted) = YEAR(DATEADD(MONTH, -1, CURRENT_TIMESTAMP))
      AND ApprovalRequiredFrom = 'GRM'
) temptable

几个实用的SQL小技巧 #

• 优化日期过滤逻辑：你现在用MONTH()和YEAR()包裹datecompleted的写法，会导致这列上的索引失效（数据库无法直接用索引计算函数结果）。可以改成直接计算上个月的起止日期，让查询更快：

  WHERE datecompleted >= DATEFROMPARTS(YEAR(DATEADD(MONTH, -1, CURRENT_TIMESTAMP)), MONTH(DATEADD(MONTH, -1, CURRENT_TIMESTAMP)), 1)
    AND datecompleted < DATEFROMPARTS(YEAR(CURRENT_TIMESTAMP), MONTH(CURRENT_TIMESTAMP), 1)
  

• 牢记CASE表达式结构：CASE永远是CASE WHEN ... THEN ... ELSE ... END的完整结构，缺一不可，它可以用来做条件判断、值替换，是SQL里非常灵活的工具。
• 利用聚合函数的特性：AVG()、SUM()这类聚合函数都会自动忽略NULL值，利用这一点可以灵活过滤不需要计算的数据，不用额外写复杂的过滤条件。

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