LINQ中GroupBy与OrderBy联用导致排序失效的问题

你遇到的这个问题其实是LINQ里GroupBy的一个常见坑——它并不会保留输入序列的排序顺序。你先把整个查询结果按Type升序排列，紧接着用GroupBy按Id分组，这时候分组后的组的顺序已经和之前的排序无关了，再取每个组的FirstOrDefault()，最终自然就看不到预期的排序效果啦。

再仔细看你的场景：同一个Id对应的是同一条对话，所以分组内所有元素的Type其实是完全相同的，这意味着我们只要调整排序的时机，就能轻松解决问题。

解决方案一：分组后再执行排序 #

把OrderBy移到GroupBy和Select之后，这样就能保证最终的对话列表是按Type升序排列的：

var textmessagesList= query.GroupBy(x=>x.Id)
                          .Select(x=>x.FirstOrDefault())
                          .OrderBy(x => x.Type) // 将排序移到分组之后
                          .Skip(offset)
                          .Take(limit)
                          .ToList();

更合理的优化：确保分组内取到正确的记录 #

另外注意到你的查询关联了消息表，同一个对话会对应多条消息，你现在用FirstOrDefault()取的是分组里的任意一条消息，这可能不是你真正想要的（比如通常我们会取对话的最新消息）。如果是这样的话，可以在分组内部先按消息时间倒序排序，再取第一条，最后再按对话类型排序：

var textmessagesList= query.GroupBy(x=>x.Id)
                          // 组内按消息时间倒序，取最新的一条消息
                          .Select(x=>x.OrderByDescending(m => m.LastMessageDateTime).FirstOrDefault())
                          .OrderBy(x => x.Type)
                          .Skip(offset)
                          .Take(limit)
                          .ToList();

为什么之前的写法不行？ #

LINQ中除了ThenBy、Take、Skip这类明确依赖顺序的操作，大多数操作符（包括GroupBy）都不会保证保留输入序列的顺序。当你先排序再分组时，分组操作会重新组织数据，之前的排序信息就被丢弃了，所以最终的分组顺序是不确定的（由LINQ提供者或数据库的执行计划决定）。

备注：内容来源于stack exchange，提问作者Learner