T-SQL使用动态变量循环处理交易日期范围的报错及优化咨询

你完全不需要依赖临时表、循环和动态SQL来完成这个需求，下面两种方法都能更简洁高效地实现你的目标，还能避免日期字符串转换带来的错误：

方法1：使用窗口函数（推荐） #

窗口函数可以直接为每条记录计算出所属用户的首次交易时间，然后直接筛选时间范围即可：

SELECT t.*
FROM (
    SELECT 
        *,
        -- 为每个用户计算首次交易时间
        MIN(transaction_ts) OVER (PARTITION BY user_id) AS first_trans_time
    FROM [dbo].[Transactions]
) t
-- 筛选首次交易时间到之后72小时内的交易
WHERE transaction_ts BETWEEN t.first_trans_time AND DATEADD(HOUR, 72, t.first_trans_time)

方法2：使用关联子查询 #

先通过分组查询得到每个用户的首次交易时间，再关联原表筛选符合条件的记录：

SELECT t1.*
FROM [dbo].[Transactions] t1
INNER JOIN (
    -- 先获取每个用户的首次交易时间
    SELECT user_id, MIN(transaction_ts) AS first_trans_time
    FROM [dbo].[Transactions]
    GROUP BY user_id
) t2 ON t1.user_id = t2.user_id
WHERE t1.transaction_ts BETWEEN t2.first_trans_time AND DATEADD(HOUR, 72, t2.first_trans_time)

原代码的问题分析 #

你之前的代码报错主要是因为动态SQL拼接时没有正确处理datetime类型：直接把datetime变量拼到字符串里会导致语法错误，正确的做法需要将日期转为标准格式的字符串并包裹单引号，但这种方式不仅繁琐，还容易引发SQL注入风险，同时循环执行多次查询的性能也很差，数据量大时会明显变慢。

而上面两种集合式查询的方法，只需要一次扫描或关联就能得到结果，性能和可读性都远优于循环写法。

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