Удаление строки из mysql в datagridview vb

Question

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

Вот код, я знаю его неопрятный и есть неиспользованные вещи, но он не должен влиять на него.

Private Sub Button2_Click_1(sender As Object, e As EventArgs) Handles Button2.Click 
    Try 
     MySqlConnection.Open() 
    Catch ex As Exception 

    End Try 
    If Me.DataGridView1.Rows.Count > 0 Then 
     If Me.DataGridView1.SelectedRows.Count > 0 Then 
      For Each row As DataGridViewRow In DataGridView1.Rows 
       Dim selUser As String = Me.DataGridView1.CurrentRow.Index 
       Dim SqlQuery As String = "DELETE FROM Report WHERE user=" & row.DataBoundItem(1).ToString 
       Dim cmd As New MySqlCommand(SqlQuery, MySqlConnection) 
       cmd.ExecuteNonQuery() 
       Dim da As New MySqlDataAdapter("select * from Report", MySqlConnection) 
       Dim ds As New DataSet 
       da.Fill(ds) 
       DataGridView1.DataSource = ds.Tables(0) 
      Next 
     End If 

    End If 

End Sub 

EDIT: Исправлено это сейчас, спасибо всем, кто ответил.

Answer 1

Основываясь на полученной ошибке, я предполагаю, что это проблема объединения имени столбца со значением. Таким образом, вы можете избежать конкатенации и sql-инъекции (чего не избежать в вашем коде) с использованием запроса на основе параметров.

Ваш код будет выглядеть так:

Dim SqlQuery As String = "DELETE FROM Report WHERE user= @user" 
Dim cmd As New MySqlCommand(SqlQuery, MySqlConnection) 
cmd.Parameters.Add(New SqlParameter("@user", row.DataBoundItem(1).ToString)) // with SqlParameter, you can pass value 

Answer 2

Quote ваш идентификатор пользователя, вы получили его, как

... WHERE user=asd 

, который прямо-таки ошибка SQL.

Вам нужно быть

... WHERE user='asd' 

Итак, доу как вы конкатенации:

Dim SqlQuery As String = "DELETE FROM Report WHERE user='" & row.DataBoundItem(1).ToString & "'" 

или использовать параметризацию, как это было предложено Акаши KC.