如果变量为空 ASP.NET MVC，则不包括 .Where()

Question

我有这个功能：

        public List<PostIndexResponse> GetPosts(string search, int? categoryId)
        {
            var currentUserId = HttpContext.Current.User.Identity.GetUserId() ?? "";
            var srch = search ?? "";
            var posts = _set.Where(x => x.PersonId.Contains(currentUserId) && x.Description.Contains(srch) && x.CategoryId == categoryId).OrderByDescending(x => x.CreatedOn)).ToList();
            return posts;
        }

仅当categoryId不为空时，我才想要x.CategoryId == categoryId 。 如何在没有 if else 条件的情况下执行此操作并基于它是否为空创建另一个查询？

Answer 1

我会把它分成几行像这样（未经测试）......

这允许您有条件地过滤categoryId ，我认为它比尝试将其全部塞进单个表达式更清洁和更易于维护。 无需创建 2 个重叠表达式

public List<PostIndexResponse> GetPosts(string search, int? categoryId)
{
    var currentUserId = HttpContext.Current.User.Identity.GetUserId() ?? "";
    var srch = search ?? "";

    var query = _set.Where(x => x.PersonId.Contains(currentUserId) && x.Description.Contains(srch));

    if (categoryId.HasValue)
        query = query.Where(x => x.CategoryId == categoryId);

    var posts = query.OrderByDescending(x => x.CreatedOn)).ToList();
    return posts;
}

另一种选择是将x.CategoryId == categoryId替换为

(categoryId == null || x.CategoryId == categoryId)

但这种表达方式已经变得越来越长，难以维持。