Skip to main content
se añadió 1 carácter en el cuerpo
Origen Enlace
Yussef
  • 23.4k
  • 4
  • 12
  • 29

TaTal como te decía en los comentarios, elegiste un camino sumamente complejo para resolver este problema. Mi concejo es que no uses Expression.

Ta como te decía en los comentarios, elegiste un camino sumamente complejo para resolver este problema. Mi concejo es que no uses Expression.

Tal como te decía en los comentarios, elegiste un camino sumamente complejo para resolver este problema. Mi concejo es que no uses Expression.

Origen Enlace
Yussef
  • 23.4k
  • 4
  • 12
  • 29

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

introducir la descripción de la imagen aquí