c ++迭代对象列表并删除对象

Question

我有一个对象列表，我想在某个对象达到 if 条件后删除它。但我在使用它时遇到问题。主要是因为 if 条件抛出了错误。

另外我不知道我是否需要创建一个临时文件夹值并让它成为我的 if 条件？老实说，我对迭代器有点困惑，任何额外的信息都可能会有所帮助。

void removeFolder(string fName)
{
    list<Folder> folders = this->getFolders();
    for (list<Folder> itr = folders.begin(); itr != folders.end();)
    {
        if (fName == *itr)
            itr = folders.erase(itr);
        else
            ++itr;
    }
}

Answer 1

我认为你的想法是正确的，但你做错了事。文件夹是：

list<Folder> folders

比您迭代不同的元素“文件夹”而不是“文件夹”：

for (list<folder> itr = folders.begin(); itr != folders.end();)

我觉得应该是这样的：

for (list<Folder>::iterator itr = folders.begin(); itr != folders.end();)

现在，一旦您迭代正确的对象，就可以进行有意义的比较，而不是字符串到对象：

if (fName == *itr)

而是将字符串与字符串进行比较，我假设您的文件夹类有一些方法来获取文件夹名称，即：

if (fName == (*itr).getFolderName())

Answer 2

我确定在某个地方已经回答了这个问题，但代码很简单：

您只需要使用std::list<Folder>::remove_if()。

void removeFolder(const std::string& fName)
{
    std::list<Folder>& folders = getFolders();
    folders.remove_if([&fName](Folder& item){return item.name == fName;});
}

正如其他人所指出的，我已尝试修复您的代码存在其他问题。特别是，您可能正在将某种name 成员与fName 变量进行比较。此外，无需将fName 字符串按值传递给removeFolder 函数，并且您可能需要修改的不是folders 的本地副本，而是函数外部存在的一些列表。