代码之家 › 专栏 › 技术社区 › It's a trap

C#-向func添加条件会导致堆栈溢出异常

entity-framework-core .net-core c#

It's a trap · 技术社区 · 2 年前

我有一个func作为规范类的一部分,它对给定的iqueryable进行排序

Func<IQueryable<T>, IOrderedQueryable<T>>? Sort { get; set; }

当我像下面这样向func添加多个条件时,它会导致堆栈溢出异常。

spec.OrderBy(sc => sc.Case.EndTime).OrderBy(sc => sc.Case.StartTime);

OrderBy方法是这样实现的

public ISpecification<T> OrderBy<TProperty>(Expression<Func<T, TProperty>> property)
    {
        _ = Sort == null ? Sort = items => items.OrderBy(property) : Sort = items => Sort(items).ThenBy(property);
        return this;
    }

链接或使用单独的线没有区别。

如果我分配规范的一个新实例并将其设置为func,这个问题就会得到解决,但我不想每次都分配给一个新实例。请建议我这里缺少什么,以及如何重用同一个实例(如果可能的话)。

1 回复 | 直到 2 年前

Jon Skeet 2 年前

这是有问题的部分:

Sort = items => Sort(items)

这就像编写一个调用自身的方法。

你呢希望就是评估 现有的 Sort 功能,而不是“结果” 分类 评估时的财产”。

我会这样重写这个方法:

public ISpecification<T> OrderBy<TProperty>(Expression<Func<T, TProperty>> property)
{
    var existingSort = Sort;
    Sort = existingSort is null
        ? items => items.OrderBy(property)
        : items => existingSort(items).ThenBy(property);
    return this;
}

(我也要附和施德荣的评论——这是一种有点违反直觉的方法,与正常的LINQ背道而驰。也许你有一些特别的理由反对LINQ对不变性和链接的期望,但这肯定是不寻常的。)

推荐文章

Glory Raj · 有什么方法可以使这两个块具有共同的功能吗

2 年前

Tomas Kubes · 是最大并发数。NET任务有限吗?

2 年前

Dansih · .Net核心自定义身份验证方案

2 年前

It's a trap · C#-向func添加条件会导致堆栈溢出异常

2 年前

Tropin Alexey Sam Fenilto · 字典序列化:系统。例外:Type<Type>不是字典

2 年前

shan127 · 如何发布。net核心控制台应用程序转换为单个。exe文件?

2 年前

user11909 · 物体怎么样。Equals()最终实现了吗?

2 年前

Matheus Dias de Souza · 通过nullable原谅null时收到警告。值类型上的值属性

2 年前

Matheus Dias de Souza · Null原谅运算符导致可为Null的类型

2 年前

John · 无法在控制台应用程序中使用AddHttpClient/IHttpClientBuilder

2 年前