1

Quiero realizar un update en sql desde una app en c# Empleo la siguiente consulta

sqlFecha = "UPDATE FACTURA SET fecha = '" + fecha + "' where codigo = '" + Convert.ToDecimal(LabelCodFactura.Text) + "'";
                InsertRow(_connectionString, sqlFecha);

donde mi variable es:

DateTime fecha = DateTime.Now;//toma la fecha del equipo

pero me enfrento con el siguiente error

Error al convertir una cadena de caracteres en fecha y/u hora en sql server

el campo en el que quiero actualizar el la fecha es tipo datetime

5
  • 2
    La primer recomendación es que no uses cadenas concatenadas para enviar comandos SQL, es suceptible a SQL Injection y es un gran fallo de seguridad al que estás expuesto. Incluso, el uso de parámetros en tu codigo C# soluciona el problema que presentas actualmente
    – Phi
    Commented el 14 ene. 2019 a las 23:59
  • ¿De dónde sale la variable "sqlFormattedDate "?
    – fredyfx
    Commented el 15 ene. 2019 a las 2:59
  • Como te comentó @Phi, lo que debes hacer es usar consultas parametrizadas. De esa manera, lo que pasas en tu consulta es el datetime en si, y no una cadena de caracteres, y solucionaras el problema que tienes y otros muchos.
    – Pikoh
    Commented el 15 ene. 2019 a las 7:55
  • de verdad un codigo de factura decimal ? los codigo deberian ser int Commented el 15 ene. 2019 a las 9:36
  • 1
    gracias @phi por el consejo
    – Ferkin San
    Commented el 15 ene. 2019 a las 13:35

1 respuesta 1

1

No se que funcionalidad realiza InsertRow() pero sino permite parámetro deberías adaptarlo o cambiarlo, tu código debería lucir como esto:

using (SqlConnection conn = new SqlConnection("connection string"))   
{   
    string sqlFecha = "UPDATE FACTURA SET fecha = @fecha where codigo = @codigo";

    SqlCommand cmd = new SqlCommand(sqlFecha, conn);   
    cmd.Parameters.AddWithValue("@fecha", DateTime.Now);  
    cmd.Parameters.AddWithValue("@codigo", Convert.ToDecimal(LabelCodFactura.Text));    

    cmd.ExecuteNonQuery();  
}

Usa siempre parametros que respeten los tipos de datos, si vas asignar una fecha qu sea un datetime no concatnes nunca en un string el valor, si el codigo es numerico no tiene que estar entre comillas simples

1
  • 1
    Puedes usar la fecha del servidor de base de datos, poniendo de esta forma tu script sql: "UPDATE FACTURA SET fecha = getdate() where codigo = @codigo", la función getdate() devuelve la fecha hora actual del servidor. Commented el 15 ene. 2019 a las 13:33

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.