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);

你无法在RemoveAll上运行IEnumerable，所以我认为在这里使用Linq是一个好主意。您将拥有没有UserId=2的项目集合，相当于删除UserId=2的所有项目。您需要从RemoveAll撤消查询，并且无论如何都会有效。

Answer 2

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

首先，您使用ToList()创建列表，然后致电RemoveAll()以删除该列表中的某些元素，但由于您尚未将该列表存储在任何位置，因此您只是将要删除的物品数量。

Answer 3

您必须在RemoveAll上致电alert.UserAlerts，因为ToList会创建新的收藏。
如果您删除该集合的所有项目，则不会更改UserAlerts。

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

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

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

List <t> RemoveAll（）不会删除项目

3 个答案: