执行含sp_executesql的存储过程时出现varchar转int类型转换错误

咱们直接点出问题根源：你遇到的Msg 8114转换错误，本质是调用sp_executesql时传入的参数顺序和你定义的参数模板顺序不匹配，导致类型不兼容的参数被错误绑定。

具体原因分析 #

看你定义的参数模板：

SET @HoldEntryParamDefinition = N'@Department VARCHAR(50),@StoreID INT,@NewId INT,@TransactionDateFrom DATETIME,@TransactionDateTo DATETIME'

这里的参数顺序是：@Department → @StoreID → @NewId → @TransactionDateFrom → @TransactionDateTo

但你执行sp_executesql时的参数传递顺序是：

EXECUTE sp_executesql @HoldEntryQry, @HoldEntryParamDefinition, @NewId, @Department, @Storeid,@TransactionDateFrom, @TransactionDateTo

相当于把@NewId（整数类型）塞给了第一个参数@Department（字符串类型），接着又把@Department（字符串）塞给了第二个参数@StoreID（整数类型）——这就触发了“字符串转整数”的失败，因为@Department是类似'Apple','Orange'的字符串，根本没法转成整数。

快速修复方案 #

最简单的方法就是让参数传递顺序和模板定义的顺序完全一致，或者更稳妥的是用命名参数传递（避免顺序搞错）：

-- 方式1：严格按模板顺序传参
EXECUTE sp_executesql @HoldEntryQry, @HoldEntryParamDefinition, 
    @Department, @StoreID, @NewId, @TransactionDateFrom, @TransactionDateTo;

-- 方式2：用命名参数（更推荐，不怕顺序搞混）
EXECUTE sp_executesql @HoldEntryQry, @HoldEntryParamDefinition, 
    @Department = @Department, 
    @StoreID = @StoreID, 
    @NewId = @NewId, 
    @TransactionDateFrom = @TransactionDateFrom, 
    @TransactionDateTo = @TransactionDateTo;

更优的改进方案：完全参数化动态SQL #

你现在的代码里手动拼接了很多变量（比如@TransactionDateFrom、@StoreID、@Department），这种做法不仅容易引发类型问题，还存在SQL注入风险。建议改成完全参数化的写法：

1. 去掉手动拼接@Department的代码：

-- 删掉这段拼接代码
-- SET @Department=''''+REPLACE(@Department,',',''',''')+''''

2. 修改动态SQL语句，用参数替代拼接，并用STRING_SPLIT处理逗号分隔的部门参数：

SET @HoldEntryQry = N'
SELECT 
    te.StoreID, 
    @NewId AS TransactionHoldID, 
    te.ItemLookupCode, 
    item.Description, 
    te.ItemID, 
    SUM(te.Quantity) Quantity, 
    SUM(te.SalesTax) Tax, 
    SUM(te.Price * te.Quantity) Value, 
    SUM(te.Price * te.Quantity) / SUM(te.Quantity) Price, 
    item.price FullPrice, 
    1 AS Taxable, 
    GETDATE() AS TransactionTime, 
    ''1899-12-30'' AS VoucherExpirationDate 
FROM TransactionEntry te 
LEFT JOIN item ON item.id = te.ItemID 
INNER JOIN item nItem ON nitem.ItemLookupCode = item.itemlookupcode 
WHERE te.TransactionDate >= @TransactionDateFrom 
  AND te.TransactionDate <= @TransactionDateTo 
  AND item.department IN (SELECT value FROM STRING_SPLIT(@Department, '','')) 
  AND te.storeid = @StoreID
GROUP BY te.StoreID, te.ItemLookupCode, item.Description, te.ItemID, item.price '

3. 保持参数模板不变，按正确顺序传参即可。

这种写法既解决了类型转换问题，也让代码更安全、更易维护。

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