EntityFramework / 从数据库中以最低价格获取两个对象

Question

我正在开发一个 web api 项目。我正在使用依赖注入，所以我在ProductRepository 类中创建了我所有的方法。具体public IQueryable<Product> GetTwoWithLowestPrice()方法。

我需要实现的是从数据库中返回 2 个价格最低的产品。

我试图从数据库中获取所有产品并升序排序，这意味着2个价格最低的产品将在开头 这个sorted 列表。因此，我将 index 位置 0 和 1（前两个）上的产品添加到我的 retVal 列表中......我可能不允许如此轻松地将列表投射到不同的集合中而不会丢失数据（我不熟悉每个集合在那里）。代码：

 public  IQueryable<Product> GetTwoWithLowestPrice()
        {
            List<Product> retVal = new List<Product>();

            var sorted = db.Products.Include(p => p.Category).OrderBy(p => p.Price);

            var list = sorted as List<Product>;

            retVal.Add(list[0]);
            retVal.Add(list[1]);


            return retVal as IQueryable<Product>;

        }

这会引发异常

('Object reference not set to an instance of an object.'   list was null.)

Answer 1

您的代码示例看起来是在不必要地混合了许多技术/实践。

首先：

public  IQueryable<Product> GetTwoWithLowestPrice()

IQueryable 是一个以 Linq 可以查询的方式公开数据的接口。它是一种非常强大的类型，可以让您简化返回类型，尤其是 EF 实体，以便理解数据的调用者可以通过多种方式进一步减少或使用它。通常，这将用于类似 Repository 模式的方法，例如：

public IQueryable<Product> GetProducts()

甚至：

public IQueryable<Product> GetProductById(int productId)

采用这种方法的原因可能是为了集中有关数据的基本规则。例如默认只返回活动产品，或者执行基本级别的规则，例如强制用户只能接收适用于他们的产品。即使对于预期返回单个产品的方法，返回 IQueryable<Product> 也可能看起来很奇怪，但它允许调用者简化查询：

产品是否存在？

bool exists = repository.GetProductById(productId).Any();

而不是使用 EF 加载整个实体以便稍后检查它是否为 Null。

它还允许您使用Select 投影数据，以将查询简化为只选择您关心的数据。 IE。如果您要填充产品列表并且只需要产品 ID、名称和价格：

var productList = repository.GetProducts()
    .Select(x => new ProductSummaryViewModel
    {
        ProductId = x.ProductId,
        Name = x.Name,
        Price = x.Price
    }).ToList();

添加对排序、分页等的支持。这同样适用于“ById”，您可以利用IQueryable 将实体及其亲属投影到您需要的值。

鉴于您有一个设计为只返回 2 种最便宜的产品的方法，IQueryable 并不是一个真正有用的返回类型。我会返回一个IEnumerable。

public  IEnumerable<Product> GetTwoWithLowestPrice()
{

接下来的事情是利用 Linq 订购和获取两种最便宜的产品。对于您的代码：

var list = sorted as List<Product>;

retVal.Add(list[0]);
retVal.Add(list[1]);

至少需要：

var list = sorted.ToList();

retVal.Add(list[0]);
retVal.Add(list[1]);

但是，这是一个代价高昂的错误，因为您告诉 EF 退回所有产品，而您只需要前两个产品。 Linq 和 EF 可以通过 Take 来解决这个问题。

public  IEnumerable<Product> GetTwoWithLowestPrice()
{
    var products = db.Products
        .Include(p => p.Category)
        .OrderBy(p => p.Price)
        .Take(2)
        .ToList();

    return products;
}

就是这样。这将返回 0-2 个最便宜的产品，包括它们的类别，并且查询将只返回这 0-2 行。

总的来说，这是一种非常具体的场景方法，如果这是一个存储库，我可能会考虑将具体的场景逻辑留在控制器中，而不是在数据转换层中使用越来越多的具体方法。这需要将 DbContext 或工作单元提升到 Controller/Presenter。

因此在存储库中使用之前的IQueryable<Product> GetProducts()，想要最便宜的两种产品的控制器代码如下所示：

var products = _repository.GetProducts()
    .OrderBy(x => x.Price)
    .Select(x => new ProductSummaryViewModel
    {
        ProductId = x.ProductId,
        Name = x.Name,
        Price = x.Price,
        CategoryName = x.Category.Name
    }).Take(2)
    .ToList();

这样做的好处是投影 (Select) 将构建一个查询，该查询仅检索必要的字段，包括相关字段 (x.Category.Name)，而不是加载 all 列产品及其关联的类别。 （注意，使用Select 时不需要.Include）我强烈建议使用仅包含视图所需信息的ViewModel，而不是返回整个实体，尤其是与视图相关的实体。它可以节省内存，加快查询速度，并且可以更快地通过网络传输。它还更少地向潜在的攻击者公开您的数据结构。

关于如何利用 EF 和 Linq 获取所需的数据集，这有点失败。