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)

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

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 ，但功能的一些列表现有的外线。