必须声明标量变量 - 使用游标

Question

这可能听起来很傻。但是我试图从两个表的比较中获取最后修改的日期值，但是我得到“必须声明标量变量@pa”错误。我有更多的列，这就是我使用游标的原因检查

declare @pa varchar(50)

declare audit_cur cursor for
     select distinct audit_field
     from #Iam
     where concat(',',@af,',') like concat('%,',audit_field,',%')

OPEN audit_cur
FETCH NEXT FROM audit_cur into @pa

declare @cmd varchar(1000)

WHILE @@FETCH_STATUS = 0
BEGIN

    set @cmd=concat('SELECT a.A_n,i.audit_field,a.',@pa,' field_value,i.field_after
FROM #Iam_audit a
JOIN (Select  a.A_n,  a.field_after,audit_field
from #iam a(nolock)
inner join (Select a_n, max(Modified_Date) as maxdate
        from #iam a2(nolock)
        where a2.Audit_field=@pa
        group by a_n
        ) as aa  on aa.a_n = a.a_n  and aa.maxdate=a.modified_Date
        where a.Audit_Field=@pa ) i 
ON i.audit_field=''',@pa,''' AND i.A_n=a.A_n AND a.',@pa,'<>i.field_after')

print @cmd -- for debug

    exec(@cmd)

    FETCH NEXT FROM audit_cur into @pa
END

CLOSE audit_cur
DEALLOCATE audit_cur

Answer 1

您可能应该使用基于集合的查询而不是游标和动态sql，但是对于快速而脏的修复，您需要将动态SQL字符串中的此部分where a.Audit_Field=@pa更改为where a.Audit_Field='''+ @pa +'''动态sql不知道@pa变量，因为它在自己的范围内运行。

此外，您的SQL提示您在audit_field中使用逗号分隔值。 这是一件非常糟糕的事情 - 欲了解更多信息，请阅读，你会看到很多理由说明为什么这个问题的答案是绝对是的！

如果您想要了解如何使用基于集合的查询而非此混乱，请发布新查询或编辑此查询以将示例数据包含为Is storing a delimited list in a database column really that bad? + DDL，并且需要结果