c# - 使用 LINQ 基于多个参数进行过滤？

Question

我正在使用 ASP.NET MVC 5、EF6 和 LINQ 开发一个电子商务网站。我的数据库中有一个产品表。在我的用户界面中，我有多个用于过滤产品的参数:

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

我写了这个操作方法:

[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 语句，只是为了过滤产品。我的整体设计有问题吗？有没有最佳实践？任何帮助将不胜感激。谢谢。

Answer 1

您可以将业务逻辑提取到帮助程序或服务类(例如 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);
}

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

希望我有用!