List<T> RemoveAll() 没有删除项目

Question

我有一个看起来像这样的对象：

{
  "Text" : "Another lovely alert",
  "Category" : 2,
  "UserAlerts" : [{ "UserId" : 2 }]
}

这被传递到 Web API 并正确绑定到：

[Key, Column(Order = 0)]
public long UserId { get; set; }

[Key, Column(Order = 1)]
public Guid AlertId { get; set; }

public virtual User User { get; set; }

public virtual Alert Alert { get; set; }

然后我运行以下命令，期望 ID 为 2 的 UserAlert 被删除，但对象没有改变。

alert.UserAlerts.ToList().RemoveAll(x => userIds.Contains(x.UserId));

alert.UserAlerts.ToList().RemoveAll(x => x.UserId == 2);

第二个查询更简单，但这也无济于事。我不知道我哪里出错了。

Answer 1

这是因为通过使用ToList，您正在创建新的 List 对象，然后从中删除项目，而不是从原始 IEnumerable 中删除。

试试这个：

alert.UserAlerts = alert.UserAlerts.Where(x => x.UserId != 2);

你不能在IEnumerable 上运行RemoveAll，所以我认为在这里使用Linq 是个好主意。您将收集到没有UserId=2 的项目，这相当于删除所有具有UserId=2 的项目。您需要从RemoveAll 反转您的查询，它在任何情况下都可以工作。

Answer 2

这当然应该删除元素，但我看不出你怎么会注意到它是否有？

首先，您使用ToList() 创建列表，然后调用RemoveAll() 从该列表中删除一些元素，但是由于您没有将该列表存储在任何地方，您只会得到删除的项目数返回。

Answer 3

您必须在alert.UserAlerts 上调用RemoveAll，因为ToList 会创建一个新集合。
如果您删除该集合的所有项目，它不会更改 UserAlerts。

alert.UserAlerts.RemoveAll(x => userIds.Contains(x.UserId));

更新：
如果UserAlerts 不是List，请使用Where 扩展方法（正如wudzik 在他的回答中所说）：

alert.UserAlerts = alert.UserAlerts.Where(x => !userIds.Contains(x.UserId));