-1

Este es el código para insertar información a la base de datos:

MySqlCommand cmd = new MySqlCommand("INSERT INTO scann (fecha, n-pedido, cliente, pedido, condiciones, precio) VALUES ('" + fecha + "','" + n + "','" + cli + "','" + ped + "','" + condiciones + "','" + pesos + "')", MyCon);
MyCon.Open();
cmd.ExecuteNonQuery();
MessageBox.Show("hecho");
MyCon.Close();

Este es el error que me tira. gracias

5
  • te recomiendo usar procedimientos almacenado y parámetros tal como indica Cristian Torres, ya que el código que estas usando es propenso a inyección de SQL. saludos. Commented el 24 may. 2019 a las 18:01
  • Te recomendaria que utilices una capa de mapeo, para las interacciones con tu modelos de datos, ejecutar consultas de esa forma es totalmente inseguro. Podrias probar con LINQ to SQL Classes es muy intuitivo.
    – M. Gress
    Commented el 24 may. 2019 a las 18:24
  • Bienvenido a Stack Overflow en español! Por favor, no uses imágenes salvo que sea absolutamente necesario. Mira por favor aca. Los errores y tu codigo deben ir como texto, siempre.
    – gbianchi
    Commented el 24 may. 2019 a las 19:26
  • 1
    Bienvenido a Stack Overflow en español! Mira Cómo preguntar para que tu pregunta sea mejor recibida. También, aprovecha y haz el recorrido de bienvenida para entender mejor cómo funcionamos y de paso obtener tu primera medalla!
    – gbianchi
    Commented el 24 may. 2019 a las 19:26
  • toma la consulta que te devuelve , pegala en mysql workbench y ejecutala, alli sabras que te falta Commented el 24 may. 2019 a las 19:27

2 respuestas 2

1

Encierra el nombre de la columna n-pedido con `back-ticks`

Así: `n-pedido`

3
  • 2
    Esto parece relamente un comentario mas que una respuesta.Podrias indicar porque es la respuesta y dar un poco de contexto?
    – gbianchi
    Commented el 24 may. 2019 a las 19:28
  • 1
    En realidad sí es una respuesta. No se si puedes leer el error que se plantea en la imagen, pues menciona que el error esta en el campo en cuestión, y el requiere poder hacer la consulta. Osea una solución.
    – jolsalazar
    Commented el 24 may. 2019 a las 20:13
  • 1
    Por eso pregunte si podias agregarle un poco de contexto. Pq si lo unico que le falta es agregar comillas pq los nombre que contienen algunos caracteres deben ir entre comillas, estaria bueno aclararlo. Si no, la cerramos como error ripografico porque faltan comillas ;)
    – gbianchi
    Commented el 24 may. 2019 a las 20:17
0

Usa parámetros para hacer el insert.

Arias algo asi

using (MySqlConnection conn = new MySqlConnection("Tu Cadena de Conexión" )
{
    string query = "INSERT INTO NombreTabla(Nombrecampo1,Nombrecampo2) VALUES(@campo1,@campo2)";
    MySqlCommand cmd = new MySqlCommand(query, conn);
    cmd.Parameters.AddWithValue("@campo1",Textbox1.Text);
    cmd.Parameters.AddWithValue("@campo2", Textbox2.Text);
    conn.Open();
    try
    {
        cmd.ExecuteNonQuery();
        MessageBox.Show("Registro Ingresado con Exito...");
    }
    catch (Exception)
    {
        MessageBox.Show("Error no se Pudo Ingresar el Registro...");
    }
}
1
  • Esta respuesta esta correcta.. sin embargo, aprovechando que AP escribio lo que hizo hasta ahora, no hubiera sido mejor que el ejemplo aprovechara lo que hizo AP? ;)
    – gbianchi
    Commented el 24 may. 2019 a las 19:28

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.