Фильтрация включает элементы в LINQ и Entity Framework

В настоящее время я использую этот код LINQ/EF в своем приложении:

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

Я знаю, что в EF вы еще не можете фильтровать включенные элементы, и я могу написать некоторый код LINQ, чтобы отфильтровывать SubCategories, которые не нужны... но код LINQ преобразуется в ужасающий SQL, -optimised. Я мог бы также написать сохраненный proc, который делает это (и писать гораздо лучший запрос, чем 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-решение работать:

Объект или сложный тип ".Category" не может быть сконструирован в запросе LINQ to Entities

Невозможно неявно преобразовать тип в System.Collections.Generic.ICollection. Явное преобразование существует (вы пропускаете листинг?)

Все это было разрешено путем добавления .ToList() перед методом Select().

Ответ 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);

Ответ 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.