Question

我似乎无法执行我之前分配给变量的lambda表达式。这是我放在一起的一个小型C＃示例程序：

public class Program
{
    public static void Main(string[] args)
    {
        int[] notOrdered = { 3, 2, 5, 8, 1, 4, 7, 9, 6 };
        Print(notOrdered);

        IEnumerable<int> ascOrdered = Order(notOrdered, true);
        Print(ascOrdered);

        IEnumerable<int> descOrdered = Order(notOrdered, false);
        Print(descOrdered);
    }

    static IEnumerable<T> Order<T>(IEnumerable<T> enumerables, bool ascending)
    {
        Expression<Func<T, object>> selector = (z) => z; // simple for demo purposes; pretend it's complex
        if (ascending)
            return enumerables.OrderBy(z => selector);
        else
            return enumerables.OrderByDescending(z => selector);
    }

    static void Print<T>(IEnumerable<T> enumerables)
    {
        foreach(T enumerable in enumerables)
            Console.Write(enumerable.ToString() + " ");
        Console.WriteLine();
    }
}

我希望它能产生这个输出：

3 2 5 8 1 4 7 9 6

1 2 3 4 5 6 7 8 9

9 8 7 6 5 4 3 2 1

但是，令人困惑的是，它产生了这个输出：

3 2 5 8 1 4 7 9 6

3 2 5 8 1 4 7 9 6

3 2 5 8 1 4 7 9 6

基本上，我只是希望能够将相同的表达式传递给两个不同的排序操作，而不必两次输出，因此我事先将其分配给selector。我有一个真实世界的用例，其中lambda表达式非常冗长/凌乱，我不想复制混乱，我宁愿只是引用一个像我这里的变量。

那么，a）导致当前输出的原因是什么？ b）如何获得我想要的输出？

Answer 1

在您的代码中，您不使用表达式。

您的代码如下：

object selector = new object();
if (ascending)
    return enumerables.OrderBy(z => selector);
else
    return enumerables.OrderByDescending(z => selector);

你的代码应该没有表达：

Func<T, object> selector = (z) => z;
if (ascending)
    return objects.OrderBy(selector);
else
    return objects.OrderByDescending(selector);

但是，如果你真的想要一个表达式（出于测试目的或其他任何目的），请在之前编译它：

Expression<Func<T, object>> selector = (z) => z;
var compiledExpression = selector.Compile();
if (ascending)
    return objects.OrderBy(compiledExpression);
else
    return objects.OrderByDescending(compiledExpression);

Answer 2

a）现在，您按Expression<Func<T,object>>（实际表达式实例）排序，而不是按对象本身排序。这有效地使排序始终使用相同的对象，这意味着order by子句不会改变任何东西（所有项都是“相等的”，因为它们等于表达式的相同实例）。

b）我相信你想要：

static IEnumerable<T> Order<T>(IEnumerable<T> enumerables, bool ascending)
{
    if (ascending)
        return enumerables.OrderBy(z => z);
    else
        return enumerables.OrderByDescending(z => z);
}

Answer 3

您只是比较表达式的实例，而不是调用selector。正确的方法应该是

static IEnumerable<T> Order<T>(IEnumerable<T> enumerables, bool ascending)
{
    Func<T, object> selector = (z) => z;
    if (ascending)
        return enumerables.OrderBy(z => selector(z));
    else
        return enumerables.OrderByDescending(z => selector(z));
}

分配lambda表达式会导致它以后不执行？

3 个答案: