Was ist falsch mit meinem Einfügen Befehl

Question

Ich versuche, Daten in meine Datenbank mit einem DataGridView in C# einzufügen. Allerdings, wenn ich klicken Sie auf Speichern Sie die folgende Fehlermeldung angezeigt:

System.Data.OleDb.OleDbException was unhandled

HResult = -2147217900
Message = Syntax error in INSERT INTO statement.
Source = Microsoft Office Access Database Engine
ErrorCode = -2147217900

Hier ist der Code, den ich habe:

private void save_btn_Click(object sender, EventArgs e) 
{ 
    OleDbConnection con = new OleDbConnection(@"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=Stock.accdb"); 
    con.Open(); 

    for (int i = 0; i < dataGridView_insert.Rows.Count; i++) 
    { 
     OleDbCommand cmd = new OleDbCommand("INSERT INTO product(OV,Reference,Cod_Client,Client,Qtd,Type_product,Posicion_product,) VALUES ('" + dataGridView_insert.Rows[i].Cells["OV"].Value + "','" + dataGridView_insert.Rows[i].Cells["Reference"].Value + "','" + dataGridView_insert.Rows[i].Cells["Cod_Client"].Value + "','" + dataGridView_insert.Rows[i].Cells["Client"].Value + "','" + dataGridView_insert.Rows[i].Cells["Qtd"].Value + "','" + dataGridView_insert.Rows[i].Cells["Type_product"].Value + "','" + dataGridView_insert.Rows[i].Cells["Posicion_product"].Value + " ' ", con);  
     cmd.ExecuteNonQuery();  
    } 

    con.Close(); 
} 

Was ist falsch?

Answer 1

Sie haben eine streunende , nach Posicion_product und Sie haben auch die schließende Klammer von VALUES in Ihrer Insert-Anweisung verpasst. Entfernen Sie es. Auch sollten Sie immer parameterized queries verwenden SQL Injection zu vermeiden:

OleDbCommand cmd = new OleDbCommand("INSERT INTO product(OV,Reference,Cod_Client,Client,Qtd,Type_product,Posicion_product) VALUES (@a,@b,@c,@d,@e,@f,@g)", con); 
cmd.Parameters.AddWithValue("@a", dataGridView_insert.Rows[i].Cells["OV"].Value); 
cmd.Parameters.AddWithValue("@b", dataGridView_insert.Rows[i].Cells["Reference"].Value); 
cmd.Parameters.AddWithValue("@c", dataGridView_insert.Rows[i].Cells["Cod_Client"].Value); 
//And continue for other parameters 

Obwohl die Art direkt angeben und verwenden Sie die Value Eigenschaft ist besser als AddWithValue:

cmd.Parameters.Add("@a", SqlDbType.VarChar).Value = dataGridView_insert.Rows[i].Cells["OV"].Value;