ASP.NET MVC EF5 如何无if-else实现KPI三状态筛选查询优化

你当前代码冗余的核心原因是两个查询分支仅存在KPI筛选条件的差异，其余筛选规则、关联加载配置、查询执行逻辑完全重复，完全可以移除if-else分支，将KPI筛选逻辑直接整合到统一的Where条件中，和你已经在使用的状态、项目、部门筛选逻辑保持一致。

优化思路 #

你现有代码里对状态、项目、部门三个字段已经用了非常标准的无分支筛选写法：(字段匹配传入值 || 传入值为"全部"标识)，KPI字段只需要套用完全相同的逻辑即可：

• 当传入request.IsKPI为1时，仅匹配IsKPI == true的数据
• 当传入request.IsKPI为0时，仅匹配IsKPI == false的数据
• 当传入值为其他值（代表「全部」）时，KPI筛选条件恒为真，不做过滤

优化后完整代码 #

var result = await _context.TaskMaster
    .Where(c => c.IsActive 
        && (c.Status == request.Status || request.Status == "All") 
        && (c.ProjectId == request.ProjectId || request.ProjectId == 0) 
        // 整合KPI筛选逻辑，无需分支
        && (request.IsKPI != 0 && request.IsKPI != 1 || c.IsKPI == (request.IsKPI == 1))
        && (c.AssignedTo.DepartmentId == request.DepartmentId || request.DepartmentId == 0))
    .Include(y => y.TaskActionLogs)
    .Include(y => y.AssignedTo)
    .Include(y => y.AssignedBy)
    .Include(y => y.Project)
    .AsSplitQuery()
    .ToListAsync();

方案说明 #

• 完全移除了冗余的if-else分支和多余的临时变量赋值，代码量缩减近一半
• EF Core会自动将该布尔逻辑翻译为高效的SQL查询，和原分支写法的执行性能完全一致，不会出现额外性能损耗
• 所有筛选逻辑集中在同一个Where语句中，后续调整筛选规则、修改关联加载配置时仅需修改一处，不会出现多分支改漏的问题
• 写法和你现有其他三个筛选条件的逻辑完全统一，代码一致性更高，后续维护成本更低
• 如果觉得行内的KPI判断语义不够直观，可以提前声明两个无分支赋值的布尔变量提升可读性，依然不需要if-else判断：

// 判断是否需要启用KPI筛选、目标KPI值，全程无分支
bool enableKpiFilter = request.IsKPI is 0 or 1;
bool targetKpiValue = request.IsKPI == 1;

将Where中的KPI条件替换为(!enableKpiFilter || c.IsKPI == targetKpiValue)即可，语义更清晰。

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