如何优化 .NET Core 中的 lambda 查询表达式 ?

Posted 2021-10-19 dotNET跨平台

咨询区

• Moutia AYED：

我的项目需要构建几个图表，产生图表的 数据源 相对比较复杂，参考如下代码:

var resultdb = await  _dbContext.TimeSheetElements
     .Where(t => t.UserId == userId &&
            t.Date.Month == month && 
            t.Date.Year == year)
     .GroupBy(t => t.Date)
     .Select(pc => 
         new ShowPointingChartViewModel
         {
             Day = pc.First().Date.DayOfWeek.ToString(),
             Date = pc.First().Date.ToString(),
             NormalHours = pc.Where(p => p.IsGuard == false && !taskType.Contains(p.UserTask.Type)).Sum(p => p.Duration),
             OutOfBuisnessHours = pc.Where(p => p.IsGuard == true).Sum(p => p.Duration),
             Holidays = pc.Where(p => p.UserTask.Type == "Holiday").Sum(p => p.Duration),
             PublicHolidays = pc.Where(p => p.UserTask.Type == "Public holiday").Sum(p => p.Duration),
             Illness = pc.Where(p => p.UserTask.Type == "Illness").Sum(p => p.Duration),
             GuardFees = pc.Count(p => p.UserTask.Type == "Night Fees"),
             Total = pc.Where(p=>p.UserTask.Type != "Night Fees").Sum(p => p.Duration)
          })
    .ToList();

实际跑起来后，我发现这个 EF 查询语句需要耗费相当长的时间，虽然结果是正确的，但我是一个性能追求者，我想怎么尽可能的对它进行优化，缩短 response 的时间 ？

回答区

• Eriawan Kusumawardhono：

从你的代码看：这个 query 非常复杂，这么重的 query 查询性能必然会低效。

我给出的建议是：你可以先根据一些 where 条件从数据库中预先提取数据，提取完之后，接下来在内存中对这些业务逻辑进行复杂的运算，比如各种 Where，Count 然后再映射到 ShowPointingChartViewModel，参考如下代码：

var resultList = await  _dbContext.TimeSheetElements
   .Where(t => t.UserId == userId && t.Date.Month == month && t.Date.Year == year)
   .GroupBy(t => t.Date).ToList();
var chartViewModel = resultList.Select(pc => 
     new ShowPointingChartViewModel
     {
         Day = pc.First().Date.DayOfWeek.ToString(),
         Date = pc.First().Date.ToString(),
         NormalHours = pc.Where(p => p.IsGuard == false && !taskType.Contains(p.UserTask.Type)).Sum(p => p.Duration),
         OutOfBuisnessHours = pc.Where(p => p.IsGuard == true).Sum(p => p.Duration),
         Holidays = pc.Where(p => p.UserTask.Type == "Holiday").Sum(p => p.Duration),
         PublicHolidays = pc.Where(p => p.UserTask.Type == "Public holiday").Sum(p => p.Duration),
         Illness = pc.Where(p => p.UserTask.Type == "Illness").Sum(p => p.Duration),
         GuardFees = pc.Count(p => p.UserTask.Type == "Night Fees"),
         Total = pc.Where(p=>p.UserTask.Type != "Night Fees").Sum(p => p.Duration)
      })
.ToList();

点评区

说实话，这个问题的优化建议还是值得学习一下的，我个人建议用 SQL Server Profile 或者 Entity Framework 自定的监控方法看看 sql 长啥样，有了这个原始sql，我们就可以进行针对性的优化。