ASP.NET Core EF中LINQ GroupBy分组失效问题排查与解决

问题描述 #

开发ASP.NET Core EF Web应用时，使用LINQ生成报表遇到异常：按CrmId分组后报表无法正常显示数据，但仅使用OrderByDescending排序时报表能正常展示（下图分别为仅排序、分组后的结果）。

原代码 #

public async Task<ContentResult> GetReportData() {
 
    var perAnalystReport = await  _context.Tats
        .ProjectTo<PerAnalystReportDto>(_mapper.ConfigurationProvider)
        .GroupBy(x => x.CrmId)
        .OrderByDescending(x => x.Id) // 此处存在逻辑错误
        .ToListAsync();

     return Content(JsonConvert.SerializeObject(perAnalystReport), "application/json");
}

PerAnalystReportDto类定义 #

public string RefNo { get; set; }
public int CrmId { get; set; }
public DateTime? ReportDate { get; set; }
public DateTime? HoReceivedDate { get; set; }
public string ApprovingAuthorityName { get; set; }
public string BranchName { get; set; }
public string AreaName { get; set; }
public string RegionName { get; set; }
public string Analyst { get; set; }
public string AccountName { get; set; }
public string RequestName { get; set; }
public string FacilityName { get; set; }
public string Description { get; set; }
public string CurrencyName { get; set; }
public int AmountFrom { get; set; }
public int AmountTo { get; set; }
public string Status { get; set; }
public string NextEntityResponsible { get; set; }
public string Remarks { get; set; }
public string Position { get; set; }
public DateTime StatusEffectiveDate { get; set; }
public int TatCount { get; set; }

public int Id { get; set; }

示例Tat数据 #

效果截图 #

问题分析与解决方案 #

原代码核心问题 #

1. 分组后数据类型不符：GroupBy(x => x.CrmId)执行后，得到的是IGrouping<int, PerAnalystReportDto>类型的集合，而非前端需要的PerAnalystReportDto集合。直接序列化这种分组对象，前端无法解析成预期的报表数据结构。
2. 排序逻辑错误：OrderByDescending(x => x.Id)中的x是IGrouping对象，该对象并不包含Id属性，这行代码会导致编译或运行时错误，同时也不符合“取每组最新数据”的需求。
3. 未提取分组内目标数据：分组后没有对每个分组内的元素进行筛选，没有获取到每组中需要展示的具体记录。

可行代码解析 #

public ContentResult GetReportData() {
    var perAnalystReport = _context.Tats
        .ProjectTo<PerAnalystReportDto>(_mapper.ConfigurationProvider)
        .ToList() // 将数据加载到内存后再处理
        .GroupBy(x => x.CrmId)
        .Select(g => g.OrderByDescending(x => x.Id).First()) // 取每组中Id最大的记录
        .ToList();

    return Content(JsonConvert.SerializeObject(perAnalystReport), "application/json");
}

这段代码的关键改进点：

• 先调用ToList()将数据库中的数据加载到内存，再进行分组和筛选操作（若数据量较大，建议改为数据库层面的分组筛选以提升性能）。
• 通过Select(g => g.OrderByDescending(x => x.Id).First())对每个分组内的元素按Id降序排序，取第一条（即最新的）记录，最终得到的是PerAnalystReportDto类型的集合，前端可以正常解析展示。

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