c# - 如何重构此代码以进行 LINQ 过滤？

Question

使用 DataTables(jQuery 的表插件)进行服务器端处理，我必须为我的数据创建具有良好性能的过滤。我这样做了，但我认为这有点丑陋。 (.Where(...)) 的一个特别部分，我必须手动将每个属性与 search 变量进行比较。可以改进吗？

using System.Linq.Dynamic; // because of special .OrderBy

public class SomeRepository
{
    public DataTableDTO GetAllFromBase(int start, int length, string sortColumn, string sortColumnDir, string search)
    {
        var dataFiltered = db.User
            .AsNoTracking()
            .Select(x => new { x.Id, x.FirstName, x.LastName, x.Description})
            .OrderBy(sortColumn + " " + sortColumnDir)
            .Where(search.Length > 0, x => x.Id.ToString().Contains(search.ToLower())
                                        || x.FirstName.ToLower().Contains(search.ToLower())
                                        || x.LastName.ToLower().Contains(search.ToLower())
                                        || x.Description.ToLower().Contains(search.ToLower()));

        var recordsFiltered = dataFiltered.Count();
        var recordsTotal = db.User.Count();

        var dataToShow = dataFiltered
            .Skip(start)
            .Take(length)
            .ToList();

        var dataForTable = new DataTableDTO
        {
            Data = dataToShow,
            RecordsTotal = recordsTotal,
            RecordsFiltered = recordsFiltered
        };

        return dataForTable;
    }
}

public static class LinqExtensions
{
    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;
    }
}    

Answer 1

您可以提取此方法:

public bool SearchForMatching(string source, string serach)
{
    return source.ToLower().Contains(search.ToLower());
} 

它将简化您的Where子句:

Where(x => SearchForMatching(x.Id, search)
           || SearchForMatching(x.FirstName, search)
           || SearchForMatching(x.LastName, search)
           || SearchForMatching(x.Description, search));

如果您不想逐个搜索每个属性并且想要检查类的任何属性是否回答搜索，则另一种选择是使用反射来迭代类的属性并检查是否有任何属性回答条件:

public bool SearchForMatching(User user, string search)
{
    return user.GetType().GetPropeties().Any(propertyInfo => propertyInfo.GetValue(user).ToString().ToLower().Contains(search.ToLower()));
}

然后在Where子句中使用这个方法:

Where(x => SearchForMatching(x, search));

或者只是将它们组合在一起:

Where(x => x.GetType().GetPropeties().Any(propertyInfo => propertyInfo.GetValue(x).ToString().ToLower().Contains(search.ToLower())); 

编辑
这两个选项应该可以很好地与 LINQ to Objects 配合使用，但可能无法与 LINQ to Entities 很好地协作，因为无法使用 LINQ to Entities 将第一个选项或反射选项转换为 SQL。

您可以使用 db.User.AsEnumerable() 将所有数据加载到内存中，然后使用这些选项中的任何一个来使用 LINQ to Objects，但我建议，它比在数据库中执行所有过滤(如第一个查询那样)的效率要低尽管保留您的第一个查询具有可读性。