Question

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

但是，以下代码在第二次迭代中出现无效操作错误，因为_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--;
    }
}

通过List <t>删除不需要的对象的最简单方法是什么？</t>

3 个答案: