使用选择而不听()ing,可能

Using select without listen()ing, possible?

本文关键字:ing 可能 选择      更新时间:2023-10-16

我正在构建一个客户端:

  1. 应该能够从服务器和标准输入接收信息
  2. 应该能够在不询问的情况下从服务器接收信息,例如当另一个客户端发送消息时。

为此,我尝试使用 select 来监视两个可能的输入。

发生的情况是,当键盘输入被监控时,我向客户端发送一条消息,我希望返回一条消息,所以没有问题。但是当服务器发送意外消息时,没有任何反应,我不知道为什么。使用 select() 是正确的方法吗?甚至可以在不listen()的情况下使用select()吗?

这是我的代码(可编译):

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <errno.h>
#include <netdb.h>
#include <sys/types.h>
#include <netinet/in.h>
#include <sys/socket.h>
#include <cstring>
#include <arpa/inet.h>
#include <iostream>
#include <fstream>
#define MAX_CLIENT_NAME 30
#define MAX_TWIT_SIZE 140
#define NUM_OF_ARG 4
#define ERROR -1
#define GREAT_SUCCESS 0
#define OK "OK"
#define EXIT "EXIT"

using std::string;
using std::cerr;
using std::endl;
using std::cout;
string clientName;
int srverfd, numbytes, status, maxSock ;
fd_set inputFdSet;        /* Socket file descriptors we want to wake
                       up for, using select() */
int establishConnection(char * serverAddress,char * port){
    if ((srverfd = socket(AF_INET, SOCK_STREAM, 0)) == -1) {
        perror("socket");
        return ERROR;
    }
    struct sockaddr_in server;
    server.sin_family = AF_INET;
    inet_aton(serverAddress, &server.sin_addr);
    server.sin_port = htons(atoi(port));
    memset(&(server.sin_zero), '', 8);
    if (connect(srverfd,(const struct sockaddr *)&server,sizeof(struct sockaddr)) == -1) {
        perror("connect");
        close(srverfd);
        return ERROR;
    }
    maxSock = srverfd;
    return GREAT_SUCCESS;
}
const char * getUserTweet(){
    string temp;
getline(std::cin,temp);
    return temp.c_str();
}
void sendMessage(string message){
    if ((numbytes = send(srverfd, message.c_str(), message.length(), 0)) == -1) {
        perror("sendMessage");
        close(srverfd);
    }
    cout<<"Message sent: "<< message << endl;
    return;
}
const char * getMessage(){
    char buf[MAX_TWIT_SIZE];
    memset(buf,'',MAX_TWIT_SIZE);
    if ((numbytes = recv(srverfd, buf, 140, 0)) == -1) {
        perror("getMessage");
        close(srverfd);
    }
    string temp = buf;
    return temp.c_str();
}
void build_select_list() {
    FD_ZERO(&inputFdSet);
    FD_SET(srverfd,&inputFdSet);
    FD_SET(STDIN_FILENO,&inputFdSet);
    if (STDIN_FILENO > maxSock)
        maxSock = STDIN_FILENO;
    return;
}
void readSocket(fd_set tempfd) {
    const char * tweet, * inMessage;
    if (FD_ISSET(srverfd,&tempfd)) {
        inMessage = getMessage();
        cout << inMessage << endl;
    }
    if (FD_ISSET(STDIN_FILENO,&tempfd)) {
        tweet = getUserTweet();
        sendMessage(tweet);
        inMessage = getMessage();
        if (strcmp(inMessage,OK) != 0) {
            cout << inMessage << endl;
        }
        if (strcmp(inMessage,EXIT) == 0) {
            return;
        }
    }
    return;
}
int main (int argc, char *argv[] ){
    int value;
    bool clientON = false;
    if(establishConnection(argv[2],argv[3])){
        cerr << "usage: failed to make connection" << endl << "exiting..." << endl;
        exit(EXIT_FAILURE);
    }
    cout << "Connected successfully" << endl;
    sendMessage("CONNECT "+clientName); //Connect
    if(strcmp(getMessage(),OK) == 0){
        clientON = true;
    }
    while(clientON){
        build_select_list();
        value = select(maxSock, &inputFdSet, NULL, NULL, NULL);
        if (value < 0) {
            perror("select");
            exit(EXIT_FAILURE);
        }
        if (value == 0) {
            continue;
        }
        else {
            readSocket(inputFdSet);
        }
    }
    sendMessage("DISCONNECT");
    if(strcmp(getMessage(),OK) == 0){
        // do nothing
    }
    close(srverfd);
    return 0;
}

您的select调用无效。第一个参数必须是任何集中的最高文件描述符加 1
正如你所看到的,srverfd上的事件不会"唤醒"select调用(除非STDIN_FILENO以某种方式小于srverfd,在这种情况下,stdin事件不会解锁select - 但这在实践中不会发生)。


您的代码还有很多其他问题。(它看起来不像C++。

getUserTweet是不可靠的(未定义的行为 - 一旦函数返回temp就会被销毁,因此当调用者尝试使用它时,您返回char*已经消失了)。getMessage也一样。要解决此问题,请在任何地方使用 std::string,并且仅在调用 C 库函数时提取char*)。

readSocket不必要地复制 FD 集(可能很昂贵)。

你真的应该摆脱所有这些全局变量 - 构建一个或两个类来封装该状态和网络函数,或类似的东西。