动态SQL查询导入活动量报错：'and Type=0'处语法错误

错误原因分析 #

你的代码存在4处关键语法问题，直接导致and Type=0位置报错：

• 字符串拼接断裂：and Cast(DateX as Date) between '+@FromDate+' and '+@ToDate这行结尾缺少' +，导致下一行的'and Type=0...无法和前面的字符串正确衔接，SQL解析时会把这部分当成独立语句，触发语法错误。
• 数据库名变量未正确替换：select * from @Database.[dbo].[History]中的@Database是硬编码字符串，不是变量引用，应该替换为存储过程参数@DatabaseName的拼接值。
• 日期参数未加单引号：@FromDate和@ToDate是字符串类型，拼接进SQL时需要用单引号包裹，否则会被解析为列名或数值，引发语法错误。
• 函数参数单引号未转义：调用dbo.fn_SplitString时，第二个参数是逗号，在动态SQL字符串中需要用两个单引号转义（''','''），否则会提前终止字符串。

修正后的完整代码 #

alter proc getVolumeCount
(@DatabaseName varchar(100),
@CompanyId as varchar(100),
@FromDate as varchar(100),
@ToDate as varchar(100)
)
as
begin
declare @SQL nvarchar(max)

set @SQL = 'use '+@DatabaseName+ '
select * from '+@DatabaseName+'.[dbo].[History]
where 1=1
and Cast(DateX as Date) between '''+@FromDate+''' and '''+@ToDate+'''
and Type=0
and Flags=3
and Id not in 
        (select ImportId from Item where CompanyId in
        (Select CAST(Item as int) from dbo.fn_SplitString('''+@CompanyId+''','',''''))
        )'

exec sp_executesql @SQL

END

额外优化建议 #

• 尽量避免直接拼接字符串生成动态SQL，推荐使用sp_executesql的参数化查询功能，能有效防止SQL注入，同时减少语法拼接错误。比如日期参数可以直接作为参数传入，无需拼接：

set @SQL = 'use '+@DatabaseName+ '
select * from '+@DatabaseName+'.[dbo].[History]
where 1=1
and Cast(DateX as Date) between @FromDateParam and @ToDateParam
and Type=0
and Flags=3
and Id not in 
        (select ImportId from Item where CompanyId in
        (Select CAST(Item as int) from dbo.fn_SplitString(@CompanyIdParam,'',''))
        )'

exec sp_executesql @SQL, 
    N'@FromDateParam date, @ToDateParam date, @CompanyIdParam varchar(100)',
    @FromDateParam = @FromDate, 
    @ToDateParam = @ToDate,
    @CompanyIdParam = @CompanyId

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