Member 3892343 Ответов: 2

SQL-строку не согласиться со списком выбранного текста


<pre> Private Sub Button3_Click(sender As Object, e As EventArgs) Handles Button3.Click
        Dim ConnectionString As String
        Dim binding As New BindingSource
        
        ConnectionString = ("Integrated Security = true;Initial catalog = " & ComboBox2.SelectedText.ToString & ";Data Source = " & ComboBox1.SelectedText & "; ")
        Dim SQL As String = "SELECT * FROM Table_1"
        Dim conn As SqlConnection = New SqlConnection(ConnectionString)

        ' open the connection


        'Create a SqlDataAdapter object
        Dim adapter1 As SqlDataAdapter = New SqlDataAdapter(SQL, conn)

        ' Call DataAdapter's Fill method to fill data from the
        ' Data Adapter to the DataSet
        Dim ds As New DataSet
        'ds.Tables.Add("Table_1")
        conn.Open()
        adapter1.Fill(ds, "Table_1")
        conn.Close()
        binding.DataSource = ds
        binding.DataMember = "Table_1"
        DataGridView1.DataSource = binding

    End Sub




приведенный выше код не работает ошибка :дополнительная информация: недопустимое имя объекта "Table_1". когда я набираю начальный каталог вот так

pre>Private Sub Button3_Click(sender As Object, e As EventArgs) Handles Button3.Click
        Dim ConnectionString As String
        Dim binding As New BindingSource
        ConnectionString = ("Integrated Security = true;Initial catalog = logistiki;Data Source = " & ComboBox1.SelectedText & "; ")
        Dim SQL As String = "SELECT * FROM Table_1"
        Dim conn As SqlConnection = New SqlConnection(ConnectionString)

        ' open the connection


        'Create a SqlDataAdapter object
        Dim adapter1 As SqlDataAdapter = New SqlDataAdapter(SQL, conn)

        ' Call DataAdapter's Fill method to fill data from the
        ' Data Adapter to the DataSet
        Dim ds As New DataSet
        'ds.Tables.Add("Table_1")
        conn.Open()
        adapter1.Fill(ds, "Table_1")
        conn.Close()
        binding.DataSource = ds
        binding.DataMember = "Table_1"
        DataGridView1.DataSource = binding

    End Sub

он работает он принимает выделенный текст combobox1 но не combobox2

кто-нибудь поможет?

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

добавлена переменная для выделенного текста combox2 в строку sql, но она тоже не работает

2 Ответов

Рейтинг:
9

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

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

Исправьте это на протяжении всего вашего приложения, и есть вероятность, что проблема, которую вы заметили, тоже исчезнет.


Рейтинг:
19

Mehul M Thakkar

если у вас много баз данных и серверов, используйте SqlConnectionStringBuilder. Вот вам пример использование SqlConnectionStringBuilder для создания строки подключения | онлайн-компилятор C# | .NET Fiddle[^].

Если у вас их мало, я бы посоветовал сохранить в конфигурационном файле и написать switch case, чтобы решить строку подключения.