horngwoei Ответов: 2

Синтаксическая ошибка в операторе INSERT into

Привет, я новичок в кодировании, получил "синтаксическую ошибку в инструкции INSERT INTO" после того, как нажал кнопку.Пожалуйста, дайте мне несколько советов и спасибо за помощь !

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

Private Sub btnAdd_Click(sender As Object, e As EventArgs) Handles btnAdd.Click
        myconnection.ConnectionString =
        "Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\Users\Horng Woei\Documents\Klinik OCEANA.mdb"
        Dim Reader As OleDbDataReader
        If (txtName.Text = "" AndAlso txtAge.Text = "" AndAlso cboGender.Text = "" AndAlso
            cboNationality.Text = "" AndAlso txtContact.Text = "" AndAlso txtDOR.Text = "" AndAlso txtIC.Text = "" AndAlso
            cboBloodType.Text = "" AndAlso txtWeight.Text = "" AndAlso txtHeight.Text = "" AndAlso txtAllergies.Text = "") And (cboBloodType.SelectedIndex = 0 AndAlso
            cboGender.SelectedIndex = 0 AndAlso cboStatus.SelectedIndex = 0 AndAlso cboNationality.SelectedIndex = 0) Then
            MsgBox("Please fill in the form completely.")


        Else
            Try
                myconnection.Open()
                Dim query As String
                query = "Insert into [PatientData] ([PatientName],[Age],[Gender],[Marital Status][Nationality],[ContactNo],[DOR],[IC Number],[Blood Type],[Weight(kg)],[Height(cm)],[Allergies])
                Values ('" & txtName.Text & "','" & txtAge.Text & "','" & cboGender.Text & "','" & cboStatus.Text & "','" & cboNationality.Text & "','" & txtContact.Text & "','" & txtDOR.Text & "','" & txtIC.Text & "','" & cboBloodType.Text & "','" & txtWeight.Text & "','" & txtHeight.Text & "','" & txtAllergies.Text & "')"
                cmd = New OleDbCommand(query, myconnection)
                Reader = cmd.ExecuteReader

                MessageBox.Show("You have stored data successfully!")
                myconnection.Close()
            Catch ex As Exception
                MsgBox(ex.Message)
            Finally
                myconnection.Dispose()
                txtName.Clear()
                txtAge.Clear()
                cboGender.ResetText()
                cboStatus.ResetText()
                cboNationality.ResetText()
                txtContact.Clear()
                txtDOR.Clear()
                txtIC.Clear()
                cboBloodType.ResetText()
                txtWeight.Clear()
                txtHeight.Clear()
                txtAllergies.Clear()
                refreshdata("")



            End Try
        End If

    End Sub

2 Ответов

Рейтинг:
6

Patrice T

Цитата:
Синтаксическая ошибка в операторе INSERT into

Из-за того, как вы строите команду SQL, невозможно помочь вам, потому что пользовательский ввод является частью команды, и у нас нет способа узнать, что такое настоящая команда.

Временное решение: отладчик позволит вам запускать код шаг за шагом и отображать/печатать реальный запрос. Вы поймете проблему для самого пользовательского ввода.

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

Рейтинг:
18

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

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

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