Forma correcta de utilizar .Net DateTime en cadena SQL parametrizada

Question

Existen varias preguntas sobre el uso de .Net DateTime en las sentencias Sql, pero ninguna de ellas resolvió mi problema.

estoy usando el siguiente código para consultar una base de datos Oracle:

private DataTable QueryByIdAndDate(string id, DateTime fdate, DateTime tdate) { 
    string query = "SELECT * FROM table WHERE ID = :id AND DATE_TIME BETWEEN :from AND :to" 
    DbCommand cmd = db.CreateCommand(); 
    cmd.CommandType = CommandType.Text; 
    cmd.CommandText = query; 

    DbParameter fromDate = CreateDateParameter(cmd, fdate); 
    fromDate.ParameterName = "from"; 
    DbParameter toDate = CreateDateParameter(cmd, tdate); 
    toDate.ParameterName = "to"; 
    DbParameter idParam = CreateStringParameter(cmd, id); 
    idParam.ParameterName = "id"; 

    cmd.Parameters.AddRange(new DbParameter[] { fromDate, toDate, idParam }); 
    return db.ExecuteQuery(cmd); 
} 

private DbParameter CreateDateParameter(DbCommand cmd, DateTime date) { 
    DbParameter param = cmd.CreateParameter(); 
    param.DbType = DbType.DateTime; 
    param.Direction = ParameterDirection.Input; 
    param.Value = date; 
    return param; 
} 

Pero no funciona correctamente. Al tratar el código a cabo de esta manera:

DataTable result = QueryByIdAndDate("12345", DateTime.Now, DateTime.Now.AddDays(1)); 

Se da el siguiente error: ORA-01847 : día del mes debe estar entre 1 y último día del mes

Estoy asumiendo que tiene que ver con la forma en que DateTime está formateado, pero no sé la manera correcta de solucionarlo de manera confiable.

Answer 1

(De acuerdo con los comentarios ...)

Parece que en este caso, el orden de los parámetros asuntos ... a pesar de que usted les ha dado nombres. No hubiera esperado esto, y es el signo de un conductor de un poco roto, pero cambiando el código para:

cmd.Parameters.AddRange(new DbParameter[] { idParam, fromDate, toDate }); 

debe solucionarlo. (Esto no es necesariamente la forma en que debería estar construyendo sus parámetros, por cierto, pero eso es un tanto irrelevante aquí.)

Haz no inicio especificando los valores de fecha/hora como cadenas. Es una muy mala idea para introducir más conversiones de cadenas de las que necesita.

Answer 2

convertir el valor DateTime a una cadena de fecha con formato:

private DbParameter CreateDateParameter(DbCommand cmd, DateTime date) 
{ 
    DbParameter param = cmd.CreateParameter(); 
    param.DbType = DbType.DateTime; 
    param.Direction = ParameterDirection.Input; 
    param.Value = date.ToString("yyyy-MM-dd HH:mm:ss", CultureInfo.InvariantCulture); 
    return param; 
} 

A pesar de que va a enviar un valor de cadena, el DbType está siendo ajustado a DateTime, por lo que el valor debe se convierten correctamente.

Answer 3

Oracle espera las fechas formateadas como 'DD-Mon-YYYY', por lo que básicamente necesita formatear el valor de su fecha y asignarlo al valor del parámetro.

Editar: Me he dado cuenta de que las versiones recientes de ODP parecen manejar mejor los parámetros de fecha y hora. Por ejemplo, esto funciona bien cuando hay una restricción en una cláusula WHERE en una columna de fecha/hora:

DateTime dt = new DateTime(2012, 5, 21); 
cmd.Parameters.Add("some_date_param", dt); 

Ejecución de una consulta de esta manera con la última versión del PAO funciona bien. Pero tengo un montón de código donde los valores de fecha/hora deben pasarse como cadenas formateadas para que Oracle los acepte.

Answer 4

Para aquellos que podrían perder, ver el comentario de @kprobst bajo la respuesta aceptada por @JonSkeet:

No importa lo bien que formatea los valores de los parámetros, etc, si usted está tratando de utilizar nombre vinculante sin establecer la propiedad BindByName en el Command.

(Toda la API del proveedor de Oracle es un montón de basura en mi humilde opinión).