Ta como te decía en los comentarios, elegiste un camino sumamente complejo para resolver este problema. Mi concejo es que no uses Expression
.
Pero para responder tu pregunta, tienes un problema en tu función GetFilterExpression
, esta función solo funciona para un caso y es cuando tienes es forma:
GetFilterExpression(x => x.PropInt == 1);
- A la izquierda una propiedad a la cual quieres rescatar el nombre
- La operación es un igual lógico
- A la derecha una constante
Cualquier otro caso no funciona
No funciona para cualquier otra comparación que igualmente retorna un booleano, porque tu siempre conviertes a =
GetFilterExpression(x => x.PropInt != 1); // retorna: PropInt = 1
GetFilterExpression(x => x.PropInt > 1); // retorna: PropInt = 1
GetFilterExpression(x => x.PropInt < 1); // retorna: PropInt = 1
No funciona con algo que no sea un número
GetFilterExpression(x => x.PropString = "unstring"); // retorna: PropInt = unstring
// Deberia retornar: PropInt = 'unstring' con comillas
No funciona con null
GetFilterExpression(x => x.PropString = null); // retorna: PropInt =
// Deberia retornar: PropInt is null
No funciona con una expresión compuesta
GetFilterExpression(x => x.PropString = null && x.PropInt = 1) ; // Se cae el programa
Y así puedo seguir señalando casos todo el día, porque básicamente lo que se requiere es traducir un lenguaje C# a SQL que es bastante trabajo
Pero el problema puntual de tu código es que tampoco funciona para variables (solo funciona para constantes)
Es decir esto no funciona:
_administradorDal.Get((e) => e.Cedula == cedulon)
_docenteDal.Get((e) => e.Cedula == user.Cedula)
_estudianteDal.Get((e) => e.Cedula == user.Cedula)
_operadorDal.Get((e) => e.Cedula == user.Cedula)
Porque lo de la derecha sería MemberExpression
no un ConstantExpression
como tu lo tienes y para obtener el valor debes compilarlo, es decir debería ser algo así:
private string GetFilterExpression(Expression<Func<T, bool>> filter)
{
var binaryExpression = (BinaryExpression)filter.Body;
var left = binaryExpression.Left as MemberExpression;
var right = binaryExpression.Right as MemberExpression;
if (left != null && right != null)
{
var valor = Expression.Lambda(right).Compile().DynamicInvoke();
var fieldName = left.Member.Name;
var value = valor;
return $"{fieldName} = {value}";
}
throw new ArgumentException("Expresión de filtro no válida.");
}
Acá te dejo un ejemplo que soluciona varios de los problemas, pero tienes que terminarlo
internal class Program
{
static void Main(string[] args)
{
int a = 2;
Console.WriteLine(GetFilterExpression<A>(x => x.PropInt == a));
Console.WriteLine(GetFilterExpression<A>(x => x.PropInt == 2));
Console.WriteLine(GetFilterExpression<A>(x => a != x.PropInt));
Console.WriteLine(GetFilterExpression<A>(x => (x.PropInt == 2 || x.PropString=="algo") && x.OtraProp==false));
Console.WriteLine(GetFilterExpression<A>(x => !x.OtraProp));
Console.WriteLine(GetFilterExpression<A>(x => x.PropString == null));
}
private static string GetFilterExpression<T>(Expression<Func<T, bool>> filter)
=> GetFilterExpression(filter.Body)
.Replace("= null", "is null") //reemplazamos para estos casos especiales
.Replace("<> null", "is not null");
//recorre el árbol recursivamente
private static string GetFilterExpression(Expression exp)
{
if (exp is BinaryExpression bexp) return GetFilterExpression(bexp);
if (exp is UnaryExpression uexp) return GetFilterExpression(uexp);
if (exp is MemberExpression mexp)
{
if(mexp?.Expression?.NodeType == ExpressionType.Parameter) return mexp.Member.Name; //retorna el nombre de la propiedad
//retorna el valor de una variable
var valor = Expression.Lambda(mexp).Compile().DynamicInvoke();
return GetConstant(valor);
}
return GetFilterMember(exp);
}
//Resuelve para los BinaryExpression
private static string GetFilterExpression(BinaryExpression exp)
{
var left = GetFilterExpression(exp.Left);
var right = GetFilterExpression(exp.Right);
var op = GetFilterMember(exp);
return $"({left} {op} {right})"; ;
}
//Resuelve para los UnaryExpression
private static string GetFilterExpression(UnaryExpression exp)
{
var right = GetFilterExpression(exp.Operand);
var op = GetFilterMember(exp);
return $"({op} {right})"; ;
}
//Resuelve para los otros casos, principalmente las operaciones
private static string GetFilterMember(Expression ex)
=> ex.NodeType switch
{
ExpressionType.AndAlso => "and",
ExpressionType.OrElse => "or",
ExpressionType.Equal => "=",
ExpressionType.NotEqual => "<>",
ExpressionType.Not => "Not",
//HAY QUE SEGUIR AGREGANDO ExpressionType a la lista, para casos de mayor, menor, mayor o igual, etc
ExpressionType.Constant => GetConstant(((ConstantExpression)ex).Value),
_ => "" ,
};
//Coloca las comillas cuando es necesario y pone null si es el caso
private static string GetConstant(object? Value)
{
if(Value is null) return "null";
if(Value is string or DateTime) return $"'{Value}'";
return Value.ToString();
}
}
//clase de pruebas
public class A
{
public string PropString { get; set; }
public int PropInt { get; set; }
public bool OtraProp { get; set; }
}
Resultado