Как правильно писать SQL запрос с параметрами?

Question

[Дмитрий Гавриленко]

Всем доброго времени суток.

При выполнении запроса на вставку новой строки в таблицу, получаю сами меркеры параметров а не их значения.

void Insert()
{
//данные

//Color
SqlParameter paramColor = new SqlParameter();
paramColor.DbType = DbType.String;
paramColor.Direction = ParameterDirection.Input;
paramColor.ParameterName = "@Color";
paramColor.Value = item.Color;

command.Parameters.Add(paramColor);

command.CommandText = "insert into Inventory (Make, PetName, Color) values ('@MakeI', '@PetNameI', '@Color')";

result = command.ExecuteNonQuery() > 0 ? true : false;

}

В коде описан только один параметр, остальные описаны точно так же.

В общем в базу мне летят не значения param*******.Value, а имена маркеров, т.е. @MakeI, @PetNameI, @ Color

Sql Server 2012 поддерживает параметры в запросах.

Answer 1

[mayorovp]

insert into Inventory (Make, PetName, Color) values ('@MakeI', '@PetNameI', '@Color')"

Ну разумеется, в базу будут лететь строковые константы - вы же их и отправляете. Вот вы когда на C# пишете, названия строковых переменных в кавычки не берете - почему же вы так поступили на языке SQL?

PS это что еще такое?!

result = command.ExecuteNonQuery() > 0 ? true : false;

Уберите лишний тернарный оператор, а то будете индусом!

Comments

[Дмитрий Гавриленко]

Ну в оригинале у меня функция возвращает bool. А за кавычки спасибо.

[Дмитрий Гавриленко]

Или есть другая причина ругать меня за тернарный?

[mayorovp]

Дмитрий Гавриленко: Так в том-то и дело! Конструкция (b ? true : false) абсолютно бессмысленна - вместо нее можно написать просто b.

Вот так правильно: result = command.ExecuteNonQuery() > 0

[Дмитрий Гавриленко]

mayorovp: а, вы имеете ввиду возвращение самих true или false не имеет смысла? Не знал этих тонкостей. Спасибо!

Answer 2

[Валерий Охотников]

Пример
https://msdn.microsoft.com/ru-ru/library/system.da...

Comments

[Дмитрий Гавриленко]

Допустим, что я там был. И данный вариант мне не подходит. Это первое, а второе тут больше уместна ссылку https://msdn.microsoft.com/ru-ru/library/c6tdby40(v=vs.110).aspx

[Валерий Охотников]

В Вашем примере параметру значение не присваивается, задаются только его свойства.

[Дмитрий Гавриленко]

Валерий Охотников: paramColor.Value = item.Color;

[Валерий Охотников]

имел ввиду Вашу ссылку на пример

[Валерий Охотников]

А что касается Вашего примера, неправильна строка
command.CommandText = "insert into Inventory (Make, PetName, Color) values ('@MakeI', '@PetNameI', 'Илья Казначеев')";