为什么C# Linq中的Take方法会导致OrderBy等排序规则失效？

问题复现代码 #

IReadOnlyList<ResultDTO> res = Repository.GetAll()
.Skip(input.SkipCount)
.Join(UserManager.Users, d=>d.CreatorUserId, c=> c.Id,
   (d, c)=>d.MapToDto(c.UserName)
)
.OrderBy(x=>x.VerifiedByAdmin)
.ThenByDescending(x=>x.CreationTime)
.Take(input.MaxResultCount)
.ToList();

问题原因 #

• 分页操作位置错误：Skip、Take属于分页操作，必须放在所有排序操作之后执行。你当前代码先执行Skip截取部分数据，再对截取后的部分数据做排序，自然不符合全表排序再分页的预期。删除Take后会查询全量数据，在内存中对全量数据排序，所以结果符合要求。
• SQLite排序稳定性限制：SQLite对排序字段值相同的行，在存在LIMIT（对应Take）、OFFSET（对应Skip）语法时，返回顺序不稳定，可能出现随机排序的情况，放大了排序失效的感知。
• 使用IReadOnlyList接收结果不影响查询翻译与执行逻辑，和当前问题无关。

修复方案 #

将所有分页操作挪到排序逻辑之后，同时新增主键作为兜底排序字段，保证SQLite下排序结果稳定，正确代码如下：

IReadOnlyList<ResultDTO> res = Repository.GetAll()
.Join(UserManager.Users, d=>d.CreatorUserId, c=> c.Id,
   (d, c)=>d.MapToDto(c.UserName)
)
.OrderBy(x=>x.VerifiedByAdmin)
.ThenByDescending(x=>x.CreationTime)
.ThenBy(x=>x.Id) // 主键兜底排序，解决同条件下SQLite排序随机问题
.Skip(input.SkipCount)
.Take(input.MaxResultCount)
.ToList();

内容的提问来源于stack exchange，提问作者Navid_pdp11