使用 LINQ 基于多个参数进行筛选

本文关键字：筛选 参数 于多个 LINQ 使用 | 更新日期: 2023-09-27 17:56:16

我正在使用 ASP.NET MVC 5，EF6和LINQ在电子商务网站上工作。我的数据库中有一个"产品"表。在我的 UI 中，我有多个参数用于筛选我的产品：

  • 不同类别的复选框
  • 最低和最高价格
  • 不同颜色的复选框

我写了这个动作方法：

[HttpPost]
public PartialViewResult FilterProducts(int[] categoriesIds, decimal minPrice, decimal? maxPrice, string[] colors)
{
        if (categoriesIds == null)
        {
            var randomProducts = db.Products
                .OrderBy(p => Guid.NewGuid());
            return PartialView("_LoadProducts", randomProducts.ToList());
        }
        else
        {
            var filteredProducts = db.Products
                .Where(p => categoriesIds.Contains(p.CategoryId)
                    && (p.DiscountedPrice >= minPrice
                        && p.DiscountedPrice <= maxPrice || maxPrice == null));
            if (colors != null)
            {
                filteredProducts = filteredProducts
                    .Where(p => colors.Contains(p.Color));
            }
            return PartialView("_LoadProducts", filteredProducts.ToList());
        }
}

这工作正常。但我对这是否可以改进感到困惑？它包括许多 if/else 语句，仅用于过滤产品。我的设计整体有问题吗？是否有任何最佳实践？任何帮助将不胜感激。谢谢。

使用 LINQ 基于多个参数进行筛选

您可以将业务逻辑提取到帮助程序或服务类（例如 ProductsService）中，并将每个实体的数据库查询提取到存储库层（例如 ProductsRepository、UsersRepository 等...）

public class ProductsRepository : BaseRepository<Product> // You can have base implementation for basic CRUD operations
{
     MyDbContext db;
     public ProductsRepository(MyDbContext db)
     {
         this.db = db;
     }
     public IQueryable<Product> GetAll()
     {
          return db.Products;
     }
}
public class ProductsService : BaseService<Product> // again here....
{
     ProductsRepository repo;
     public ProductsService()
     {
          repo = new ProductsRepository(new MyDbContext()); // Intentionally not using Dependency Injection here for simplicity....
     }
     public List<Product> FilterBy(int[] categoriesIds, decimal minPrice, decimal? maxPrice, string[] colors){
     {
         if (categoriesIds == null)
         {
             var randomProducts = repo.GetAll().OrderBy(p => Guid.NewGuid());
             return randomProducts.ToList();
         }
         else
         {
          var filteredProducts = repo.GetAll()
            .Where(p => categoriesIds.Contains(p.CategoryId)
                && (p.DiscountedPrice >= minPrice
                    && p.DiscountedPrice <= maxPrice || maxPrice == null));
        if (colors != null)
        {
            filteredProducts = filteredProducts
                .Where(p => colors.Contains(p.Color));
        }
        return filteredProducts.ToList();
    }
}

MVC

ProductsService productsService;
public MyController() // constructor
{
   productsService = new ProductsService(); // Again, no Dependency Injection here for simplicity
}
[HttpPost]
public PartialViewResult FilterProducts(int[] categoriesIds, decimal minPrice, decimal? maxPrice, string[] colors)
{
     List<Product> products = productsService.FilterBy(categoriesIds, minPrice, maxPrice, colors);
     return PartialView("_LoadProducts", products);
}

那么这里有什么意义...如您所见，您的代码比平时多，但这种方法使您的代码更好地分离且易于重用。可以在应用程序中的任何位置反复使用 FilterBy 方法ProductsService，而无需多次重复查询。此外，您的控制器更轻且易于阅读，这是最好的方法 - 您不应该直接在控制器中进行繁重的业务逻辑/数据库操作。这就是为什么最好将数据库查询和业务逻辑分离到单独的文件中，并在可能的情况下重用它们。这导致更少的错误。这是 SOLID 原则的第一条规则 - 一个类或方法应该只有一个职责 - 这意味着如果你的方法要从数据库中返回项目，它应该只做这个，仅此而已。

希望我有用！

添加到@GeorgeFindulov的答案

将输入作为标准的预定义对象

public class FilterDto
{
    public int[] Categories { get; set; }
    public int? MinPrice{ get; set; }
    public int? MaxPrice{ get; set; }
    //...
}

这使您的控制器干净且对于新输入保持不变。 如果将来有一些新的筛选器，只需将一个新属性添加到类 FilterD，以保持操作方法不变。您可以使用自动映射器将此 DTO 转换为业务对象。

[HttpGet]
public PartialViewResult FilterProducts(FilterDto filters)
{
    FilterModel filterModel = Mapper.Map<FilterModel>(filters);
    List<Product> products = productsService.FilterBy(filterModel);
    return PartialView("_LoadProducts", products);
}

注意：您的服务应始终返回业务模型而不是特定模型，然后控制器转换为 DTO 或视图模型。

相关文章：