1

我编写了一个简单的 TCP 服务器来为许多客户提供服务（它与 telnet 配合得很好）。但是，我还想编写自己的简单 TCP 客户端，带有多路复用套接字和标准输入。与我照常写的服务器连接，连接代码没问题。

我对多路复用套接字和标准输入有问题。我的代码是最大的问题：

    void multiplexClient(FILE *fp, int sockfd, char inbuffer[], char outbuffer[])
    {
        int maxfd;
        fd_set rset;

        FD_ZERO(&rset);
        for(;;)
        {
            FD_SET(fileno(fp), &rset);
            FD_SET(sockfd, &rset);
            maxfd = std::max(fileno(fp), sockfd) + 1;

            int res = select(maxfd, &rset, NULL, NULL, NULL);

            if(res < 0)
            {
                printf("'select error\n");
            }

            if(FD_ISSET(sockfd, &rset))
            {
                if(read(sockfd, inbuffer, sizeof(inbuffer)) == 0)
                   printf("read error\n");
                printf("Received: %s\n", inbuffer);
                fflush(stdout);
                 memset(inbuffer, 0, sizeof(inbuffer));
            }

            else if(FD_ISSET(fileno(fp), &rset))
            {
                fprintf(stdout, "\n> ");
                if(fgets(outbuffer, sizeof(outbuffer), fp) == NULL)
                    return;
                write(sockfd, outbuffer, strlen(outbuffer));
                printf("Send: %s\n", outbuffer);
                fflush(stdout);
                 memset(outbuffer, 0, sizeof(outbuffer));
            }
        }
    }

// int main
// sockfd = socket...
// connect

while(1)
{
 multiplexClient(stdin, socket_fd, inbuffer, outbuffer);
}

// ...
    return 0;
}

我的第一个客户：

$ ./client localhost 9034
hello, other client!

> Send: hel
Received: hel
Received: lo! ��~�~���~�
Received: :)

Received: wha

我的第二个客户：

$ ./client localhost 9034
hello! :)

> Send: hel
whats is wrong with this chat?!

> Send: lo!

> Send:  :)

> Send: 


> Send: wha

如何解决这个问题？

4

2 回答 2

3

您的问题出在此处（稍作解释）：

read(sockfd, inbuffer, sizeof(inbuffer));
printf("Received: %s\n", inbuffer);

该read调用返回写入的字节数（您忽略）并且缓冲区不是C 字符串（即，它不是以空值结尾的）。

您应该拥有以下内容：

quant = read (sockfd, inbuffer, sizeof(inbuffer));
printf ("Received: %*.*s\n", quant, quant, inbuffer);
于 2012-12-14T12:13:07.253 回答
1

FD_ZERO()在将文件选择器重新添加到集合之前，您缺少一个。

此外，这是 C++，因为它使用std::max().

于 2012-12-14T12:09:23.177 回答