2

可能重复：
调用 erase() 后 std::map::iterator 出现问题

也许你可以用我目前遇到的问题来启发我。所以问题是当我试图擦除地图中的一些元素时，我得到了一个糟糕的访问内存。我们假设以下 typedef ：

typedef std::map < std::string *, Document *, pStringCompare > Map;

我们假设在执行以下所有代码之前，我们已经实例化了一个包含两个元素（例如）的映射。此代码运行良好：

Map::iterator it = documents.begin();

std::string *s = it->first;
Document *d = it->second;

documents.erase(it);

delete d;
delete s;

但是当我试图用迭代器循环时，我得到了错误。

for (Map::iterator it = documents.begin() ; it != documents.end() ; it++)
{
    std::string s = * ( it->first);
    Document dd = * (it->second);
    std::cout << s << " || " << dd;
    documents.erase(it);  // This line causes the bad access memory error.
}

谢谢你的帮助 ！真的很感激！

阿列克斯

4

2 回答 2

6

您正在访问无效的迭代器。一旦你将它传递给erase()它就不再有效，所以你不能在你的for循环中增加它。有关详细信息，请参阅迭代器无效规则。

您的循环应该像这样构造以避免访问无效的迭代器：

for (Map::iterator it = documents.begin() ; it != documents.end() ; )
{
    std::string s = * ( it->first);
    Document dd = * (it->second);
    std::cout << s << " || " << dd;
    documents.erase(it++);
}

虽然我真的建议您阅读更多文档并了解这里发生了什么。例如，看到这个。

祝你好运！

于 2012-11-09T19:11:28.040 回答
1

Alternatively:

while ((Map::iterator iter = documents.begin()) != documents.end())
{
   std::string s = * ( iter->first);
   Document dd = * (iter->second);
   std::cout << s << " || " << dd;
   documents.erase(iter);
}
于 2012-11-09T19:27:32.257 回答