tengo un problema con una expresión lambda básicamente yo tengo un crud de Dapper que en el metodo get recibe una expresion lambda y esta la envio a otro metodo que crea la sentencia que necesita where (sql), y cuando la obtiene la añade a la sentencia inicial y la ejecuta, funcionaba todo bien hasta que no se porque motivo hay una de las peticiones que se envia de forma incorrecta, como que se envia toda la anidacion y cuando este metodo va construir where lo interpreta como que es nullo y procede a fallecer
private string GetFilterExpression(Expression<Func<T, bool>> filter)
{
var binaryExpression = (BinaryExpression)filter.Body;
var left = binaryExpression.Left as MemberExpression;
var right = binaryExpression.Right as ConstantExpression;
if (left != null && right != null)
{
var fieldName = left.Member.Name;
var value = right.Value;
// Devuelve la cláusula WHERE con el filtro.
return $"{fieldName} = {value}";
}
throw new ArgumentException("Expresión de filtro no válida.");
}
Aca dejo el metodo get
public T Get(Expression<Func<T, bool>> filter)
{
string tableName= typeof(T).Name;
if (GetFilterExpression(filter) != null)
{
string sql = $"SELECT * FROM {tableName} WHERE {GetFilterExpression(filter)}";
String expr= GetFilterExpression(filter);
System.Console.WriteLine("Datos de la consulta");
System.Console.WriteLine(expr);
System.Console.WriteLine(sql);
var result= _connection.QueryFirstOrDefault<T>(sql);
return result;
}
else
{
string sql = $"SELECT * FROM {tableName}";
return _connection.QueryFirstOrDefault<T>(sql);
}
}
y aca es donde empieza a fallar
public virtual List<Usuario> GetAll(Expression<Func<Usuario, bool>>? filter = null)
{
List<Usuario> users = base.GetAll(filter);
int cedulon;
foreach (var user in users)
{
if (_administradorDal.Get((e) => e.Cedula == cedulon) != null){
user.Rol = "Administrador";
}
else if (_docenteDal.Get((e) => e.Cedula == user.Cedula) != null)
{
user.Rol = "Docente";
}
else if (_estudianteDal.Get((e) => e.Cedula == user.Cedula) != null)
{
user.Rol = "Estudiante";
}
else if (_operadorDal.Get((e) => e.Cedula == user.Cedula) != null)
{
user.Rol = "Operador";
}
}
return users;
}
la expresion que llega a GetFilterExpressions es:
{e => (e.Cedula == value(DpUserDal+<>c__DisplayClass7_0).user.Cedula)}
Ya probe no enviar el user.cedula, tipo guardarlo en un int previamente
GetFilterExpression
solo funciona para un solo caso (cuando es=
), no funciona para los distintos, ni para expresiones anidadas (and
yor
) ni menos con paréntesis. Un árbol de expresiones se recorre justamente como árbol para ir armando la sentencia, si tu no necesitas eso mejor pasar 2 argumentos "normales" en la función y ya. Ahora quizás este es solo un código de ejemplo y tu tienes lo que te digo en el originalGet
cuando no hay filtro retorna el primer elemento al azar, tampoco le veo sentido. Semánticamente elGet
debería retornar uno, pero para eso hay que pasar su Id siempre, no le veo sentido a retornar cualquiera de la tabla