Question

因此，我是LINQ的新手，并试图弄清楚如何过滤项目。那是我的任务

public async Task<PagedList<Item>> GetItems (ItemParams itemParams) {
    var items = _context.Items.AsQueryable ();
    if (itemParams.CategoryId > 0) {
        var category = GetCategory (itemParams.CategoryId);

        items = items.Where (i => FilterItems (i, category.Result));

    }
    return await PagedList<Item>.CreatAsync (items, itemParams.PageNumber, itemParams.PageSize);
}

以及决定退回哪些项目的函数

static bool FilterItems (Item item, Category category) {

    if (item.CategoryId == category.Id) {
        return true;
    }
    if (category.Children.Count > 0) {
        foreach (Category cat in category.Children) {
          return  FilterItems (item, cat);
        }
    }
    return false;
}

获取类别功能

    public async Task<Category> GetCategory (int? id) {
        if (id == null) {
            return null;
        }
        var categories = _context.Categories.Include (x => x.Children).AsEnumerable ().Where (c => c.Id == id);
        categories = Traverse (categories);
        var category = await Task.FromResult (categories.First (c => c.Id == id));
        return category;
    }

Answer 1

您的过滤器将无法正常工作，因为foreach在第一个循环中返回。另外，FilterItems这个名称也不直观。

static bool ContainsItem(Category category, Item item)
{
    return
        category.Id == item.CategoryId ||
        category.Children.Any(c => ContainsItem(c, item);
}

由于C＃对|| Operator进行了短路评估，所以如果第一个匹配，则第二项将不会被评估。请注意，这不是一个肮脏的把戏，而是C＃规范的一部分。

Answer 2

从代码中看，如果项目属于当前类别或任何子类别，FilterItems()似乎返回true？

如果是这样，我认为您当前的代码有问题：对FilterItems()的递归调用会立即返回第一个子类别的结果，而无需检查其他子类别

这是修改后的代码：

public static bool InCategoryOrSubCategory(Item item, Category category)
{
    return item.CategoryId == category.Id ||
        category.Children.Any(subCategory => InCategoryOrSubCategory(item, subCategory));
}

我将名称更改为InCategoryOrSubCategory，以使其更清晰

这是一个咖喱版本，可以使Where的调用稍微好一点（但是该方法本身更加隐秘）：

public static Func<Item, bool> InCategoryOrSubCategory(Category category) 
{
    return item => 
        item.CategoryId == category.Id || 
        category.Children.Any(subCategory => InCategoryOrSubCategory(subCategory)(item));
}

用法：

items.Where(InCategoryOrSubCategory(category))

有没有更好的方式编写LINQ函数？

2 个答案: