C# 重构 lambda 表达式

Question

我有几个共享属性的 Expression> 表达式。例如，

Expression<Func<User, bool>> e1 = 
  (User u) => u.IsActive && u.Group != "PROCESS" && u.Name != null;
Expression<Func<User, bool>> e2 = 
  (User u) => u.IsActive && u.Group != "PROCESS" && u.Name != "A";
Expression<Func<User, bool>> e3 = 
  (User u) => u.IsActive && u.Group != "PROCESS" && u.Name != "B";

是否有一种简单的方法可以将 u.IsActive && u.Group != "PROCESS" 在变量中并在 e1、e2 和 e3 中使用它？编辑：我仍然想要同一棵树。

似乎我可以通过使用 Expression.Lambda>(BinaryExpression.AndAlso( 等...) 构建表达式来做到这一点，但它并没有简化我的代码，而是使读。

Answer 1

我相信没有更干净的方法可以为您的案例做到这一点。您可以使用 BinaryExpression 正如您所提到的。您可以将 BinaryExpression 和 Expression.Lambda 调用封装到一个方法中并调用该方法（例如 PredicateBuilder.And），但在我看来，这些都不像当前语法那么干净。

Answer 2

lambda 表达式的问题在于它们是不可变的，并且您无法轻松替换 lambda 的参数。我最初的想法是做这样的事情（不幸的是，这不会成功）：

public static class ExpressionExtesions
{
    public static Expression<Func<T, bool>> And<T>(this Expression<Func<T, bool>> baseCondition, Expression<Func<T, bool>> additionalCondition)
    {
        var and = Expression.AndAlso(baseCondition.Body, additionalCondition.Body);
        return Expression.Lambda<Func<T, bool>>(and, baseCondition.Parameters);  // additionalCondition.Body still uses its own parameters so this fails on Compile()
    }
}

并在代码中使用：

Expression<Func<User, bool>> e = usr => usr.IsActive && usr.Group != "PROCESS";

var e1 = e.And(u => u.Name != null);
var e2 = e.And(u => u.Name != "A");
var e3 = e.And(u => u.Name != "B");

可能的解决方案

您可以尝试使用旨在实现表达式生成器的项目之一。我没有使用过其中任何一个，但谷歌提供了大量链接，例如：

• Metalinq
• LINQ：构建可查询的提供程序 - 第二部分
• 处理 Linq 的不可变表达式树

另一种方法

，如果您是在 LINQ 中使用这些表达式来过滤值，您可以使用不同的方法（不要组合表达式，而是组合过滤器）：

var activeUsers = allUsers.Where(usr => usr.IsActive && usr.Group != "PROCESS");

var usersAll = activeUsers.Where(u => u.Name != null);
var usersNotA = activeUsers.Where(u => u.Name != "A");
var usersNotB = activeUsers.Where(u => u.Name != "B");

Answer 3

我认为不一定有比您已经使用的答案更好的答案。正如 Mehrdad 提到的，您必须使用 BinaryExpression 构建更深的树，我认为这将是当前代码可读性的倒退。

根据您的使用情况，您可能可以通过利用闭包语​​义并执行以下操作来节省一些代码行：

string name = null;

Expression<Func<User, bool>> e = 
  (User u) => u.IsActive && u.Group != "PROCESS" && u.Name != name;

var expr = e.Compile();

name = "A";
var result = expr.Invoke(u); //True (assume u.Name = "B")

name = "B";
result = expr.Invoke(u); //False

...但是这是否有任何用途将取决于您对已编译的委托。可能对你来说完全没用，但值得一提以防万一！

Answer 4

var test = new Func<User, bool>(u=> u.IsActive && u.Group != "PROCESS");
Expression<Func<User, bool>> e1 = (User u) => test(u) && u.Name != null;
Expression<Func<User, bool>> e2 = (User u) => test(u) && u.Name != "A";
Expression<Func<User, bool>> e3 = (User u) => test(u) && u.Name != "B";