通过列表进行预告的最简单方法是什么 <T> 删除多余的对象？

Question

在我的应用程序中，_collection是一个List，我需要从中删除所有与条件不匹配的User对象 。

但是，以下代码在第二次迭代中获取无效操作错误，因为_collection本身已更改：

foreach (User user in _collection)
{
    if (!user.IsApproved())
    {
        _collection.Remove(user);
    }
}

我可以创建另一个List集合并来回复制它们，但后来我遇到了非克隆引用类型等问题。

有没有办法比将_collection复制到另一个另一个List变量更优雅？

Answer 1

_collection.RemoveAll(user => !user.IsApproved());

如果你还在 2.0：

_collection.RemoveAll(delegate(User u) { return !u.IsApproved(); });

顺便说一句，如果您不想触摸原始列表，您可以获得另一个已批准用户列表：

_collection.FindAll(user => user.IsApproved());

Answer 2

您始终可以从顶部索引开始并向下迭代到0：

for (int i = _collection.Count - 1; i >= 0; i--)
{
    User user = _collection[i];
    if (!user.IsApproved())
    {
        _collection.RemoveAt(i);
    }
}

不过，Mehrdad的回答看起来非常优雅。

Answer 3

只要有可能在循环中修改集合，请选择for循环。 Mehrdad给出的解决方案很可爱，绝对值得一试！

这是我在处理可修改集合时发现有用的代码：

for(int index=0;index < _collection.Count; index++)
{
    if (!_collection[index].IsApproved)
    {
        _collection.RemoveAt(index);
        index--;
    }
}