Sql server 存储过程始终从代码中获取第一个参数

Sql server 存储过程始终从代码中获取第一个参数,sql-server,vb.net,Sql Server,Vb.net,我有这样一个存储过程: ALTER PROCEDURE [dbo].[T_TransactionSummary] @locations nvarchar AS BEGIN .............. ............. AND (Location_tbl.Locid IN (@locations)) 我的locid字段是整数 此locid来自我的列表框。如果我选择一项,1 locid将出现。如果我选择2项,2 locid将出现。。 我有一个ListBox,它将@loc

我有这样一个存储过程:

ALTER PROCEDURE [dbo].[T_TransactionSummary]  
 @locations nvarchar
AS
    BEGIN 
..............
.............
AND (Location_tbl.Locid IN (@locations))
我的locid字段是整数 此locid来自我的列表框。如果我选择一项,1 locid将出现。如果我选择2项,2 locid将出现。。 我有一个ListBox,它将@locations参数填充为一个整数,我的ListBox值如下

cnt = LSTlocations.SelectedItems.Count
 Dim list As New List(Of Integer)
        Dim locid As Integer
        If cnt > 0 Then
            For i = 0 To cnt - 1
                Dim locationanme As String = LSTlocations.SelectedItems(i).ToString
                locid = RecordID("Locid", "Location_tbl", "LocName", locationanme)
                list.Add(locid)
            Next
End If
 Dim da As New SqlDataAdapter
        Dim ds As New DataSet
        Dim cmd23 As New SqlCommand("T_TransactionSummary", con.connect)
        cmd23.CommandType = CommandType.StoredProcedure
        cmd23.Parameters.Add("@locations", SqlDbType.Int).Value = String.Join(",", list)
        da.SelectCommand = cmd23
        da.Fill(ds)  

现在,我的locationid从listbox仅传递到存储过程1、2、3。但是存储过程总是取第一个值,我的意思是在这种情况下取1。

以下是第一个问题

cmd23.Parameters.Add("@locations", SqlDbType.Int).Value = String.Join(",", list)
您正在将@locations参数添加为int。当您为其赋值时,VB正在将字符串转换为int。这意味着1,2,3,4将变为1

将其更改为SqlDbType.VarChar

至于第二点,我不完全确定你能做到:

AND (Location_tbl.Locid IN (@locations))
您可能希望改为查看表值参数。

首先。。。您当前拥有的是一个在第一个字符处被截断的字符串

声明@locations NVARCHAR; 设置@locations='1,2,3'; 选择@个地点; 结果:

1. 你需要说

@瓦查尔马克斯酒店 您不需要NVARCHAR来存储逗号分隔的整数列表。我假设您可能有一个很长的整数列表,但MAX可能是8000

然后,您不能在@locations中说-这将无法正常工作,或者您将收到一条关于将“1,2,3…”转换为int的错误消息,或者它无法找到值-该值与整个字符串(而不是集合)进行比较。因此,您可以使用动态SQL来实现这一点,例如

设置@sql=@sql++,其中位置位于“++@locations+'…;”; 但这也充满了其他各种问题,包括可维护性和SQL注入的风险。我强烈建议改为使用表值参数。基本上,您可以创建如下类型:

ALTER PROCEDURE [dbo].[T_TransactionSummary]  
 @locations nvarchar
AS
    BEGIN 
..............
.............
AND (Location_tbl.Locid IN (@locations))
创建类型dbo.Integers作为TABLEItem INT主键; 然后按如下方式使用参数:

@位置dbo.Integers只读 你可以说:

如果存在,请从@locations中选择1,其中Item=Location\u tbl.Locid 在VB.Net代码中,将listbox选择填充到DataTable中,而不是int或字符串,并使用SqlDbType.Structured将DataTable作为参数传递。我这里有一些例子,但它们是C:


还有。

先生……我必须改变什么?@user2878851我想我已经在这里给出了非常彻底和有用的说明。@AaronBertrand:我同意。您的答案看起来很完整。先生,我首先执行:创建类型dbo.Integers作为TABLEItem INT主键;然后我将@locations dbo.Integers READONLY添加到我的sp中。但是获取错误必须声明标量变量@locations@user2878851这不是一个为我编写所有代码的论坛。尤其是当您只在第一时间包含了非常简短的原始代码段时。你看到我如何改变@locations变量的用法了吗?也许我应该给它起个不同的名字?你看过我提供的任何链接吗?我的答案帮助你学习如何使用这些技术?@user2878851:看看Aaron的答案。他告诉您将sp参数更改为@locations dbo.Integers READONLY的部分是一个TVP。我刚才在存储过程中给出了如下内容:@locations integer READONLY..现在我得到了一个错误:参数@locations不能声明为READONLY,因为它不是表值参数。您是否先执行了任何简单的调试?比如打印@位置;?不,先生…我为什么要这样做?因为在这种情况下,它会显示您传递的值被截断了。