SQL Поиск оптимизация

[nelo_001]

Добрый день , помогите доработать код , чтобы выводил результат в гривью.

SqlConnection con = new SqlConnection("Data Source=(LocalDB)\\MSSQLLocalDB;AttachDbFilename=|DataDirectory|\\Database1.mdf;Integrated Security=True");
SqlDataAdapter da = new SqlDataAdapter();
DataSet ds = new DataSet();

Кнопка1

    da.SelectCommand = new SqlCommand("SELECT * FROM [FROMBD] WHERE [Las] ='" + toolStripTextBox1.Text + "'", con);

ГридВью как был каменным так камнем и остался ниче не поменялось

[Alex11223]

Причем тут оптимизация? Оптимизация это когда все работает, но надо эффективнее.


Не увидел ничего связанного с GridView в этом коде.

[nelo_001]

еще раз доделал.
//

  DataTable dt = new DataTable();

            da.SelectCommand = new SqlCommand("SELECT * FROM [FROMBD] WHERE [Las] like" + toolStripTextBox1.Text,con);
            da.Fill(dt);
            tableDataGridView.DataSource = dt;

Ругается на 2 строчку:

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

Дополнительные сведения: An expression of non-boolean type specified in a context where a condition is expected, near 'likeячс'.

[Alex11223]

Ну видимо в SQL запросе ошибка, наверно пробел после like пропущен, и может быть кавычки.

Вообще не надо так писать запросы конкатенацией строк, используйте prepared statement
https://en.wikipedia.org/wiki/Prepar...t#C.23_ADO.NET

[nelo_001]

Сообщение от Alex11223

Ну видимо в SQL запросе ошибка, наверно пробел после like пропущен, и может быть кавычки.

Вообще не надо так писать запросы конкатенацией строк, используйте prepared statement
https://en.wikipedia.org/wiki/Prepar...t#C.23_ADO.NET

мне кажется так гораздо проще ?
// ещераз изменил код

            da.SelectCommand = new SqlCommand("SELECT * FROM [SuperBD] WHERE [Las] = '" + toolStripTextBox1.Text + "'", con);
         
            
            da.Fill(dt);
            tableDataGridView.DataSource = dt;

теперь оно не дает ошибки а дает пустой гридвью

// насчет Prepared statement, мастера ado.net разве не пишут так как я ?

[come-on]

Так пишут только подмастерья, мастера все сугубо на параметрах и хранимых процедурах

[Alex11223]

Ну по ссылке выше ж написано чем оно лучше. Например, про SQL инъекции злых хацкеров.
Да и просто удобнее так, сложнее запутаться в кавычках и т.п., конвертации типов к нужному виду.

А про пустой, ну видимо ничего нет по этому запросу. Попробуйте выполнить его в СУБД без своей программы.

[nelo_001]

Ясно ! Нашел в чем ошибка ! все работает ! очень смешная ошибка , проекту без году 5 минут , а уже имеет 2 бд ! Я сам в душе не чаю откуда 2 бд , но да ладно ...
// уважаемый мастер , можете кусочек кода, или статью на более широкое обьяснения про prepared statement.
Снимок.PNG

[come-on]

Это на картинке 2 бд? это 2 балицы в одной бд

[nelo_001]

Сообщение от come-on

Это на картинке 2 бд? это 2 балицы в одной бд

да .... издержки времени .....