Question

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

//------------------------------
//--------------- 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＆amp; 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++;    
}

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

1 个答案: