带有 STL 映射和列表的指针

Question

我确实在网上对此进行了很好的搜索，并找到了很多关于向量的帮助，但在 STL 列表上却没有任何帮助。我正在尝试从地图中的列表中删除指针。 （有点棘手）。我真的很想使用指针而不是恢复到另一个（更糟糕的）做事系统。我将显示代码

bool RoomDB::removePlayer( int socketid, int roomid )   {

list<GamePlayer>::iterator itit;

 for( itit == roomlist[roomid].playerlist.begin(); itit != itit.end(); itit++ ) {
    if( itit.socketno == socketid )
    itit.erase( itit );
}

Answer 1

我觉得应该是这样的：

roomlist[roomid].playerlist.remove_if(
    [socketid](GamePlayer const & gp) { return gp.socketno == socketid; }
);

如果您没有 lambda，则必须自己编写一个小谓词，或者您使用手动方式；但现在要小心循环：

for (std::list<GamePlayer>::iterator it = roomlist[roomid].playerlist.begin();
     it != roomlist[roomid].playerlist.end();  /* no increment */ )
{
   if (it->socketno == socketid)
   {
     it = roomlist[roomid].playerlist.erase(it);
   }
   else
   {
     ++it;
   }
}

Answer 2

有很多事情是错误的。我将尝试列出所有这些，尽管我可能会错过一些：

1. 应该是list<GamePlayer*>::iterator itit;，如果如您所说，您的列表包含指针
2. 应该是 itit = roomlist[roomid].playerlist.begin();，而不是 itit = ...，就像 KerrekSB 说的那样
3. 应该是itit != roomlist[roomid].playerlist.end()，而不是itit != itit.end()
4. 应该是if( (*itit)->socketno == socketid )，而不是if( itit.socketno == socketid )（同样，如果您的列表包含指针）
5. 应该是roomlist[roomid].playerlist.erase(itit)，而不是itit.erase(itit)。