Member 11297177 Ответов: 4

Число значений в предложении VALUES должно совпадать с числом столбцов, указанным в инструкции INSERT.


string str = "insert into TblCheckOut(Name,Address,Mobile,RoomType,InDate,OutDate,Days,RoomCharge,RoomBill,Otherservices,TotalBill,Status) values ('" + combocustomer.Text + "','" + lbladdress.Text + "','" + lblmobile.Text + "','" + lbltype.Text + "','" + dateTimePicker1.Text + "','" + dateTimePicker2.Text + "','" + lbldays.Text + "','" + txtroomcharge.Text + "','" + txtOtherServices.Text + "','" + txtTotoalBill.Text + "','" + combostatus.Text + "')";



Найдите приведенный выше код с запросом , но я столкнулся с проблемой, не могли бы вы помочь с этой ошибкой

Что я уже пробовал:

<pre>string str = "insert into TblCheckOut(Name,Address,Mobile,RoomType,InDate,OutDate,Days,RoomCharge,RoomBill,Otherservices,TotalBill,Status) values ('" + combocustomer.Text + "','" + lbladdress.Text + "','" + lblmobile.Text + "','" + lbltype.Text + "','" + dateTimePicker1.Text + "','" + dateTimePicker2.Text + "','" + lbldays.Text + "','" + txtroomcharge.Text + "','" + txtOtherServices.Text + "','" + txtTotoalBill.Text + "','" + combostatus.Text + "')";

4 Ответов

Рейтинг:
2

Patrice T

string str = "insert into TblCheckOut(Name,Address,Mobile,RoomType,InDate,OutDate,Days,RoomCharge,RoomBill,Otherservices,TotalBill,Status) values ('" + combocustomer.Text + "','" + lbladdress.Text + "','" + lblmobile.Text + "','" + lbltype.Text + "','" + dateTimePicker1.Text + "','" + dateTimePicker2.Text + "','" + lbldays.Text + "','" + txtroomcharge.Text + "','" + txtOtherServices.Text + "','" + txtTotoalBill.Text + "','" + combostatus.Text + "')";

Не обязательно решение вашего вопроса, но у вас есть еще одна проблема.
Никогда не создавайте SQL-запрос путем объединения строк. Рано или поздно вы сделаете это с помощью пользовательских вводов, и это откроет дверь к уязвимости под названием "SQL-инъекция", она опасна для вашей базы данных и подвержена ошибкам.
Одна кавычка в имени - и ваша программа выйдет из строя. Если пользователь вводит имя, например "Брайан О'Коннер", может привести к сбою вашего приложения, это уязвимость SQL-инъекции, и сбой-это наименьшая из проблем, вредоносный пользовательский ввод, и он продвигается к командам SQL со всеми учетными данными.
SQL-инъекция - Википедия[^]
SQL-инъекция[^]
Атаки SQL-инъекций на примере[^]
PHP: SQL-инъекция - руководство пользователя[^]
Как я могу объяснить SQL-инъекцию без технического жаргона? - Обмен Стеками Информационной Безопасности[^]


Рейтинг:
2

MadMyche

Правильный способ сделать это, как уже говорилось ранее, заключается в использовании Коллекция Параметров Sql[^].

Теперь самый простой способ реализовать в вашем случае будет разбить часть INSERT table and columns вашей команды на несколько строк следующим образом

INSERT TblCheckOut
( Name
, Address
, Mobile
, RoomType
, InDate
, OutDate
, Days
, RoomCharge
, RoomBill
, Otherservices
, TotalBill
, Status
) 
А затем реплицируйте это в свои значения, предварительно ожидая амперсанда на различные имена столбцов, подобные этому для половины значений вашего запроса
VALUES
( @Name
, @Address
, @Mobile
, @RoomType
, @InDate
, @OutDate
, @Days
, @RoomCharge
, @RoomBill
, @Otherservices
, @TotalBill
, @Status
)
А потом возьми их ценности еще раз; скопируйте их и предварительно добавьте эти строки с помощью cmd.Parameters.AddWithValue (заменять УМК чтобы отразить имя переменной SqlCommand)
cmd.Parameters.AddWithValue("@Name", );
cmd.Parameters.AddWithValue("@Address", );
cmd.Parameters.AddWithValue("@Mobile", );
cmd.Parameters.AddWithValue("@RoomType", );
cmd.Parameters.AddWithValue("@InDate", );
cmd.Parameters.AddWithValue("@OutDate", );
cmd.Parameters.AddWithValue("@Days", );
cmd.Parameters.AddWithValue("@RoomCharge", );
cmd.Parameters.AddWithValue("@RoomBill", );
cmd.Parameters.AddWithValue("@Otherservices", );
cmd.Parameters.AddWithValue("@TotalBill", );
cmd.Parameters.AddWithValue("@Status", );
А затем вы заполняете значения из ваших текстовых полей, выпадающих списков и т. д... И теперь мы можем увидеть, какой предмет вы забыли
cmd.Parameters.AddWithValue("@Name", combocustomer.Text);
cmd.Parameters.AddWithValue("@Address", lbladdress.Text);
cmd.Parameters.AddWithValue("@Mobile", lblmobile.Text);
cmd.Parameters.AddWithValue("@RoomType", lbltype.Text);
cmd.Parameters.AddWithValue("@InDate", dateTimePicker1.Text);
cmd.Parameters.AddWithValue("@OutDate", dateTimePicker2.Text);
cmd.Parameters.AddWithValue("@Days", lbldays.Text);
cmd.Parameters.AddWithValue("@RoomCharge", txtroomcharge.Text);
cmd.Parameters.AddWithValue("@RoomBill", );                    // missing value
cmd.Parameters.AddWithValue("@Otherservices", txtOtherServices.Text);
cmd.Parameters.AddWithValue("@TotalBill", txtTotoalBill.Text);
cmd.Parameters.AddWithValue("@Status", combostatus.Text);
И на самом деле в этом состоянии ваша программа, скорее всего, не будет компилироваться.

Очевидно, что как только вы все это проработаете, вы сможете удалить все разрывы строк; поэтому ваш окончательный код должен быть похож на этот
string str = "INSERT TblCheckOut (Name,Address,Mobile,RoomType,InDate,OutDate,Days,RoomCharge,RoomBill,Otherservices,TotalBill,Status) VALUES (@Name,@Address,@Mobile,@RoomType,@InDate,@OutDate,@Days,@RoomCharge,@RoomBill,@Otherservices,@TotalBill,@Status)";

SqlCommand cmd = new SqlCommand(str, conn);
cmd.Parameters.AddWithValue("@Name", combocustomer.Text);
cmd.Parameters.AddWithValue("@Address", lbladdress.Text);
cmd.Parameters.AddWithValue("@Mobile", lblmobile.Text);
cmd.Parameters.AddWithValue("@RoomType", lbltype.Text);
cmd.Parameters.AddWithValue("@InDate", dateTimePicker1.Text);
cmd.Parameters.AddWithValue("@OutDate", dateTimePicker2.Text);
cmd.Parameters.AddWithValue("@Days", lbldays.Text);
cmd.Parameters.AddWithValue("@RoomCharge", txtroomcharge.Text);
cmd.Parameters.AddWithValue("@RoomBill", );                      // missing value
cmd.Parameters.AddWithValue("@Otherservices", txtOtherServices.Text);
cmd.Parameters.AddWithValue("@TotalBill", txtTotoalBill.Text);
cmd.Parameters.AddWithValue("@Status", combostatus.Text);


Теперь еще несколько замечаний по этому поводу:
1. все ваши значения вводятся в виде текста; некоторые из них кажутся числовыми или датами
2. у вас есть имена столбцов, которых обычно избегают, поскольку они являются специальными или зарезервированными словами


Рейтинг:
2

Member 11297177

Я понял это , я пропустил здесь одно значение txtbill.text. работает нормально .


MadMyche

Если бы вы все делали правильно, у вас не было бы этой проблемы с самого начала.

Dave Kreskowiak

Да, это будет прекрасно работать, вплоть до того момента, когда кто-то введет Апостроф в одно из этих полей. Если кто-то введет имя типа "Пит О'Лири", ваш код снова выйдет из строя.

Перепишите код с помощью параметров, и у вас не будет этой проблемы.

Рейтинг:
0

OriginalGriff

Даже если вы думаете, что исправили это, не делайте этого так!
Никогда не объединяйте строки для построения команды SQL. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого всегда используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

Да, и проверьте свои входные данные, прежде чем передавать их в SQL - числа должны быть проверены с помощью TryParse и числового значения, передаваемого в качестве параметра, значения DateTimePicker должны передаваться как DateTime, а не строка.
И этот лот подразумевает, что вы храните все как VARCHAR или NVARCHAR, что очень плохая идея - это означает, что вам нужно преобразовать его, когда вы хотите сделать с ним что - то полезное - сравнить его, суммировать, отсортировать-и именно тогда вы найдете все плохие данные в своей БД.
Проверяйте вводимые пользователем данные - мы все делаем ошибки - и храните только хорошие данные в соответствующих типах данных. Это кажется дополнительной работой, но это довольно тривиально, и это экономит вам массу работы позже.