Pass a list to the stored procedure doesn't work

When I try this query it works:

SELECT *
FROM tbl_Users
WHERE UserId IN (555, 3695, 8787))

But when I put it in stored procedure it get value only if I pass one ID. If I pass more ID's

I got nothing returned from database:

CREATE PROCEDURE myStoredProcedure
    @m_UserIdList varchar(500)
AS    
    SELECT u.*, p.*
    FROM tbl_Users u 
    INNER JOIN tbl_Bunker b 
    ON u.BunkerId = b.Bunker
    WHERE u.UserId IN (@m_UserIdList)

Answers


Yes, it is treating @m_UserIdList as one single value and comparing that to u.UserId. You need to split out the values and test them separately. Oded beat me to the punch, but he is right -- you should use a table-valued param. Here is the link from Oded:

http://www.sommarskog.se/arrays-in-sql-2008.html

This site offers a bunch of different approaches to this problem:

http://vyaskn.tripod.com/passing_arrays_to_stored_procedures.htm

One of the approaches would be to use dynamic SQL, however you should be aware of the dangers of doing this. Read this first:

http://www.sommarskog.se/dynamic_sql.html

If you decide to go that route, something like this could work:

DECLARE @SQL varchar(600)

SET @SQL = 
'SELECT u.*, p.*
FROM tbl_Users u
    INNER JOIN tbl_Bunker b ON u.BunkerId = b.Bunker
WHERE u.UserId IN (' + @m_UserIdList+ ')'

EXEC(@SQL)  

That's because your select statement is the equivalent of:

SELECT u.*, p.* 
FROM tbl_Users u  
INNER JOIN tbl_Bunker b ON u.BunkerId = b.Bunker 
WHERE u.UserId IN ('555, 3695, 8787') 

You're basically searching to see if UserId is a string that literally matches '555, 3695, 8787', which is not what you want.


Shark explained it better than I was going to. If I remember right you can do this, but it's been a long time since I had to do what you are after so I might be wrong.

EXEC('SELECT u.*, p.*
      FROM tbl_Users u 
      INNER JOIN tbl_Bunker b ON u.BunkerId = b.Bunker
      WHERE u.UserId IN (' + @m_UserIdList + ')')

@m_UserIdList is a VARCHAR(500), not a list of values, so this will not work. You can try and parse the passed in string into a table (and there are plenty of ways of doing so - just search this site).

However, since you are using SQL Server 2008, you should take a look at table valued parameters - these allow you to pass a table of values to a stored procedure.


Since it wasn't mentioned, you can use the XML data type.

BEGIN
DECLARE @xml XML
SELECT @xml = '<n>1</n><n>2</n><n>3</n>'

SELECT t.n.value('.','int')  FROM @xml.nodes('*') as t(n)
IF 1 IN (SELECT t.n.value('.','int')  FROM @xml.nodes('*') as t(n))
    PRINT('Yep')
IF 4 NOT IN (SELECT t.n.value('.','int')  FROM @xml.nodes('*') as t(n))
    PRINT('It Works')
END

It's not as efficient as table valued columns but it's easier to setup/use for the "Caller".

If the caller already has an array or list of elements to convert them to XML he just has to do a String.Join with "</n><n>" as the delimiter then if the resulting string is not empty prepend "<n>" and append "</n>"


Need Your Help

How to convert web page to PDF?

php html pdf

I have PHP file that generate a report from the database, and I want this page to be converted into PDF File, so I can be save and print. How to convert web page to PDF? Is there any tool available...

RestKit: How to POST a NSManagedObject as JSON without any nesting attributes?

ios iphone objective-c restkit restkit-0.20

This should be a really easy one, but sadly I haven't found any answer...

About UNIX Resources Network

Original, collect and organize Developers related documents, information and materials, contains jQuery, Html, CSS, MySQL, .NET, ASP.NET, SQL, objective-c, iPhone, Ruby on Rails, C, SQL Server, Ruby, Arrays, Regex, ASP.NET MVC, WPF, XML, Ajax, DataBase, and so on.