我来帮你梳理下这个插入数据的存储过程写法，顺便提几个关键注意点，避免踩坑～

首先明确你的需求：要往Table6插入数据，且数据的时间范围介于当前日期减1和Table6已有的最大日期之间。你给出的代码片段已经在获取Table6的最大日期，但还有几个可以优化和完善的地方：

1. 先处理空值和时间范围边界 #

首先要考虑Table6为空表的情况（此时MAX(Date)会返回NULL），同时明确时间范围的上下限：

DECLARE @DateMax DATETIME, @DateMin DATETIME
-- 获取Table6的最大日期，若表为空则默认设为当前日期减1
SELECT @DateMax = ISNULL(MAX(Date), DATEADD(DAY, -1, GETDATE())) FROM Table6
-- 时间范围下限固定为当前日期减1
SET @DateMin = DATEADD(DAY, -1, GETDATE())

2. 优先用静态SQL（避免动态SQL的风险） #

如果不需要动态拼接其他条件，直接用静态SQL插入是最安全高效的：

INSERT INTO Table6 (One, Two, Three, Four, Five, Time)
SELECT 
    c.One, e.Two, e.Three, a.Four, d.Five, a.Time 
FROM Table1 a 
INNER JOIN Table2 b ON a.column1 = b.column1 
INNER JOIN Table3 c ON b.column2 = c.column2 
INNER JOIN Table4 d ON a.column3 = d.column3 
INNER JOIN Table5 e ON a.column4 = e.column4 
WHERE 
    -- 这里用BETWEEN要注意是否包含边界，若需要左闭右开可以改成 >= 和 <
    a.Time BETWEEN @DateMin AND @DateMax

3. 必须用动态SQL时，一定要参数化 #

如果业务上确实需要动态拼接SQL（比如有其他可变条件），绝对不要直接拼接变量，否则会有SQL注入风险和日期格式问题，改用参数化执行：

DECLARE @SQLCommand NVARCHAR(MAX)
SET @SQLCommand = N'
SELECT 
    c.One, e.Two, e.Three, a.Four, d.Five, a.Time 
FROM Table1 a 
INNER JOIN Table2 b ON a.column1 = b.column1 
INNER JOIN Table3 c ON b.column2 = c.column2 
INNER JOIN Table4 d ON a.column3 = d.column3 
INNER JOIN Table5 e ON a.column4 = e.column4 
WHERE 
    a.Time BETWEEN @MinDate AND @MaxDate'

-- 用sp_executesql传递参数，安全且避免格式问题
EXEC sp_executesql @SQLCommand, 
    N'@MinDate DATETIME, @MaxDate DATETIME',
    @MinDate = @DateMin,
    @MaxDate = @DateMax

几个额外的注意点 #

• 日期类型统一：确保Table6的Date列和Table1的Time列是相同的日期类型（比如DATETIME2比DATETIME更推荐），避免隐式转换导致的性能损耗
• 索引优化：如果Table1的Time列数据量很大，建议给这个列建非聚集索引，能大幅加快查询速度
• 边界逻辑调整：如果你的需求是a.Time要大于@DateMax且小于等于当前减1，记得调整WHERE条件的运算符，比如a.Time > @DateMax AND a.Time <= @DateMin

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