如何使用结构擦除地图中的值？ [C++]

Question

我在一个小型 TCP 客户端-服务器中工作，我使用“map”和一个结构来存储客户端跟踪号、他的 ip 和他的套接字。 主要问题是，当我想从“地图”中删除他的数据（连接已关闭）时，它确实如此，但是当它出现 2 个或更多客户端时，服务器似乎出现延迟，后来，当 1 个或更多客户端关闭他的连接时，服务器应用程序停止响应（但接受连接）......可能在无限循环中停止？ 无论如何...这是我的代码：

//------------------------------
//--------------- main.h

struct ClientData{
    int sock;
    char IP [65];
};

class ServerNetwork
{
public:
    map<int, ClientData> MapClient;
...


//------------------------------
//--------------- ServerNetwork.cpp

void ServerNetwork::SendToAll(char * packets, int totalSize)
{
    SOCKET currentSocket;
    map<int, ClientData>::iterator iter;
    int iSendResult;
    if (MapClient.empty()){
        return;
    }
    for (iter = MapClient.begin(); iter != MapClient.end(); iter++ )
    {
        int currentID = iter->first;
        currentSocket = iter->second.sock;
        iSendResult = SendInfo(currentSocket, packets, totalSize);

        if (iSendResult == SOCKET_ERROR)
        {
            closesocket(currentSocket);
            printf("Closed Connection from Client # %d\n", currentID);
            RemoveClient(currentID);
            iter = sessions.begin();
            iter++;
            break;
        }
        iter++;
    }
}

void ServerNetwork::RemoveClient(int ClientID)
{
    MapClient.erase(ClientID);
    printf("Deleted!\n");
}

MapClient持有客户端ID+(ClientSock&ClientIP) 我正在使用 VisualStudio 2013。

谢谢！

Answer 1

您应该真正利用 Map::erase 在擦除后返回下一个迭代器这一事实。因此，将擦除重构为 removeclient 是一个坏主意。

iter=MapClient.begin()
while(iter != MapClient.end()
{
   ....
   if(...)
   {       
      closesocket(currentSocket);
      printf...
      iter = MapClient.erase(iter);
      continue; //do this to avoid the iter++ below
   }
   iter++;    
}