Question

我目前在我的应用程序中有这个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]);
        }

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

<ul class="treeview ui-accordion-content ui-helper-reset ui-widget-content ui-corner-bottom ui-accordion ui-widget ui-sortable ui-accordion-content-active">
@foreach (var categoryitem in Model.SubCategories.OrderBy(c => c.CategoryOrder))
{

    @if(!Model.Deleted)
    { 
        <li class="treelistitem" id="@Model.CategoryId">
            <div class="ui-accordion-header ui-state-default ui-corner-all ui-accordion-icons ui-sortable-handle first">
            <span class="clickable">
                <span class="ui-accordion-header-icon ui-icon treeviewicon treeviewplus"></span>
                <i class="glyphicon glyphicon-folder-open rightfolderpadding"></i><span class="categoryname">@Model.CategoryName</span>
            </span>
            </div>
           </li>
    }
}   
</ul>

在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的解决方案起作用时遇到了一些错误：

实体或复杂类型＆＃39; .Category＆＃39;不能在LINQ to Entities查询中构造

无法将类型隐式转换为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.

过滤包括LINQ和实体框架中的项目

3 个答案: