我正在尝试这样做:
for ( std::list< Cursor::Enum >::reverse_iterator i = m_CursorStack.rbegin(); i != m_CursorStack.rend(); ++i )
{
if ( *i == pCursor )
{
m_CursorStack.erase( i );
break;
}
}
但是erase需要一个迭代器，而不是反向迭代器。有没有一种方法可以将反向迭代器转换为常规迭代器，或者另一种方法可以从列表中删除该元素?
当前回答
如果m_CursorStack是一个向量，你可以通过取index来擦除:
m_CursorStack.erase(m_CursorStack.begin() + m_CursorStack.size() + int(m_CursorStack.rbegin() - i) - 1);
其他回答
下面这段代码将擦除的结果转换回反向迭代器，以便在反向迭代时擦除容器中的元素。有点奇怪，但它即使在擦除第一个或最后一个元素时也能工作:
std::set<int> set{1,2,3,4,5};
for (auto itr = set.rbegin(); itr != set.rend(); )
{
if (*itr == 3)
{
auto it = set.erase(--itr.base());
itr = std::reverse_iterator(it);
}
else
++itr;
}
只是想澄清一些事情:在上面的一些评论和回答中，erase的便携版本被提到为(++i).base()。然而，除非我遗漏了一些东西，否则正确的语句是(++ri).base()，这意味着你“增加”reverse_iterator(而不是迭代器)。
昨天我遇到了类似的事情，这篇文章很有帮助。谢谢每一个人。
typedef std::map<size_t, some_class*> TMap;
TMap Map;
.......
for( TMap::const_reverse_iterator It = Map.rbegin(), end = Map.rend(); It != end; It++ )
{
TMap::const_iterator Obsolete = It.base(); // conversion into const_iterator
It++;
Map.erase( Obsolete );
It--;
}
．.． 或者从列表中删除该元素的另一种方法?
这需要-std=c++11标志(用于auto):
auto it=vt.end();
while (it>vt.begin())
{
it--;
if (*it == pCursor) //{ delete *it;
it = vt.erase(it); //}
}
如果你不需要在运行过程中删除所有内容，那么要解决这个问题，你可以使用erase-remove习语:
m_CursorStack.erase(std::remove(m_CursorStack.begin(), m_CursorStack.end(), pCursor), m_CursorStack.end());
remove将容器中与pCursor匹配的所有项交换到最后，并返回指向第一个匹配项的迭代器。然后，使用范围的擦除将从第一个匹配擦除，直到结束。不匹配元素的顺序保持不变。
如果您正在使用std::vector，这可能会更快，其中在内容中间擦除可能涉及大量复制或移动。
当然，上面解释使用reverse_iterator::base()的答案是有趣的，值得了解，为了解决所述的确切问题，我认为std::remove更合适。
