Member 13936784 Ответов: 3

Проблема с форматами данных , ошибка запроса insert из приложения C#

Я использую следующий код в кнопке сохранения приложения c# windows.

КМД = новая команда sqlcommand("вставить в sman_targets(sman_cd,from_dt,to_dt,target_amt) значения('"+элемент combobox1.Текст +"','"+СТ1.Текст+"','"+ст2.Текст+"','"+поле textbox2.Текст+"')",кон);

УМК.Метод executenonquery();

DT1,DT2-это элементы управления datetime picker в (коротком формате) в форме c# windows.


Я использовал datatime "datetime" для столбцов таблицы в базе данных sql server.

при нажатии кнопки он показывает следующие ошибки. Кажется, здесь есть какая-то проблема
с форматами данных.

Необработанное исключение типа "System.Data.SqlClient.SqlException" произошло в System.Data.dll

Дополнительная информация: преобразование типа данных char в тип данных datetime привело к выходу значения datetime за пределы диапазона.

плз дайте решение.

спасибо и с уважением

Мадху

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

Я попытался изменить двойные кавычки / одиночные кавычки оператора insert и т. д
также путем изменения свойства format элемента управления datatime picker.
Но появляется та же самая ошибка.

Richard MacCutchan

Вы берете текстовое значение datepicker и пытаетесь опубликовать его как тип datetime. Сделайте то, что советует OriginalGriff, сначала получите правильные значения и типы и передайте их как значения параметров, а не как текст.

3 Ответов

Рейтинг:
26

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;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

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

Исправьте это на протяжении всего вашего приложения, проверьте вводимые пользователем данные (и преобразуйте их в соответствующие типы данных .NET), и ваша проблема исчезнет в то же время. А не ваша база данных.

Рейтинг:
2

MadMyche

После того как вы пройдете и прочтете как это сделать должным образом создайте инструкцию SQL; вы должны получить что-то похожее на следующий протокод. Возможно, вам понадобится/захочется работать с переменными, входящими в обрезку пробелов и проверку значения DateTime.

cmd = new SqlCommand("INSERT sman_targets(sman_cd,from_dt,to_dt,target_amt) VALUES (@comboBox1, @DT1, @DT2, @textBox2)", con);

cmd.Parameters.AddWithValue("@comboBox1", comboBox1.Text);
cmd.Parameters.AddWithValue("@DT1", (DateTime)DT1.Text);
cmd.Parameters.AddWithValue("@DT2", (DateTime)DT2.Text);
cmd.Parameters.AddWithValue("@textBox2", textBox2.Text);

cmd.ExecuteNonQuery();

Рейтинг:
18

Patrice T 

cmd = new SqlCommand("insert into sman_targets(sman_cd,from_dt,to_dt,target_amt) values('"+comboBox1.Text +"','"+DT1.Text+"','"+DT2.Text+"','"+textBox2.Text+"')",con);

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