5

是否可以访问 std::for_each 迭代器,所以我可以使用 lambda 从 std::list 中擦除当前元素(如下所示)

typedef std::shared_ptr<IEvent>    EventPtr;
std::list<EventPtr> EventQueue;
EventType evt;
...

std::for_each( 
    EventQueue.begin(), EventQueue.end(),

    [&]( EventPtr pEvent )
    {
        if( pEvent->EventType() == evt.EventType() )
            EventQueue.erase( ???Iterator??? );
    }
);

我读过关于使用 [](typename T::value_type x){ delete x; } 这里关于 SO,但 VS2010 似乎不喜欢这个声明(下划线 T 作为错误源)。

4

3 回答 3

8

您使用了错误的算法。使用remove_if

EventQueue.remove_if([&](EventPtr const& pEvent)
{
    return pEvent->EventType() == evt.EventType();
});

STL 算法不允许您访问用于迭代的迭代器。在大多数情况下,这是一件好事。

(此外,请考虑您是否真的想使用std::list;它不太可能是您的用例的正确容器。考虑std::vector,您将使用擦除/删除习惯用法来删除满足特定谓词的元素。)

于 2012-06-28T14:29:51.793 回答
1

不,请使用常规 for 代替。

for( auto it = EventQueue.begin(); it != EventQueue.end(); ++it )
{
  auto pEvent = *it;
  if( pEvent->EventType() == evt.EventType() )
      it = EventQueue.erase( it );
);
于 2012-06-28T14:26:16.167 回答
0

擦除不是您可能需要了解 lambda 中的迭代器的唯一时间。为了以更一般的方式执行此操作,我使用 & 运算符(隐式转换为迭代器),如下所示:

int main (int argc, char* argv []) {
  size_t tmp [6] = {0, 1, 2, 3, 4, 5};
  std::list<size_t> ls ((size_t*)tmp, (size_t*) &tmp [6]);
  //printing next element
  std::for_each ((const size_t*)tmp, (const size_t*) &tmp [5], [] (const size_t& s) {
    std::cout << s << "->";
    std::cout << *(&s +1) << "   ";
  });
  std::cout << std::endl;
}
于 2019-02-13T14:59:47.340 回答