Dr. Johnny Fever Ответов: 3

C# не может получить datatable для заполнения

У меня есть текстовое поле для ввода данных и кнопка для поиска этих данных, возвращенных в datagridview. Данные не передаются в datatable.


private void button1_Click(object sender, EventArgs e)
        {
            SqlConnection con = new SqlConnection("Data Source = AIRS1C49; Initial Catalog = BoundBook; Integrated Security = True");
            SqlCommand cmd = new SqlCommand();
            SqlDataAdapter sda = new SqlDataAdapter(@"SELECT Firearm.Serial_Number, Firearm.Manufacturer, Firearm.Country_of_Mfg, Firearm.Model, Firearm.Type, Firearm.[Action], Firearm.Gage_Caliber, Firearm.Status, Firearm.Acquired, Firearm.Disposed, Acquisition.Acq_ID,
                         Acquisition.Acq_Date, Acquisition.Acq_Reason, Acquisition.Serial_Number AS Expr1, Acquisition.Acq_Note, Acquisition.DataEntryDate_A, Acquisition.FFL_ID, FFL.FFL_ID AS Expr2, FFL.FFL_Name, FFL.FFL_Address1,
                         FFL.FFL_Address2, FFL.FFL_City, FFL.FFL_State, FFL.FFL_ZipCode, FFL.FFL_Email, FFL.FFL_Phone, FFL.FFL_License_Number, FFL.FFL_Expiration_Date, Disposition.Disp_ID, Disposition.Disp_Date, Disposition.Disp_Reason,
                         Disposition.Serial_Number AS Expr3, Disposition.FFL_ID AS Expr4, Disposition.Disp_Note, Disposition.DataEntryDate_D
                         FROM            Firearm INNER JOIN
                         Acquisition ON Firearm.Serial_Number = Acquisition.Serial_Number INNER JOIN
                         FFL ON Acquisition.FFL_ID = FFL.FFL_ID INNER JOIN
                         Disposition ON Firearm.Serial_Number = Disposition.Serial_Number AND FFL.FFL_ID = Disposition.FFL_ID
                         WHERE FireArm.Serial_Number = '" + textBox1.Text + "%'", con);
           
            DataSet dt = new DataSet();
            sda.Fill(dt);
            dataGridView1.DataSource = dt;


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

Я пробовал использовать меньший запрос, но получаю те же результаты, ничего не передавая в datatable. Кто-то сказал мне, что мне нужно открыть и закрыть соединение, поэтому я добавил мошенничество.Open(); и con.Close(); но это не имело никакого значения, поэтому я удалил их. Я использую ту же строку подключения в другой форме, и она работает там, поэтому я не могу себе представить, что это проблема.

Есть идеи?

3 Ответов

Рейтинг:
5

Ali Javani

всегда в любом коде подключения sql
откройте и закройте экземпляр класса SqlConnection . это очень важно

Dave Kreskowiak

Это не так важно, как вы думаете. Есть вещи, которые откроют и закроют соединение для вас, например DataAdapter или TableAdapter.

Рейтинг:
20

Patrice T 

SqlDataAdapter sda = new SqlDataAdapter(@"SELECT Firearm.Serial_Number, Firearm.Manufacturer, Firearm.Country_of_Mfg, Firearm.Model, Firearm.Type, Firearm.[Action], Firearm.Gage_Caliber, Firearm.Status, Firearm.Acquired, Firearm.Disposed, Acquisition.Acq_ID,
             ...
             WHERE FireArm.Serial_Number = '" + textBox1.Text + "%'", con);

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

Dr. Johnny Fever

Значит, я должен использовать параметры, чтобы предотвратить это?

Patrice T

да

Рейтинг:
17

OriginalGriff

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

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

Поэтому первое, что вам нужно сделать, это пройти через все ваше приложение и исправить это - оставить одну конкатенацию, и вам лучше часто делать резервную копию!

Когда вы это сделаете, посмотрите на свой запрос, а когда вы не получите нужные данные, начните с предложения WHERE, поскольку именно оно определяет, какие данные будут возвращены:
... WHERE FireArm.Serial_Number = '" + textBox1.Text + "%'"
Equals-это точное совпадение; он не признает подстановочные знаки. Вы это имели в виду:
... WHERE FireArm.Serial_Number LIKE @SerNo + '%'"