Исключение при использовании методов выражения OrElse и AndAlso - PullRequest
Новая
       22

Исключение при использовании методов выражения OrElse и AndAlso

12 голосов
/ 10 февраля 2012

Я пытаюсь построить дерево выражений программно.

У меня на входе есть список классов условий, которые имеют следующую форму: 

public class Filter
{
    public string field { get; set; }
    public string operator { get; set; }
    public string value { get; set; }
}

Когда я строю Expression объект Я создаю Expression для каждого условия следующим образом 

foreach ( Filter sf in rules ) {
    Expression ex = sf.ToExpression( query );
    if ( mainExpression == null ) {
        mainExpression = ex;
    }
    else {
        if ( logicalCondition == "AND" ) {
            mainExpression = Expression.And( mainExpression, ex );
        }
        else if ( logicalCondition == "OR" ) {
            mainExpression = Expression.Or( mainExpression, ex );
        }
    }
}

Метод Filter.ToExpression () реализован следующим образом 

public override Expression ToExpression( IQueryable query ) {
    ParameterExpression parameter = Expression.Parameter( query.ElementType, "p" );
    MemberExpression memberAccess = null;
    foreach ( var property in field.Split( '.' ) )
        memberAccess = MemberExpression.Property( memberAccess ?? ( parameter as Expression ), property );
    ConstantExpression filter = Expression.Constant( Convert.ChangeType( value, memberAccess.Type ) );
    WhereOperation condition = (WhereOperation)StringEnum.Parse( typeof( WhereOperation ), operator );
    LambdaExpression lambda = BuildLambdaExpression( memberAccess, filter, parameter, condition, value );
    return lambda;
}

Все работает, когда у меня естьодно условие, но когда я пытаюсь объединить выражения, используя один из статических методов And, Or, AndAlso, OrElse, я получаю InvalidOperationException, который говорит:

двоичный оператор Or не определен для типов 'System.Func 2[MyObject,System.Boolean]' and 'System.Func 2 [MyObject, System.Boolean]'.

Я немного запутался.Может кто-нибудь лучше объяснить причины исключения и предложить решение?

Большое спасибо!

1 Ответ

29 голосов
/ 10 февраля 2012

Вы объединяете a => a == 3 и a => a == 4 в (a => a == 3) || (a => a == 4), но вместо этого вы должны попытаться сделать это a => (a == 3 || a == 4). Это не так сложно сделать вручную, но кто-то уже сделал это для вас . Ищите «Объединение выражений».

Редактировать : по запросу простой пример того, как сделать это вручную.

Редактировать 2 : он использует ExpressionVisitor, который является новым для .NET 4, но на MSDN вы можете найти полезную реализацию для более ранних версий . Я предполагаю, что код MSDN не может рассматриваться как «третье лицо» для ваших целей. Вам нужно только изменить метод protected virtual Expression Visit(Expression exp) на public. А поскольку Enumerable.Zip недоступен для вас и не является необходимостью, его больше нет. 

using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Linq;
using System.Linq.Expressions;

namespace DemoApp
{
    <include ExpressionVisitor definition here for .NET 3.5>

    public class ExpressionParameterReplacer : ExpressionVisitor
    {
        public ExpressionParameterReplacer(IList<ParameterExpression> fromParameters, IList<ParameterExpression> toParameters)
        {
            ParameterReplacements = new Dictionary<ParameterExpression, ParameterExpression>();
            for (int i = 0; i != fromParameters.Count && i != toParameters.Count; i++)
                ParameterReplacements.Add(fromParameters[i], toParameters[i]);
        }
        private IDictionary<ParameterExpression, ParameterExpression> ParameterReplacements
        {
            get;
            set;
        }
        protected override Expression VisitParameter(ParameterExpression node)
        {
            ParameterExpression replacement;
            if (ParameterReplacements.TryGetValue(node, out replacement))
                node = replacement;
            return base.VisitParameter(node);
        }
    }

    class Program
    {
        static void Main(string[] args)
        {
            Expression<Func<int, bool>> exprA = a => a == 3;
            Expression<Func<int, bool>> exprB = b => b == 4;
            Expression<Func<int, bool>> exprC =
                Expression.Lambda<Func<int, bool>>(
                    Expression.OrElse(
                        exprA.Body,
                        new ExpressionParameterReplacer(exprB.Parameters, exprA.Parameters).Visit(exprB.Body)),
                    exprA.Parameters);
            Console.WriteLine(exprA.ToString());
            Console.WriteLine(exprB.ToString());
            Console.WriteLine(exprC.ToString());
            Func<int, bool> funcA = exprA.Compile();
            Func<int, bool> funcB = exprB.Compile();
            Func<int, bool> funcC = exprC.Compile();
            Debug.Assert(funcA(3) && !funcA(4) && !funcA(5));
            Debug.Assert(!funcB(3) && funcB(4) && !funcB(5));
            Debug.Assert(funcC(3) && funcC(4) && !funcC(5));
        }
    }
}
...