3

El siguiente código me da error a la hora que quiero insertar y no se porque me da error.

 conectar.Open();
 MySqlCommand add= new MySqlCommand("insert into vuelos1 values('" + FechaLlegada.Text + "," + FechaSalida.Text + ",'" + Destino1.Text + "'," + comboBox1.Text + ",'" + comboBox2.Text + "', " + textBox1.Text);

 add.ExecuteNonQuery();
 conectar.Close();
 MessageBox.Show("Registro Creado");

Tengo un campo de ID que es nulo, auto_increment y no se como ponerlo tampoco, aquí les dejo los campos que tiene mi tabla en MYSQL.

introducir la descripción de la imagen aquí

Este es el error que me sale

introducir la descripción de la imagen aquí

5
  • 1
    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! a tu string, le falta un cierre de comillas simples y cerrar los parentesis. Siempre fijate cual es el string resultante cuando sumas varios, a ver que sale..
    – gbianchi
    Commented el 20 ene. 2021 a las 14:01
  • Por favor, siempre recuerda agregar tu error... si te tira un error, pone el mensaje de error en la pregunta
    – gbianchi
    Commented el 20 ene. 2021 a las 14:02
  • @gbianchi ahí puse el error que me sale
    – Justin
    Commented el 20 ene. 2021 a las 14:08
  • Todavía me sigue dando error el código
    – Justin
    Commented el 20 ene. 2021 a las 14:10
  • 1
    En el código publicado no se ve que le asignas la conexión al MySqlCommand y a juzgar por el mensaje de error ese el problema. Commented el 20 ene. 2021 a las 14:14

1 respuesta 1

1

Las inserciones de texto o de fechas siempre deben ir rodeadas de comillas simples

insert into vuelos1 values('" + FechaLlegada.Text + "," + FechaSalida.Text + ",'" +
 Destino1.Text + "'," + comboBox1.Text + ",'" + comboBox2.Text + "', " + textBox1.Text

Cuando debería ser

insert into vuelos1 values('" + FechaLlegada.Text + "','" + FechaSalida.Text + "','" +
 Destino1.Text + "','" + comboBox1.Text + "','" + comboBox2.Text + "', " + textBox1.Text + "'"

En tu código

MySqlCommand add= new MySqlCommand("insert into vuelos1 values('" + FechaLlegada.Text + "','" + FechaSalida.Text + "','" +
     Destino1.Text + "','" + comboBox1.Text + "','" + comboBox2.Text + "', " + textBox1.Text + "'");

Estoy asumiendo que los datos que insertas se corresponden con las columnas de las tablas. Sería conveniente (siempre lo es) especificarlas:

MySqlCommand add= new MySqlCommand("insert into vuelos1 (campo1, campo2, campo3, campo4, ...) values('" + FechaLlegada.Text + "','" + FechaSalida.Text + "','" +
     Destino1.Text + "','" + comboBox1.Text + "','" + comboBox2.Text + "', " + textBox1.Text + "'");
2
  • 1
    No tengo idea de C#, pero: no se puede evitar usar concatenación? Huele a SQL injection :)
    – Alfabravo
    Commented el 20 ene. 2021 a las 15:47
  • 2
    Exactamente. Lo correcto sería utilizar Parameters.AddWithValue del MySqlCommand para definir los parámetros de entrada a la consulta. Commented el 20 ene. 2021 a las 15:55

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.