将多条记录添加到数据库时，内部联接的 LINQ 查询不起作用

Posted 2023-04-12

【中文标题】将多条记录添加到数据库时，内部联接的 LINQ 查询不起作用

【英文标题】：LINQ query with Inner Join not working when multiple records are added to database

【发布时间】：2021-06-26 02:43:38

【问题描述】：

我有一个函数需要返回除已被淘汰的用户之外的所有用户，我编写了以下查询

var userLists = await _repositoryContext.User
             Join(_repositoryContext.Eliminations,
             u => currentUser.userID,
             m => m.userOneId,
             (u, m) => new  Eliminations = m, Users = u )
            .Where(u_m => u_m.Users.userID != currentUser.userId && u_m.Eliminations.userTwoId != u_m.Users.userID).Select(u_m => u_m.Users).ToListAsync();

这就是用户表的样子

 userID: cf321a6a-70de-45c9-997d-de7d2be99a36
 name: test1
 email: [e-mail]
-------------------------------------------------------------
 userID: b1b2f69a-2100-499b-a0e6-42bbf87290ee
 name: test2
 email: [e-mail]
-------------------------------------------------------------
 userID: b2be45d2-f065-4321-842d-cc35d29c374f
 name: test3
 email: [e-mail]
-------------------------------------------------------------

当 Eliminations 表如下所示时，此查询可以正常工作

 userMatchID: 7b2bc383-a818-4662-8f62-4f407650da9e
 userOneWin: 1
 userTwoWin: 0
 userOneId: cf321a6a-70de-45c9-997d-de7d2be99a36
 userTwoId: b2be45d2-f065-4321-842d-cc35d29c374f

当test3 user 在消除表中时，查询工作正常并返回test2 user

 userID: b1b2f69a-2100-499b-a0e6-42bbf87290ee
 name: test2
 email: [e-mail]

但是当test2 user 被添加到淘汰表中时，如下所示

 userMatchID: 7b2bc383-a818-4662-8f62-4f407650da9e
 userOneWin: 1
 userTwoWin: 0
 userOneId: cf321a6a-70de-45c9-997d-de7d2be99a36
 userTwoId: b2be45d2-f065-4321-842d-cc35d29c374f
--------------------------------------------------------------
 userMatchID: b9be0fd3-7e4f-4311-901e-c28715ee13d1
 userOneWin: 1
 userTwoWin: 0
 userOneId: cf321a6a-70de-45c9-997d-de7d2be99a36
 userTwoId: b1b2f69a-2100-499b-a0e6-42bbf87290ee

查询失败并发送两个当前可用的用户，而不是发送一个空列表。看起来像这样

-------------------------------------------------------------
 userID: b1b2f69a-2100-499b-a0e6-42bbf87290ee
 name: test2
 email: [e-mail]
-------------------------------------------------------------
 userID: b2be45d2-f065-4321-842d-cc35d29c374f
 name: test3
 email: [e-mail]
-------------------------------------------------------------

但我想要的结果是一个没有记录的空列表，test1 user 在 Linq 查询中无论如何都被省略了。我在这里做错了吗？

【问题讨论】：

不清楚您要返回什么以及您期望什么。从我读到的关于您如何使用 Join 构建查询的内容来看，您似乎会返回两个对用户 ID 的引用：cf321a6a-70de-45c9-997d-de7d2be99a36，这可能不是您所期望的。

@StevePy 我已经用每个阶段的结果和预期结果更新了问题

【参考方案1】：

我怀疑这会起作用，因为这不是您在直接数据库中的做法。如果您有两个用户列表，则仅通过内部连接无法回答“这些用户中的哪些不在该列表中” - 将它们全部连接在一起并要求 this.userid != that.userid 将在“中只有一个用户”时起作用that" 列表，但只要添加两个，您就会从“this”列表中获取所有用户，因为每个用户都不等于该列表中的至少一个用户

This
A
B

That
A

只返回 B，因为 B != A，（并且 A == A）

This
A
B
C

That
A
B

返回 A 因为 A != B，返回 B 因为 B != A，返回 C 两次，因为 C != A 和 C != B

简而言之，不要为此使用内连接。看一个更像的模式

var elim1 = DbContext.Eliminations.Where(e => !e.UserOneWin).Select(e => e.UserOneId);
var elim2 = DbContext.Eliminations.Where(e => !e.UserTwoWin).Select(e => e.UserTwoId);
DbContext.Users.Where(u => !elim1.Any(e => e.UserOneId == u.UserId) && !elim2.Any(e => e.UserTwoId == u.UserId))

这应该映射到更像一个 sql

Users u
WHERE 
  NOT EXISTS (SELECT null FROM eliminations WHERE u.UserId = userOneId) AND 
  NOT EXISTS (SELECT null FROM eliminations WHERE u.UserId = userTwoId) 

EXISTS 比内部连接更适合回答这个问题。您也可以将用户内部连接到不同的赢家而不是输家。老实说，我不认为你的桌子设计在这里对你有任何好处。我将消除映射一个获胜用户 ID 和一个失败用户 ID，而不是一个用户 1 和一个用户 2，另一列表示用户是否赢了，另一列表示用户是否输了。在任何时候你想要“获胜者名单”或“失败者名单”时，你必须两次击中你的淘汰表，有效地将所有获胜者1为真的userOne合并到获胜者2为真的userTwo，这更尴尬不仅仅是有一个获胜者 ID 和一个失败者 ID 列

【讨论】：

我意识到当记录不断堆积时可能很难做到，所以我决定在代码库中添加一个函数来比较和删除列表中的用户