2

私はソケットプログラミングとCも初めてです。チュートリアルに従って、スレッドを使用して複数のクライアントをサポートする単純なエコー クライアント サーバー アプリケーションを作成しています。

アプリケーションでは、クライアントが無効なメッセージをエコーすることを除いて、すべて正常に機能します 。入力した最長のメッセージの最初の文字を置き換えているようです

Windows 7でvmwareを使用してFedora 14ターミナルで作業しています

これは、クライアントとサーバーをコンパイルして実行する方法です

サーバ

[sachith@localhost ~]$ gcc -pthread -o s server.c
[sachith@localhost ~]$ ./s
Socket created
bind done
Waiting for incoming connections...
Connection accepted

*クライアント // ここに問題があります *

[sachith@localhost ~]$ gcc client.c -o c
[sachith@localhost ~]$ ./c
Socket created
Connected

Enter message : Hello
Server reply :
Hello
Enter message : 123
Server reply :
123lo
Enter message : x
Server reply :
x23lo
Enter message : y
Server reply :
y23lo
Enter message :

以下は私のコードです

//サーバーコード

#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
    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));
    }

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

        //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);
    }

    close(sock);
    return 0;
}
4

2 に答える 2

1

受信バッファを memset するか、単語の最後の位置に '\0' を追加する必要があります。例:

これはあなたのコードです：

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, 0x00, 2000);
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));
}

または変更:

while( (read_size = recv(client_sock , client_message , 2000 , 0)) > 0 )
{
    *(client_message + read_size) = '\0';
    //Send the message back to client
    write(client_sock , client_message , strlen(client_message));
}
于 2013-03-28T02:46:44.923 に答える
0

私はこれが古い質問であることを知っています。もう少し簡単な解決策があることがわかります。

strlen を実行する必要はありません。データの長さはすでにわかっているため、read_size を再利用してください。

while( (read_size = recv(client_sock , client_message , 2000 , 0)) > 0 )
{
  //Send the message back to client
  write(client_sock , client_message , read_size);
}
于 2015-02-22T02:52:28.160 に答える