CODEWITHSUNDEEP
c#ado.net
chobo
chobo

Reputation: 32311

Using parameter for In() not working

I can't seem to get this to work with sql parameters

string itemIds = "86,74,32";

cmd.Parameters.AddWithValue("Items", itemIds);

                    string sql = "SELECT * " +
                                 "FROM Items " +
                                 "WHERE ItemIds IN(@Items)";

Upvotes: 0

Views: 66

Answers (2)

adt
adt

Reputation: 4360

You should add each parameter seperately.

string item1 = "86";
string item2 = "32";

cmd.Parameters.AddWithValue("item1", item1);
cmd.Parameters.AddWithValue("item2", item2);

                    string sql = "SELECT * " +
                                 "FROM Items " +
                                 "WHERE ItemIds IN(@item1,@item2)";

This will probably work.

Edit with for loop

string [] numbers = {"12", "23", "34"};
string [] parameters = new string[numbers.Length];
for (int i = 0; i < numbers.Length; i++)
{
     cmd.Parameters.AddWithValue("param"+i, numbers[i]);
     parameters[i] = "@param" + i;
}

var str = String.Join(",", parameters);
string sql = String.Format("SELECT * FROM Items WHERE ItemIds IN({0})",str);

Upvotes: 2

Christian Specht
Christian Specht

Reputation: 36441

You can't put the whole list of In() arguments in one SQL parameter, you have to put each one in a separate parameter instead.

Here is an elegant way to construct the query string and add the parameters:
Parameterize an SQL IN clause

Upvotes: 1

Related Questions