c-为什么这个套接字/文件描述符分配无效

Question

我正在尝试用c编写一个简单的服务器，用于玩双人游戏。它检查传入连接，如果没有player1，它会保存player1的文件描述符（稍后用于发送和接收），如果没有player2，它也会这样做。我设置了这个循环，我从这里修改了它。我的问题是，我想从一个接收，然后发送到另一个，但我的作业似乎无效。当我尝试发送到player2时，它失败或发送垃圾。有时，发送到player1会发送回服务器（？）。我是否正确地使用了select并正确地循环使用了文件描述符集？如有任何反馈，我们将不胜感激。

// add the listener to the master set
FD_SET(listener, &master);
// keep track of the biggest file descriptor
fdmax = listener; // so far, it's this one
// main loop
while (1) {
    read_fds = master; // copy it
    if (select(fdmax+1, &read_fds, NULL, NULL, NULL) == -1) {
         error("select");
    }
    // run through the existing connections looking for data to read
    for(i = 0; i <= fdmax; i++) {
        //This indicates that someone is trying to do something
        if (FD_ISSET(i, &read_fds)) {
            if (i == listener) {
                addrlen = sizeof remoteaddr;
                newfd = accept(listener, (struct sockaddr *)&remoteaddr, &addrlen);
                if (newfd == -1) {
                    error("accept");
                } else {
                    FD_SET(newfd, &master);
                    if (newfd > fdmax) {
                        fdmax = newfd;
                    }
                    /* If we have the maximum number of players, we tell if that it's busy */
                    if (players >= 2) {
                         toobusy(fdmax); close(fdmax); FD_CLR(fdmax, &master);
                    }  else {                                                        
                         //Problem here?
                         if (player1_fd == -1) {
                               player1_fd = newfd;                                  
                         }
                         if ((player1_fd != -1) && (player2_fd == -1)) {
                               player2_fd = newfd;                                   
                         }
                         players++;
                         if (players == 2) {
                               sendhandles(); //says two players exist
                         }
                    }
                }
            } else {
                //Possible problems here
                if (i == player1_fd || i == player2_fd) {
                     receive(i); //Processes the messages
                }
            }
        }
    }
}

Answer 1

toobusy部分应该使用newfd，而不是fdmax。否则，在这个代码中就不会有容易发现的错误。

您的评论"有时，发送到player1会发送回服务器（？）"让我认为player1_fd和player2_fd可能未初始化，或者可能初始化为0而不是-1。您应该在循环之前仔细检查是否将它们设置为-1。

一些额外的注意事项：

• 是否确定master已初始化0？你打电话给FD_ZERO了吗
• 您应该使用FD_COPY将master复制到read_fds

最后，我建议使用库来处理事件，例如libevent或libev。