Question

我编写了一个C ++类的基本网络编程集（其中只有一个tcp_socket，一个udp_socket和一个ip_address包类类）。我在使用I / O多路复用服务器时遇到了一些麻烦。让我解释一下：

方案是这样的：

class base_socket
{
   //code for all sockets methods (both tcp and udp this class throws
   //errors and derived classes will cathc them.
   ~base_socket() { ::close(sock_fd); }
};

class udp_socket : public base_socket
{ 
  //code for the udp_socket
  virtual ~udp_socket();
};

class tcp_socket : public base_socket
{
  //code for the tcp_socket
  virtual ~tcp_socket();
};

在此计划中，根据应用程序的上下文，我添加了一个不可告人的抽象程度：类似

class client_t : public tcp_socket
{
  //code for dependent client tasks
};

主程序代码是这样的

int main(int argc , char *[] argv)
{
  int maxfd;
  fd_set rset;
  std::vector<base_socket> clientsV;

while(1)
{     
  FD_ZERO( &rset);

    FD_SET( {/*listening socket fd*/, &rset);

    if( clientsV.size() > 0)
        maxfd = // socket fd with max value between sockets
    else 
        maxfd = //listen socket

    for (auto it = clientsV.begin() ; it != clientsV.end(); ++it)
        FD_SET( /*client's socket fd*/, &rset);

    select( maxfd+1, &rset, NULL, NULL, NULL) < 0);

    if( FD_ISSET( /*listeing_socket*/, &rset)) 
    { 
       client_t * newclient = new client_t();
       listening_socket.accept(newclient);
       newClient->send_message("HELO");
       clientsV.push_back(*newClient);
    }


  }    
}

这适用于第一个客户端但是当第二个客户端到来时它获得HELO响应，但在第二个clientsV.push_back（* newClient）上第一个连接被关闭（）。对于出了什么问题的任何想法？

Answer 1

您的clientsV成员应该是std::vector<base_socket*>（即保持指针引用）。如果需要将现有元素重新定位到其他区域，您有可能在client_t期间创建和销毁临时push_back对象（请参阅Why does vector::push_back and emplace_back call value_type::constructor twice?）。

发生这种情况时，会调用~client_t()并且套接字现已关闭。

此外，您有内存泄漏。您正在分配一个带有new的指针，但您存储了已取消引用的副本。您应该按原样存储指针（并在需要时管理delete）。

C ++ I / O多路复用服务器过早关闭连接

1 个答案: