为什么puts()函数不能在客户端 - 服务器套接字代码中打印多个输出?

时间:2015-09-23 06:22:45

标签: c++ sockets puts

这是我的客户端 - 服务器代码,一次只能为一个用户工作,客户端可以发送任意数量的消息。

我在服务器上显示收到的消息及其回复(实际上是反向) - 以及“已发送消息”消息。

在客户端,我显示输入消息和服务器回复:“message got”

但我的代码中存在一个问题:当我尝试发送多条消息时,我在服务器端得到了正确的输出,但没有在客户端获得,这只是“消息得到”,但在此之后放置函数它没有将任何输出打印到std-out。

我尝试了很多东西,但我还没有找到办法。
是否有任何冲刷,或者我遗失了什么? 请告诉我。

server code::::::  
    #include <stdio.h>  
    #include <string.h>   
    #include <sys/socket.h>  
    #include <arpa/inet.h>     
    #include <unistd.h>   
    #include <stdlib.h>  
    int main(int argc , char *argv[])  
{  
     int socket_desc , client_sock , c , read_size;
    struct sockaddr_in server , client;
    char client_message[2000];


    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( 42969 );

    //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);
     while(1)
    {
    //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");
    pid_t p=fork();
      if(p==0)
     {
       close(socket_desc);
    //Receive a message from client
    while( (read_size = recv(client_sock , client_message , sizeof(client_message) , 0)) > 0 )
    {
        puts(client_message);   
    char message[20000];
    int i,j=0;
    for(i=strlen(client_message)-1;i>=0;i--)
    message[j++]=client_message[i];
       message[j]='\0';
puts(message);
        //Send the message back to client
       if( send(client_sock , message , sizeof(message),0)>0)
                   puts("message sent");
       // memset(client_message,'\0',sizeof(client_message));
     //memset(message,'\0',sizeof(message));
    }

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




client code::::

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

int main(int argc , char *argv[])
{
    int sock;
    struct sockaddr_in server;
   // char message[1000] ,
    char 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 (42969 );

    //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 : ");
     char message[1000];
        scanf("%s" , message);
        //Send some data
        if( send(sock , message , sizeof(message) , 0) < 0)
        {
            puts("Send failed");
            return 1;
        }

        //Receive a reply from the server
        if( recv(sock , server_reply , sizeof(server_reply) , 0) > 0)
        {
            puts("message got");

        }

        puts("Server reply :");
        puts(server_reply);
fflush(stdout);
 //memset(server_reply,'\0',sizeof(server_reply));
     // memset(message,'\0',sizeof(message));
    }

    close(sock);
    return 0;
}

server code:::::: #include <stdio.h> #include <string.h> #include <sys/socket.h> #include <arpa/inet.h> #include <unistd.h> #include <stdlib.h> int main(int argc , char *argv[]) { int socket_desc , client_sock , c , read_size; struct sockaddr_in server , client; char client_message[2000]; 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( 42969 ); //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); while(1) { //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"); pid_t p=fork(); if(p==0) { close(socket_desc); //Receive a message from client while( (read_size = recv(client_sock , client_message , sizeof(client_message) , 0)) > 0 ) { puts(client_message); char message[20000]; int i,j=0; for(i=strlen(client_message)-1;i>=0;i--) message[j++]=client_message[i]; message[j]='\0'; puts(message); //Send the message back to client if( send(client_sock , message , sizeof(message),0)>0) puts("message sent"); // memset(client_message,'\0',sizeof(client_message)); //memset(message,'\0',sizeof(message)); } if(read_size == 0) { puts("Client disconnected"); fflush(stdout); } else if(read_size == -1) { perror("recv failed"); } close(client_sock); exit(1); } } close(socket_desc); return 0; } client code:::: #include<stdio.h> #include<string.h> #include<sys/socket.h> #include<arpa/inet.h> #include<stdlib.h> int main(int argc , char *argv[]) { int sock; struct sockaddr_in server; // char message[1000] , char 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 (42969 ); //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 : "); char message[1000]; scanf("%s" , message); //Send some data if( send(sock , message , sizeof(message) , 0) < 0) { puts("Send failed"); return 1; } //Receive a reply from the server if( recv(sock , server_reply , sizeof(server_reply) , 0) > 0) { puts("message got"); } puts("Server reply :"); puts(server_reply); fflush(stdout); //memset(server_reply,'\0',sizeof(server_reply)); // memset(message,'\0',sizeof(message)); } close(sock); return 0; }

2 个答案:

答案 0 :(得分:1)

此代码中存在不同的缺陷。

服务器端:

  • 你在一个循环中接受套接字client_sock,但只在子代中关闭它:你正在泄漏套接字描述符 - &gt;父母也应该close(client_sock);
  • 客户端发送最大大小为1000的消息,该服务器读取大小为2000的缓冲区。直到此处为止。但是你使用strlen找到有用的部分并发回2000字节(意味着你发送垃圾)。

客户方:

  • 你从stdin中读取一个字符串,其中scanf是一个缓冲区或大小为1000.好的。但是然后你发送1000字节,消息和垃圾。
  • 您没有留言信息。因此,第一条消息可能需要多次读取(实际上,如果您需要至少2个,因为您当前发送2000个字节并以1000块的形式读取它!)

您正在混合缓冲区大小,字符串长度和发送/接收的字节数。在服务器中,你应该使用接收到的字节数,并且永远不会输出,如果你知道消息是空的,从不输出超过它的长度。如果要交换多条消息,请使用分隔策略(以null(不常见)或\n结尾)发送第一个或包含大小的2个字节。

答案 1 :(得分:0)

您在服务器端代码中的这个地方遇到问题:

   if( send(client_sock , message , sizeof(message),0)>0)
               puts("message sent");

您必须替换:

   if( send(client_sock , message , strlen(message) + 1,0)>0)
               puts("message sent");