如何让LINQ在条件不满足时跳过Where子句？优化冗余if-else逻辑

嘿，这种嵌套if else把查询逻辑拆得七零八落的，确实头疼！我之前也遇到过一模一样的场景，用LINQ延迟执行+链式条件拼接就能完美解决，完全不用写一堆分支判断，代码瞬间清爽很多！

先看看你现在的冗余代码（补全了部分逻辑） #

List<T> GetTs(string createBy, bool isAdmin, string departmentCode, ...) 
{ 
    if(isAdmin) 
    { 
        if(!string.IsNullOrEmpty(createBy)) 
        { 
            var query = context.table 
                .Where(x => string.Compare(x.CreateBy, createBy, StringComparison.OrdinalIgnoreCase) == 0) 
                .ToList(); 
            return query; 
        } 
        else 
        { 
            var query = context.table.ToList(); 
            return query; 
        } 
    } 
    else 
    { 
        if(!string.IsNullOrEmpty(createBy)) 
        { 
            var query = context.table 
                .Where(x => string.Compare(x.CreateBy, createBy, StringComparison.OrdinalIgnoreCase) == 0) 
                .Where(x => x.DepartmentCode == departmentCode) 
                .ToList(); 
            return query; 
        } 
        else 
        { 
            var query = context.table 
                .Where(x => x.DepartmentCode == departmentCode) 
                .ToList(); 
            return query; 
        } 
    } 
}

优化后的简洁写法 #

核心思路是利用LINQ的延迟执行特性：IQueryable<T>（EF等ORM返回的类型）不会立即执行查询，只有当调用ToList()、FirstOrDefault()这类触发枚举的方法时，才会把所有拼接的条件转换成SQL发送到数据库。我们可以按需逐个添加Where条件，不满足的就跳过：

List<T> GetTs(string createBy, bool isAdmin, string departmentCode, ...) 
{ 
    // 初始化基础查询（此时还没访问数据库）
    IQueryable<T> query = context.table;

    // 仅当createBy不为空时，添加创建人过滤条件
    if (!string.IsNullOrEmpty(createBy))
    {
        query = query.Where(x => string.Compare(x.CreateBy, createBy, StringComparison.OrdinalIgnoreCase) == 0);
    }

    // 非管理员需要额外过滤部门编码
    if (!isAdmin)
    {
        query = query.Where(x => x.DepartmentCode == departmentCode);
    }

    // 最后统一执行查询，触发数据库访问
    return query.ToList();
}

为什么这么好用？ #

1. 逻辑清晰：每个条件单独判断是否添加，不用嵌套分支，后期维护时找对应条件一目了然；
2. 性能无损：所有Where条件会被组合成一个SQL查询（EF会自动拼接），不会产生多次数据库访问，和原生分支写法性能一致；
3. 扩展性强：如果后续要加更多过滤条件（比如时间范围、状态筛选），直接加新的if块拼接Where就行，不用改原有逻辑。

进阶小技巧 #

如果有大量重复的条件判断，还可以把常用的过滤逻辑抽成扩展方法，进一步简化代码：

public static IQueryable<T> FilterByCreateBy(this IQueryable<T> query, string createBy)
{
    if (!string.IsNullOrEmpty(createBy))
    {
        return query.Where(x => string.Compare(x.CreateBy, createBy, StringComparison.OrdinalIgnoreCase) == 0);
    }
    return query;
}

// 调用时更简洁
query = query.FilterByCreateBy(createBy);

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