c++选择问题

Question

所以我正在尝试构建一个异步服务器...这是迄今为止我所拥有的内容的摘要：

int sockfd;
int max;
fd_set socks;
set<int> conns;

bind();
listen(sockfd);

while(1){
    FD_ZERO(&socks);
    max = sockfd;
    FD_SET(sockfd, &socks);
    for(set<int>::iterator it=conns.begin(); it!=conns.end(); it++){
        FD_SET(*it, &socks);
        if(max < *it){
            max = *it;
        }
    }

    int res = select(max+1, &socks, NULL, NULL, NULL);

    if(res < 0){
        cerr << "ERROR with select" << endl;
        break;
    }else if(res){
        if(FD_ISSET(sockfd, &socks)){
            //new connection
            int new_sockfd = accept();
            conns.insert(new_sockfd);
        }else{
            for(set<int>::iterator it=conns.begin(); it!=conns.end(); it++){
                if(FD_ISSET(*it, &socks){
                    char buffer[256];
                    read(buffer, 256, *it);
                    cout << buffer << endl;
                    close(*it);
                    conns.erase(*it);
                }
            }
        }
    }
}

最终发生的事情是...如果我连接客户端 1，然后连接客户端 2。然后我尝试使用 Client-2 然后 Client-1 发送数据...它有效...

但是，如果我连接 client-1 然后连接 client-2...然后尝试使用 client- 发送数据1. Select() 返回 -1...

有帮助吗？

Answer 1

查看 select 的手册页。重要的部分是：

在以下条件下，pselect() 和 select() 将失败并将 errno 设置为：
EBADF
一个或多个文件描述符集指定的文件描述符不是有效的打开文件描述符。
EINTR
该功能在任何选定事件发生之前和超时间隔到期之前被中断。

如果为中断信号设置了 SA_RESTART，则函数是否重新启动或返回 [EINTR] 由实现定义。
EINVAL
指定的超时间隔无效。
EINVAL
nfds 参数小于 0 或大于 FD_SETSIZE。
EINVAL
指定的文件描述符之一指的是从多路复用器下游链接（直接或间接）的 STREAM 或多路复用器。

errno 应该告诉你出了什么问题。

这只是一个问题，但是当您关闭连接时，您的文件描述符将变得无效。我猜 select 的错误应该是 EBADF

Answer 2

我认为你的代码从集合中删除是可疑的。一旦你调用conns.erase(*it)，你的迭代器就无效了（并且增加它会导致未定义的行为）。

将循环更改为如下所示应该可以解决问题：

for(set<int>::iterator it=conns.begin(); it!=conns.end();)
{
    set<int>::iterator cur = it++;
    if(FD_ISSET(*cur, &socks)){
        char buffer[256];
        read(buffer, 256, *cur);
        cout << buffer << endl;
        close(*cur);
        conns.erase(*cur);
    }
}