我正在尝试这样做:

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需要一个迭代器,而不是反向迭代器。有没有一种方法可以将反向迭代器转换为常规迭代器,或者另一种方法可以从列表中删除该元素?


当前回答

只是想澄清一些事情:在上面的一些评论和回答中,erase的便携版本被提到为(++i).base()。然而,除非我遗漏了一些东西,否则正确的语句是(++ri).base(),这意味着你“增加”reverse_iterator(而不是迭代器)。

昨天我遇到了类似的事情,这篇文章很有帮助。谢谢每一个人。

其他回答

请注意m_CursorStack。如果在for循环中使用erase((++i).base())可能是一个问题(见原始问题),因为它改变了i的值。正确的表达式是m_CursorStack.erase((i+1).base())

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--;
}

虽然在这里使用reverse_iterator的base()方法并对结果进行递减,但值得注意的是,reverse_iterators的状态与常规迭代器不同。一般来说,您应该更喜欢常规迭代器而不是reverse_iterators(以及const_iterators和const_reverse_iterators),原因正是如此。有关原因,请参阅多布斯医生杂志进行深入讨论。

经过更多的研究和测试,我找到了解决方案。显然,根据标准[24.4.1/1],i.base()和i之间的关系是:

&*(reverse_iterator(i)) == &*(i - 1)

(摘自多布斯博士的文章):

因此,在获取base()时需要应用偏移量。因此解决方案是:

m_CursorStack.erase( --(i.base()) );

EDIT

针对c++ 11进行更新。

Reverse_iterator I不变:

m_CursorStack.erase( std::next(i).base() );

Reverse_iterator I是高级的:

std::advance(i, 1);
m_CursorStack.erase( i.base() );

我发现这比之前的解清楚多了。你需要什么就用什么。

如果你不需要在运行过程中删除所有内容,那么要解决这个问题,你可以使用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更合适。