oldbexception en consulta insert

[seeweed]

Hola
al intriducir este codigo:

Dim cmd As New OleDb.OleDbCommand("Insert into classes values('" & cbokidclass.Text & "','" & txtname.Text & "','" & cbokday.Text & "','" & cbokhour.Text & "','" & cbokteacher.Text & "','" & cbokgroup.Text & "','" & cboksubject.Text & "','" & cbokroom.Text & "')", cn)
cn.Open()
cmd.ExecuteNonQuery()
cn.Close()
mostrar()
MsgBox("Done")

me presenta un error en cmd.ExecuteNonQuery()
diciendome: Error de sintaxis (falta operador) en la expresión de consulta ''11b's')'.
como podria resolver este error?

[Aquaventus]

Hola seeweed, veo que una de tus cadenas que adjuntas a la query puede contener una " ' ". Verifica que los parametros que estas pasando no contengan eso, ya que estaría afectando tu query. Saludos!.

[eperedo]

Usa parámetros, así evitas errores de sintaxis, sql inyection y tu código será más fácil de leer :)

Código c#:

"Insert Into Classes(Columna1, Columna2, Columna3) Values(@Columna1, @Columna2, @Columna3)
 
//Para darle los valores a esos parámetros usas lo sgte:
 
cmd.Parameters.AddWithValue("@Columna1", cboksubject.Text)
cmd.Parameters.AddWithValue("@Columna2", cbokroom.Text)
cmd.Parameters.AddWithValue("@Columna3", cbokroom.Text)
 
cn.Open()
cmd.ExecuteNonQuery()
cn.Close()
 

[seeweed]

Iniciado por eperedo

Usa parámetros, así evitas errores de sintaxis, sql inyection y tu código será más fácil de leer :)

Código c#:

"Insert Into Classes(Columna1, Columna2, Columna3) Values(@Columna1, @Columna2, @Columna3)
 
//Para darle los valores a esos parámetros usas lo sgte:
 
cmd.Parameters.AddWithValue("@Columna1", cboksubject.Text)
cmd.Parameters.AddWithValue("@Columna2", cbokroom.Text)
cmd.Parameters.AddWithValue("@Columna3", cbokroom.Text)
 
cn.Open()
cmd.ExecuteNonQuery()
cn.Close()
 

Gracias por la ayuda, pero disculpa, tengo una duda:
¿Qué tengo que reemplazar por @Columna1 ,@Columna2 no entiendo muy bien que significa ese arroba

[Aquaventus]

Si verás en esta linea :

Código vb:

"Insert Into Classes(Columna1, Columna2, Columna3) Values(@Columna1, @Columna2, @Columna3)
 

es insertar valores pero con variables, a estas variables les setearás el contenido que és :

Código vb:

cmd.Parameters.AddWithValue("@Columna1", cboksubject.Text)
cmd.Parameters.AddWithValue("@Columna2", cbokroom.Text)
cmd.Parameters.AddWithValue("@Columna3", cbokroom2.Text)
 

si gusta cambiale de nombre a @columna1,@columna2... pero con el arroba(@) al inicio

[seeweed]

gracias, aquaventus, te pregunto, con este método no tendré entonces el oledb.execption descrito anteriormente?

[eperedo]

Nop, .Net se encargará de generar el script sql y correrlo cuando llames a cmd.ExecuteNonQuery() :)

[Aquaventus]

Probalo y nos cuentas! , porque en la anterior tu definías la query y aquí la amolda.