过滤包括LINQ和entity framework中的项目

我目前在我的应用程序中有这个LINQ / EF代码：

var rootCategoryItem = DatabaseContext.Categories .Include("SubCategories") .OrderBy(c => c.CategoryOrder) .Single(c => c.CategoryId == 1); 

我知道在EF中你不能过滤包含的项目，我可以编写一些LINQ代码来过滤掉不需要的子类别……但是LINQ代码被转换为一个非常不优化的可怕的SQL。 我还可以编写一个存储过程来执行此操作（并编写比LINQ更好的查询），但我真的想使用纯EF。

所以我留下了两个选项（除非有人可以看到其他选项）。

第一个是遍历子类别，删除不需要的子类别：

  var subCategoriesToFilter = rootCategoryItem.SubCategories.ToList(); for (int i = 0; i < subCategoriesToFilter.Count; i++) { if (subCategoriesToFilter[i].Deleted) rootCategoryItem.SubCategories.Remove(subCategoriesToFilter[i]); } 

第二种选择是在我看来这样做：

 
 @foreach (var categoryitem in Model.SubCategories.OrderBy(c => c.CategoryOrder)) { @if(!Model.Deleted) { 
 
   @Model.CategoryName  
 
 } }  

在2中，哪一个是最好的选择？ 或者我还有另一种选择吗？

解决方案

好吧，Servy是非常正确的，我必须修改他的答案才能使它工作：

  var rootCategoryItem = DatabaseContext.Categories .OrderBy(c => c.CategoryId) .ToList().Select(c => new Category() { SubCategories = c.SubCategories.Where(sub => !sub.Deleted).ToList(), //make sure only undeleted subcategories are returned CategoryId = c.CategoryId, CategoryName = c.CategoryName, Category_ParentID = c.Category_ParentID, CategoryOrder = c.CategoryOrder, Parent_Category = c.Parent_Category, Deleted = c.Deleted }).Single(c => c.CategoryId == 1); 

我试图让Servy的解决方案工作有几个错误：

无法在LINQ to Entities查询中构造实体或复杂类型“.Category”

无法将类型隐式转换为System.Collections.Generic.ICollection。 存在显式转换（您是否错过了演员？）

通过在Select（）方法之前添加.ToList（）来解决这个问题。

Answer 1

虽然您无法过滤通过Include的集合，但您可以使用Select并将该集合投影到过滤集合中。

 var rootCategoryItem = DatabaseContext.Categories .OrderBy(c => c.CategoryOrder) .Select(c => new Category() { SubCategories = c.SubCategories.Where(sub => !sub.Deleted) .OrderBy(sub => sub.CategoryOrder), c.CategoryId, c.CategoryName, //include any other fields needed here }) .Single(c => c.CategoryId == 1); 

Answer 2

我通过这种方式看起来更清洁，更短。 不确定数据库的影响

  var rootCategoryItem = DatabaseContext.Categories.SingleOrDefault(); if (rootCategoryItem == null) return null; { rootCategoryItem.Items = rootCategoryItem ?.Items.Where(x => !x.IsDeleted).ToList(); return rootCategoryItem; } 

Answer 3

您在此处的担忧是演示问题（仅显示未删除的类别）。 这表明方法2是您的最佳选择。

但是，我怀疑您需要经常在系统中使用未删除的类别。 这表明您应该有一个函数可以一致地返回未删除的类别，供您在任何需要的地方使用。

出于这个原因，我会建议方法1。