从控制器查询从存储库获得的List 增加耦合?

| 我有一个用C#编码的ASP.NET MVC应用程序。该应用程序的结构如下: 控制者 资料库 LINQ到实体(实体框架) 视图 我使用存储库(
_ProductRep
)查询LINQ to Entities,并将实际实体或
List<T>
而不是
IQueriables<T>
提供给Controller。 对于我毫无疑问的情况,我想提供一些帮助。我有以下代码:
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
} 
其中“ 6”是产品可能具有的所有可能物品的列表。 我想知道这个解决方案是否合理地增加了耦合(并违反了Demeter的定律)还是可以接受的,因为我实际上是在查询
List<T>
而不是
IQueriable<T>
。如果我错了,请指正我,但是我猜想,由于List不需要访问EF DataContext,因此Controller和EF之间没有耦合。 万一我错了,我唯一能想到的解决方案就是用Repository方法(仍然要实现)替换查询:
var lproductItem_monthlyReport_ProductRep.GetArchiveReport(product.Prod_ID, lmonth, lyear, item.Item_ID);
但是,使用此解决方案时,存储库在每个循环周期使用4个条件进行一个查询,而在以前的解决方案中,存储库仅使用一个条件进行查询。 您能在这个问题上给我启发吗?谢谢。 PS:循环中需要变量
lproduct_monthlyReport
lproductItem_monthlyReport
,我不能只使用其中之一 PPS:我知道我应该在控制器和存储库之间有一个业务服务层,这是我的下一步。     
已邀请:
从存储库返回列表会给您带来糟糕的性能,因为您失去了推迟执行的行为。基本上,您的存储库将把每条记录而不是相关实体检索到内存中,并将它们转换为列表,然后在内存中进行处理。如果要访问相关实体,则需要另一个数据库命中项。如果您坚持使用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。     
我可能会像这样的功能(ѭ14),在您的查询中执行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
}
    

要回复问题请先登录注册