新增2条LINQ语句后查询耗时增加10秒，求排查优化方案

1. 第一步：抓取LINQ生成的原生SQL确认问题根因 #

你首先需要拿到这两句LINQ最终生成的SQL语句，以及完整查询的执行日志，重点排查两个问题：

• 是否出现N+1查询：如果你这两句LINQ是嵌套在遍历demand表的循环或者主查询的投影逻辑中，1.3万条demand记录会触发2.6万次独立的子查询，这是10秒额外耗时的最常见原因
• 是否生成了逻辑冗余的SQL：比如关联条件错误、全表扫描无匹配索引等
  不同ORM抓SQL的方式：
• EF Core：可以启用EnableSensitiveDataLogging()查看生成的SQL
• EF6：可以用Database.Log属性输出SQL到控制台/日志

2. 索引优化（必做） #

DemandXPCILStatus表目前没有针对你的查询条件的索引，每次查询都会触发全表扫描，按你的查询逻辑创建联合索引即可：

CREATE NONCLUSTERED INDEX IX_DemandXPCILStatus_DemandID_Active ON DemandXPCILStatus
(
    DemandID ASC,
    Active ASC
)
INCLUDE (PCILStatusID, ID, PCILTime, LastUpdatedOn) -- 包含你需要返回的字段，避免回表查询

这个索引可以直接命中你两句LINQ的查询过滤条件，查询速度会提升至少一个数量级。

3. LINQ写法优化 #

你第二句LINQ的关联逻辑写得不合理，关联条件里混入了和两表关联无关的过滤条件，会导致生成的SQL执行计划效率极低，优化后逻辑如下：

优化点1：合并两次查询，避免重复扫描DemandXPCILStatus表 #

你完全可以先查DemandXPCILStatus，再基于结果取PCILStatus，不需要两次查询DemandXPCILStatus：

// 先查关联关系，只用扫一次DemandXPCILStatus表
var demandXPCILStatus = demandXPCILStatuses.FirstOrDefault(s => s.DemandID == demand.ID && s.Active == true);
DemandXPCILStatus = demandXPCILStatus;
// PCILStatus表仅6条数据，建议提前全量加载到内存用字典映射，完全避免数据库查询
PCILStatus = demandXPCILStatus != null ? _pcilStatusDict.GetValueOrDefault(demandXPCILStatus.PCILStatusID) : null;

优化点2：小表预加载 #

PCILStatus表只有6条记录，你可以在程序启动时或者查询前一次性全量加载到内存，构建Dictionary<int, PCILStatus>的ID映射字典，后续取PCILStatus直接查内存，不需要每次联表查询数据库。

4. 避免N+1查询的终极方案 #

如果你这两句逻辑是写在主查询的Select投影里，建议改成左关联到主查询中，一次性加载所有需求对应的关联数据，避免逐行触发子查询：

var query = from d in demands
            join s in demandXPCILStatuses on d.ID equals s.DemandID into sGroup
            from s in sGroup.Where(x => x.Active == true).Take(1).DefaultIfEmpty()
            join p in PCILStatuses on s.PCILStatusID equals p.ID into pGroup
            from p in pGroup.Take(1).DefaultIfEmpty()
            select new 
            {
                Demand = d,
                DemandXPCILStatus = s,
                PCILStatus = p
            };

这种写法会生成单次关联查询，一次性返回所有需要的数据，就算1.3万条需求查询耗时也只会在百毫秒级别。

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