Linq where 子句"if item is not null, evaluate, otherwise, get all data"

Question

我正试图在web客户端上实现一个过滤表单，该表单可以有任意数量的项目，要么已填写，要么为空。我的WebAPI方法将采用一个模型，该模型是表单的所有属性，它会像：

RequisitionType (enum: RequisitionType.RequisitionedBy, RequisitionType.CreatedBy)
Vendor (string or null)
Plant (string or null)
CompanyCode (string or null)
CreatedFrom (datetime)
CreatedTo (datetime)

如果没有许多疯狂的if (condition) { filter }，我如何实现linq查询？

我尝试过这样的方法，首先检查过滤器是否有值，如果有，我会将字段与该值进行比较，否则我会将该字段与自身进行比较，但编译器不允许这样做。

在下面的示例中，model是从客户端传递给控制器的内容。

var data =  logic.GetPurchaseReqs()
            .Where(pr => {
                model.RequisitionType == RequisitionType.RequisitionedBy ? pr.PrReqId.ToUpper() == model.Username.ToUpper() : pr.PrReqId.ToUpper() == pr.PrReqId.ToUpper()
                && model.RequisitionType == RequisitionType.CreatedBy ? pr.PrCreId.ToUpper() == model.Username.ToUpper() : pr.PrCreId.ToUpper() == pr.PrCreId.ToUpper()
                && model.Vendor != null ? pr.Vendor == model.Vendor : pr.Vendor == pr.Vendor
                [etc...]
            })
            .ToList();

Answer 1

我相信简化这一过程的最简单方法是：

logic.GetPurchaseReqs()
     .Where(pr => model.RequisitionType != RequisitionType.RequisitionedBy || pr.PrReqId.ToUpper() == model.Username.ToUpper())
     .Where(pr => model.RequisitionType != RequisitionType.CreatedBy || pr.PrCreId.ToUpper() == model.Username.ToUpper())
     .Where(pr => model.Vendor == null || pr.Vendor == model.Vendor)
     //[...]
     .ToList();

请注意，添加几个Where子句会带来很小的开销（只是JIT调用函数所需的处理能力），但IMHO提高了可读性。

Answer 2

为了可读性，由于代码量几乎相同，我肯定会选择if(condition)解决方案。

var data = logic.GetPurchaseReqs(); //.AsEnumerable() migt be needed here
if(model.RequisitionType == RequisitionType.RequisitionedBy)
  data = data.Where(pr => pr.PrReqId.ToUpper() == model.Username.ToUpper())
if(model.Vendor != null)
  data = data.Where(pr => pr.Vendor == model.Vendor);
if(model.CompanyCode != null)
  data = data.Where(pr => pr.CompanyCode == model.CompanyCode);
...

Answer 3

不是完全按照你想要做的，而是基于你的标题：

var data = logic.GetPurchaseReqs();
if(condition)
{
    data = data.Where(pr => pr.... etc
}
var result = data.ToList();

Answer 4

这就是我对它的编程方式。注意没有括号以及我处理请购单类型的方式。这假设请购单类型不能为null。。。如果它能。。。则必须添加第三个条件CCD_ 5。否则，您检查的每个条件都会被添加为另一个&& (condition != null ? conditionCheck : true)

var data = logic.GetPruchaseReqs().Where(pr =>
    ((model.RequisitionType == RequisitionType.RequisitionedBy && pr.PrReqId.Equals(model.Username, StringComparison.InvariantCultureIgnoreCase)) || (model.RequisitionType == RequisitionType.CreatedBy && pr.PrCreId.Equals(model.Username, StringComparison.InvariantCultureIgnoreCase))) 
    && (model.Vendor != null ? pr.Vendor.Equals(model.Vendor) : true) 
    && (condition != null ? conditionCheck : true)).ToList();

目标是设置=>的右侧，以计算到单个布尔值true/false。您应该能够针对（pr）的任何给定值来评估语句，并得到true或false。

Answer 5

最好的解决方案是为linq创建一个扩展方法。它将接受一个条件语句和一个where表达式，如果条件为true，它将使用where表达式，否则它将跳过它。这可以极大地清理代码，消除对if语句的需要。

public static class LinqExtension
{
    public static IQueryable<T> Where<T>(this IQueryable<T> query, bool condition, Expression<Func<T, bool>> whereClause)
    {
        if (condition)
        {
            return query.Where(whereClause);
        }
        return query;
    }
}

现在，您可以在不使用if语句的情况下执行条件where语句。

var data = logic.GetPurchaseReqs()
                .Where(model.RequisitionType == RequisitionType.RequisitionedBy, pr => pr.PrReqId.ToUpper() == model.Username.ToUpper())
                .Where(model.Vendor != null, pr => pr.Vendor == model.Vendor)
                .Where(model.CompanyCode != null, pr => pr.CompanyCode == model.CompanyCode);