horngwoei Ответов: 2

Синтаксическая ошибка в инструкции UPDATE


привет, у меня есть "(синтаксическая ошибка в инструкции UPDATE.)", когда я обновляю запись в базе данных access с помощью VB.net код.Спасибо за помощь.

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

Private Sub btnEdit_Click(sender As Object, e As EventArgs) Handles btnEdit.Click
     myconnection.ConnectionString =
     "Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\Users\Horng Woei\Documents\Klinik OCEANA.mdb"

     myconnection.Open()
     Dim cmd As New OleDbCommand
     Dim ds As New DataSet
     da = New OleDbDataAdapter("select * from [PatientData]", myconnection)
     If da.Fill(ds) Then

         Dim query As String = "update [PatientData] set [PatientName],[Age],[Gender],[Nationality],[ContactNo],[DOR],[IC],[Blood Type],[Weight(kg)],[Height(cm)],[Allergies]
         = '" + txtName.Text + "','" + txtAge.Text + "','" + cboGender.Text + "','" + cboNationality.Text + "','" + txtContact.Text + "','" + txtDOR.Text + "','" + txtIC.Text + "','" + cboBloodType.Text + "','" + txtWeight.Text + "','" + txtHeight.Text + "','" + txtAllergies.Text + "'
         where [PatientID] = '" + txtID.Text + "'"

         cmd.Parameters.AddWithValue("@PatientName", txtName.Text)
         cmd.Parameters.AddWithValue("@Age", txtAge.Text)
         cmd.Parameters.AddWithValue("@Gender", cboGender.Text)
         cmd.Parameters.AddWithValue("@Nationality", cboNationality.Text)
         cmd.Parameters.AddWithValue("@ContactNo", txtContact.Text)
         cmd.Parameters.AddWithValue("@DOR", txtDOR.Text)
         cmd.Parameters.AddWithValue("@IC", txtIC.Text)
         cmd.Parameters.AddWithValue("@Blood Type", cboBloodType.Text)
         cmd.Parameters.AddWithValue("@Weight(kg)", txtWeight.Text)
         cmd.Parameters.AddWithValue("@Height(cm)", txtHeight.Text)
         cmd.Parameters.AddWithValue("@Allergies", txtAllergies.Text)
         cmd = New OleDbCommand(query, myconnection)

         cmd.ExecuteNonQuery()
         cmd.Dispose()
         MsgBox("Data Updated Successfully!")
         myconnection.Close()
         filterrecords("")
     End If

 End Sub

2 Ответов

Рейтинг:
6

Patrice T

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

Dim query As String = "update [PatientData] set [PatientName],[Age],[Gender],[Nationality],[ContactNo],[DOR],[IC],[Blood Type],[Weight(kg)],[Height(cm)],[Allergies]
         = '" + txtName.Text + "','" + txtAge.Text + "','" + cboGender.Text + "','" + cboNationality.Text + "','" + txtContact.Text + "','" + txtDOR.Text + "','" + txtIC.Text + "','" + cboBloodType.Text + "','" + txtWeight.Text + "','" + txtHeight.Text + "','" + txtAllergies.Text + "'
         where [PatientID] = '" + txtID.Text + "'"


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


Рейтинг:
13

OriginalGriff

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

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

2) это не синтаксис обновления или даже близко к нему. Инструкция UPDATE выглядит следующим образом:
UPDATE MyTable SET MyColumn1 = @ParameterValueForColumn1, MyColumn2 = @ParameterValueForColumn2 WHERE ...