0

在测试这段代码时，我发现这个回显服务器可以很好地处理一个单词。如果尝试回显一个句子服务器也会回显它，但之后它会用前一个回显的句子最后一个单词来回显输入。我使用 memset 清除了缓冲区，但仍然遇到同样的问题。无法理解问题出在哪里。

服务器代码

#include<stdio.h>
#include<string.h>  //strlen
#include<sys/socket.h>
#include<arpa/inet.h>   //inet_addr
#include<unistd.h>  //write

int main(int argc , char *argv[])
{
    int socket_desc , client_sock , c , read_size;
    struct sockaddr_in server , client;
    char client_message[2000];

    //Create socket
    socket_desc = socket(AF_INET , SOCK_STREAM , 0);
    if (socket_desc == -1)
    {
        printf("Could not create socket");
    }
    puts("Socket created");

    //Prepare the sockaddr_in structure
    server.sin_family = AF_INET;
    server.sin_addr.s_addr = INADDR_ANY;
    server.sin_port = htons( 8888 );

    //Bind
    if( bind(socket_desc,(struct sockaddr *)&server , sizeof(server)) < 0)
    {
        //print the error message
        perror("bind failed. Error");
        return 1;
    }
    puts("bind done");

    //Listen
    listen(socket_desc , 3);

    //Accept and incoming connection
    puts("Waiting for incoming connections...");
    c = sizeof(struct sockaddr_in);

    //accept connection from an incoming client
    client_sock = accept(socket_desc, (struct sockaddr *)&client, (socklen_t*)&c);
    if (client_sock < 0)
    {
        perror("accept failed");
        return 1;
    }
    puts("Connection accepted");

    //Receive a message from client
    memset(client_message, 0, sizeof(client_message));
    while( (read_size = recv(client_sock , client_message , 2000 , 0)) > 0 )
    {


        //Send the message back to client
        write(client_sock , client_message , strlen(client_message));
        memset(client_message, 0, sizeof(client_message));
    }

    if(read_size == 0)
    {
        puts("Client disconnected");
        fflush(stdout);
    }
    else if(read_size == -1)
    {
        perror("recv failed");
    }

    return 0;
}

客户端代码：

#include<stdio.h>   
#include<string.h>  
#include<sys/socket.h>  
#include<arpa/inet.h>   

int main(int argc , char *argv[])
{
    int sock;
    struct sockaddr_in server;
    char message[1000] , server_reply[2000];

    //Create socket
    sock = socket(AF_INET , SOCK_STREAM , 0);
    if (sock == -1)
    {
        printf("Could not create socket");
    }
    puts("Socket created");

    server.sin_addr.s_addr = inet_addr("127.0.0.1");
    server.sin_family = AF_INET;
    server.sin_port = htons( 8888 );

    //Connect to remote server
    if (connect(sock , (struct sockaddr *)&server , sizeof(server)) < 0)
    {
        perror("connect failed. Error");
        return 1;
    }

    puts("Connected\n");

    //keep communicating with server
    while(1)
    {
        printf("Enter message : ");
        //scanf("%s" , message);
    fgets(message, sizeof(message), stdin);

        //Send some data
        if( send(sock , message , strlen(message) , 0) < 0)
        {
            puts("Send failed");
            return 1;
        }

        //Receive a reply from the server
        if( recv(sock , server_reply , 2000 , 0) < 0)
        {
            puts("recv failed");
            break;
        }

        puts("Server reply :");
        puts(server_reply);

    memset(message, 0, sizeof(message));
    memset(server_reply, 0, sizeof(server_reply));
    }

    close(sock);
    return 0;
}

样本输出

Enter message : sample
Server reply :
sample

Enter message : sample test
Server reply :
sample test

Enter message : gamer
Server reply :
gamer
 test

Enter message : pro       
Server reply :
pro
r
 test
4

2 回答 2

2

你的问题是这一行：

write(client_sock , client_message , strlen(client_message));

您的客户不发送\0终结者。因此，如果您先发送一个长字符串，然后发送一个短字符串，那么短字符串将覆盖开头。将其更改为：

write(client_sock , client_message , read_size); // send exactly as much as you got

在客户端打印仅与收到的一样多：

ssize_t answer_size;
if((answer_size = recv(sock , server_reply , sizeof(server_reply) , 0)) < 0)
{
    puts("recv failed");
    break;
}

printf("Server reply :\n%.*s\n", answer_size, server_reply);

另外请记住，出于实现特定的原因，您可能会或不会收到与第一个一起发送的所有内容，recv建议循环运行它while（如果需要，使用非阻塞模式）。也一样send。

于 2013-09-09T19:25:25.113 回答
1

send()andreceive()函数都是阻塞的，只有在套接字关闭或处理完一个完整的缓冲区时才会返回。您的客户端代码定义message[1000]，而您的服务器代码在响应之前等待一个client_message[2000]数组。

除非您最终要使用select()异步 I/O 来恢复实际的网络级消息，否则您需要在消息前面加上一个长度，以允许您将读取限制为实际数据，或者使用终止符，然后逐个字符处理传入的数据。其中最直接的方法是在每次交换开始时将明确的消息长度包含在流中。

不正确的响应是因为您重用了发送缓冲区而不清除它。如果您memset(message, 0, 1000)在开头包含 a，while()您将避免该问题。

潜在的问题是，即使只有几个字符包含有价值的数据，您的客户端也会始终发送整个缓冲区。这对您的应用程序的影响将取决于网络速度、消息频率、景观中的参与方数量和主机容忍低效率的能力。

于 2013-09-09T19:32:48.927 回答