从控制器中查询从存储库获得的 List<T> 增加耦合？

Question

我有一个用 C# 编码的 ASP.NET MVC 应用程序。应用程序的结构如下：

1. 控制器

2. 存储库

3. LINQ to Entities（实体框架）

4. 查看

我使用存储库 (_ProductRep) 查询 LINQ to Entities 并将实际实体或 List<T> 提供给控制器，而不是 IQueriables<T>。

对于我有更多疑问的情况，我想获得一些帮助。我有以下代码：

List<Monthly_Report> lproduct_monthlyReport = _ProductRep.GetArchiveReport(product.Prod_ID, lmonth, lyear);

得到lproduct_monthlyReport 后，我需要在 foreach 中查询它并获取特定记录。目前我实现了这样的解决方案：

foreach (var item in litemList)
{    
   var lproductItem_monthlyReport = lproduct_monthlyReport.Single(m => m.Item_ID == item.Item_ID);
   // Other code
} 

其中litemList 是产品可以拥有的所有可能项目的列表。

我想知道这个解决方案是否明智地增加了耦合（并且违反了law of Demeter）或者它是可以接受的，因为我实际上是在查询List<T>而不是IQueriable<T>。如果我错了，请纠正我，但我猜由于 List 不需要访问 EF DataContext，因此 Controller 和 EF 之间没有耦合。

万一我错了，我能想到的唯一解决方案是用存储库方法替换查询（我仍然必须实现）：

var lproductItem_monthlyReport_ProductRep.GetArchiveReport(product.Prod_ID, lmonth, lyear, item.Item_ID);

使用此解决方案，但是存储库在每个循环周期中使用 4 个条件进行一次查询，而在之前的解决方案中，存储库仅使用一个条件进行查询。

请您就这个问题给我指点一下吗？谢谢。

PS：我在循环中需要两个变量 lproduct_monthlyReport 和 lproductItem_monthlyReport，我不能只使用其中一个

PPS：我知道我应该在 Controller 和 Repository 之间建立一个业务服务层，这是我的下一步。

Answer 1

从存储库返回列表会给你带来糟糕的性能，因为你失去了延迟执行的行为。基本上，您的存储库会将每条记录（而不是相关实体）检索到内存中，并将它们转换为列表，然后在内存中进行处理。如果要访问相关实体，则需要另一个数据库命中。如果您坚持使用 IEnumerable（或 IQueryable），那么您将向客户端隐藏实体框架行为的细微差别，但仍会获得延迟加载和延迟执行等优势。

暂时忽略你的存储库的细节，如果你这样做：

List<Product> products = MyEntities.Products.ToList();

Product product1 = products.Single(p => p.Id = 1);

它的表现会比这差得多：

IEnumerable<Product> products = MyEntities.Products;

Product product1 = products.Single(p => p.Id = 1);

第一个将在没有 WHERE 子句的数据库中执行 SELECT，然后为每个结果实例化 .Net 对象，然后查询该内存列表。在您访问 product1 上的属性之前，第二个将不执行任何操作，此时将发出数据库命令以仅检索 1 个产品，并仅实例化该 1 个产品。

两者之间的差异对于小数据集可能并不明显，但随着数据集变大，这种差异会越来越严重。加入一个连接的实体（或更糟糕的实体集合），您可能会获得数千个数据库命中，如果您坚持使用 IEnumerable，您将获得 1 个。

Answer 2

我可能会有像 GetArchiveReport(int prodID, int lmonth, int lyear, IEnumerable<int> itemIDs) 这样的函数，它会在您的查询中执行 itemIDs.Contains(tbl.ID)

var SelectedReports = _ProductRep.GetArchiveReport(product.Prod_ID, lmonth, lyear, litemList.Select(item => item.Item_ID));
foreach(var prodItem in SelectedReports)
{
  //Do code
}