Linq技术问询：如何依据子属性值筛选父实体

嘿，你的代码里有几个明显的问题，咱们一步步捋清楚，然后给出正确的实现方式：

首先，先说说你原来代码的问题：

• 过早调用AsEnumerable()会把所有Market和关联的UserAlert都加载到内存里，数据量大的时候不仅慢还占内存，而且后续查询没法利用数据库的索引优化
• where markets.Any(...)这行逻辑错了：你应该检查当前Market自己的UserAlerts里有没有符合条件的项，而不是去遍历整个markets集合
• 直接用==比较两个集合是做引用对比，根本没法判断子集合里有没有关联当前用户的UserAlert

下面是两种靠谱的解决办法，优先选第一种：

方法一：让数据库帮你过滤（推荐，效率最高） #

只要你的实体导航属性配置正确（Market有ICollection<UserAlert> UserAlerts，UserAlert关联User），直接用LINQ to Entities写查询就行，EF会自动把它翻译成高效的SQL：

var markets = _context.Markets
    // 如果你需要把符合条件的UserAlert也加载到内存里，就加这行Include；如果只是筛选Market，不加也能正常过滤
    .Include(m => m.UserAlerts)
    // 核心逻辑：检查当前Market的UserAlerts里，有没有关联当前用户的项
    .Where(m => m.UserAlerts.Any(ua => ua.User == theUser))
    .ToList();

这里的Any()会被EF转成SQL的EXISTS子查询，数据库直接帮你完成筛选，比拉到内存里处理快多了。

方法二：一定要在内存里处理？（仅特殊场景用） #

如果因为某些特殊需求，必须先把数据加载到内存，那也得写对逻辑：

var markets = _context.Markets
    .Include(m => m.UserAlerts)
    .AsEnumerable() // 这一步才把数据拉到内存
    // 检查当前Market的UserAlerts里有没有目标用户的项
    .Where(m => m.UserAlerts.Any(ua => ua.User == theUser))
    .ToList();

⚠️ 注意：这种方式尽量少用，数据量一大就会卡爆！

关于Contains/Intersect的补充 #

你提到没法用这两个方法，其实是场景不对：如果已经提前拿到了当前用户的所有UserAlert，倒是可以这么写，但效率不如第一种方法：

var userAlerts = _context.UserAlerts.Where(ua => ua.User == theUser).ToList();
var markets = _context.Markets
    .Include(m => m.UserAlerts)
    .Where(m => m.UserAlerts.Intersect(userAlerts).Any())
    .ToList();

但这种方式需要先把UserAlert加载到内存，还是不如直接让数据库做筛选来得高效，所以优先选第一种方案~

内容的提问来源于stack exchange，提问作者Beau D'Amore