Linq中OrderBy后接GroupBy时未取到每组最大ActionId记录问题

你的写法不生效本质是两个认知偏差：

• 你把LINQ to Entities的执行逻辑等同于内存里的LINQ to Objects了。如果是在内存集合上跑这段逻辑，先排序再分组取First确实能拿到每组ActionId最高的记录，但EF这类ORM会把整条查询翻译成SQL发给数据库执行，写在GroupBy()前面的OrderByDescending()不会被当作组内排序依据——SQL标准里GROUP BY本身是聚合操作，不保证组内记录顺序，多数数据库的查询优化器会直接忽略GROUP BY之前的ORDER BY子句，根本不会按你预期的顺序保留组内记录。
• EF对GroupBy()后直接取FirstOrDefault()的翻译逻辑本身就有局限，不管是EF6还是早期版本的EF Core，翻译这类语句时都不会关联你之前写的排序规则，只会随机返回组内任意一条记录。

推荐写法（EF Core 3.0+、支持窗口函数的数据库适用） #

把排序逻辑写在分组内部的Select里，让ORM正确翻译为分组排序取首条的逻辑，EF会自动生成带ROW_NUMBER()窗口函数的SQL，性能好逻辑清晰：

var list = db.Actions
    .Where(z => z.RunId == RunId)
    .GroupBy(c => new
    {
        c.ActionName,
        c.MachineNumber,
    })
    .Select(g => g.OrderByDescending(x => x.ActionId).First())
    .ToList();

兼容写法（适用于老版本EF、不支持窗口函数的数据库） #

先查出每个分组对应的最大ActionId，再通过关联原表拿到完整记录，兼容性最强，所有ORM和数据库版本都支持：

// 第一步：查询每个分组下的最大ActionId
var groupMaxIds = db.Actions
    .Where(z => z.RunId == RunId)
    .GroupBy(c => new { c.ActionName, c.MachineNumber })
    .Select(g => new 
    {
        g.Key.ActionName,
        g.Key.MachineNumber,
        MaxId = g.Max(x => x.ActionId)
    });

// 第二步：关联原表取对应完整记录
var list = (from action in db.Actions
            join maxItem in groupMaxIds
            on new 
            {
                action.ActionName,
                action.MachineNumber,
                ActionId = action.ActionId
            }
            equals new 
            {
                maxItem.ActionName,
                maxItem.MachineNumber,
                ActionId = maxItem.MaxId
            }
            where action.RunId == RunId
            select action).ToList();

注意：不要用AsEnumerable()把全量数据拉到内存再分组排序，数据量稍大就会出现严重的性能问题，尽量让筛选、分组逻辑在数据库侧完成。

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