【问题标题】:Filtering include items in LINQ and Entity Framework [duplicate]过滤包括 LINQ 和实体框架中的项目 [重复]
【发布时间】:2023-03-08 08:06:01
【问题描述】:

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

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

我知道在 EF 中您还不能过滤 Included 项目,我可以编写一些 LINQ 代码来过滤掉不需要的 SubCategories...但是 LINQ 代码被转换为一个非常糟糕的 SQL -优化。我还可以编写一个存储过程来执行此操作(并编写比 LINQ 更好的查询),但我真的想使用纯 EF。

所以我有 2 个选项(除非有人可以看到其他选项)。

首先是遍历子类别,删除不需要的:

        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 的解决方案发挥作用时遇到了几个错误:

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

无法将类型隐式转换为 System.Collections.Generic.ICollection。存在显式转换(您是否缺少演员表?)

这一切都通过在 Select() 方法之前添加 .ToList() 解决了。

【问题讨论】:

  • 您确定您的解决方案是正确的吗?您添加的 ToList() 将从数据库中加载整个 Categories 表。
  • 正是@JoshMouch 所说的。它会起作用,因为您要返回整个表,然后在结果上使用 LinqToEntities 进一步过滤它。我也在尝试保存数据库负载:) 如果我找到更好的方法会报告

标签: c# linq entity-framework


【解决方案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);

【讨论】:

  • 我确实喜欢这个解决方案,但是,我在这一行得到一个错误:SubCategories = c.SubCategories.Where(sub => !sub.Deleted) .OrderBy(sub => sub.CategoryOrder) , 无法将类型“System.Linq.IOrderedEnumerable”隐式转换为“System.Collections.Generic.ICollection”。存在显式转换(您是否缺少演员表?)
  • @binks 然后将IEnumerable具体化为ICollection
  • 嗯...我尝试过强制转换 (ICollection)c.SubCategories.Where(sub => !sub.Deleted) .OrderBy(sub => sub.CategoryOrder),虽然那工作,然后我收到一个新错误:无法在 LINQ to Entities 查询中构造实体或复杂类型“.Category”。
  • 好的,但是使用 .ToList() 应该可以工作,因为它继承自 ICollection...但我仍然收到新错误:无法在 LINQ 中构造实体或复杂类型 '.Category'实体查询。
  • @binks 然后你不需要在投影中使用Category 类型。投影到匿名类型或新类型。
【解决方案2】:

我发现这种方式看起来更干净更短。不确定对数据库的影响

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

【讨论】:

    【解决方案3】:

    您在此处的问题是一个演示问题(仅显示未删除的类别)。这表明方法 2 是您的最佳选择。

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

    因此,我会推荐方法1。

    【讨论】:

      猜你喜欢
      • 2013-06-21
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      • 2021-10-30
      • 1970-01-01
      • 1970-01-01
      相关资源
      最近更新 更多