2

En ¿Cómo prevenir SQL inyección en Visual Basic .Net? consulté acerca de como prevenir ataques por inyección SQL.

Ahora estoy intentando grabar en la base de datos usando parámetros pero me da un error diciendo que no indiqué la cantidad suficiente de parámetros.

Mi código actual es:

Dim sql As String
Dim conn = New SQLiteConnection("Data Source=BaseDatos.sqlite;Version=3")

Dim cmdGuardar As SQLiteCommand = New SQLiteCommand(sql, conn)

Try
Using (conn)

conn.Open()
sql = "INSERT INTO actos (parteA, tipo, parteB) VALUES(@param1, @param2, @param3)"
'Línea que debería usar pero me da un error
'SQLiteCommand cmd = New SQLiteCommand(sql, conn)
'Así que pruebo dar la misma instrucción en dos líneas diferentes
cmdGuardar.CommandText = sql
cmdGuardar.Connection = conn
cmdGuardar.Parameters.AddWithValue("@param1", cboParteA.Text)
cmdGuardar.Parameters.AddWithValue("@param2", cboTipo.Text)
cmdGuardar.Parameters.AddWithValue("@param2", cboParteB.Text)

cmdGuardar.ExecuteNonQuery()

conn.Close()

End Using

Catch ex As Exception
    MsgBox(ex.ToString())
End Try

1 respuesta 1

1

Tu codigo deberia lucir como esto

Try
    Using  conn As New SQLiteConnection("connstring")

        conn.Open()
        sql = "INSERT INTO actos (parteA, tipo, parteB) VALUES(@param1, @param2, @param3)"
        Dim cmdGuardar As SQLiteCommand = New SQLiteCommand(sql, conn)
        cmdGuardar.Parameters.AddWithValue("@param1", cboParteA.Text)
        cmdGuardar.Parameters.AddWithValue("@param2", cboTipo.Text)
        cmdGuardar.Parameters.AddWithValue("@param2", cboParteB.Text)

        cmdGuardar.ExecuteNonQuery()

        conn.Close()

    End Using

Catch ex As Exception
    MessageBox.Show(ex.Message)
End Try         

la idea es definir la conexion y el command lo mas local posible

2
  • Correcto!! Incluso yo en realidad definía las variables por encima del cabezal del Form para que el ámbito de las mismas fuera todo el formulario pero por algún motivo no las tomaba. La solución final que tomé fue crear variables conn2 y sql2 dentro del Sub del botón y ahora funciona. Muchas gracias por tu ayuda, me sirve mucho para aprender. Ahora con un ejemplo funcional puedo seguir adelante. Saludos
    – fedeteka
    Commented el 24 jun. 2016 a las 18:14
  • 1
    Te recomendaría que uses Procedimientos Almacenados desde tu base de datos, y luego solo usarlos desde Vb, es más ordenado y seguro con lo Inyeccion Sql. Te dejo un Link: goo.gl/9OVrmt Commented el 7 jul. 2016 a las 15:29

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.