Сервер читает аргумент командной строки, переданный клиенту

Question

Я хочу передать аргумент клиенту, отправить его на сервер, а затем распечатать его там, в настоящее время я не получаю вывод.

Текущие коды:

Клиент:

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

#define PORT 8080 

int main(int argc, char const *argv[]) 
{ 
    int sock = 0, valread; 
    struct sockaddr_in serv_addr; 

    char buffer[1024] = {0}; 
    if ((sock = socket(AF_INET, SOCK_STREAM, 0)) < 0) 
    { 
        printf("\n Socket creation error \n"); 
        return -1; 
    } 

    serv_addr.sin_family = AF_INET; 
    serv_addr.sin_port = htons(PORT); 


    if(inet_pton(AF_INET, "127.0.0.1", &serv_addr.sin_addr)<=0)  
    { 
        printf("Invalid address \n"); 
        return -1; 
    } 

    if (connect(sock, (struct sockaddr *)&serv_addr, sizeof(serv_addr)) < 0) 
    { 
        printf("Connection Failed \n"); 
        return -1; 
    } 
      int i;
     if(argc>=2){
     for(i=0; i<argc; i++) {
    if(write( sock, argv[i], 1 + strlen( argv[i] ))<0)
    {
    printf("Error in write() ! %s\n", strerror(errno));
        return -1; 
    }
}
}
    return 0; 
} 

Сервер:

#include <unistd.h> 
#include <stdio.h> 
#include <sys/socket.h> 
#include <stdlib.h> 
#include <netinet/in.h> 
#include <string.h> 
#include<errno.h>
#define PORT 8080 
int main(int argc, char const *argv[]) 
{ 
    int server_fd, new_socket, valread; 
    struct sockaddr_in address; 
    int opt = 1; 
    int addrlen = sizeof(address); 
    char buffer[1024] = {0}; 
    char *hello = "Hello from server"; 


    if ((server_fd = socket(AF_INET, SOCK_STREAM, 0)) == 0) 
    { 
        perror("socket failed"); 
        exit(EXIT_FAILURE); 
    } 


    if (setsockopt(server_fd, SOL_SOCKET, SO_REUSEADDR , &opt, sizeof(opt))) 
    { 
        perror("setsockopt"); 
        exit(EXIT_FAILURE); 
    } 
    address.sin_family = AF_INET; 
    address.sin_addr.s_addr = INADDR_ANY; 
    address.sin_port = htons( PORT ); 


    if (bind(server_fd, (struct sockaddr *)&address,  
                                 sizeof(address))<0) 
    { 
        perror("bind failed"); 
        exit(EXIT_FAILURE); 
    } 
    if (listen(server_fd, 3) < 0) 
    { 
        perror("listen"); 
        exit(EXIT_FAILURE); 
    } 
    if ((new_socket = accept(server_fd, (struct sockaddr *)&address,  
                       (socklen_t*)&addrlen))<0) 
    { 
        perror("accept"); 
        exit(EXIT_FAILURE); 
    } 

valread = read(new_socket , buffer, 1024); 
puts("The client sent the value: ");
int* valuerec=(int *)buffer;
int val2=*valuerec;

    return 0; 
} 

В настоящее время я не получаю номер в качестве вывода, как я могу это исправить? В конечном итоге моей целью было бы отправить и получить несколько аргументов, но пока я отправляю только один.

Answer 1

следующий предложенный код:

1. безупречная компиляция
2. выполняет необходимые функции сервера
3. правильно проверяет ошибки
4. правильно обрабатывает ошибки
5. использует соответствующую сигнатуру для функции: main()
6. не включает заголовочные файлы, содержимое которых не используется
7. использует значимые имена, а не числа 'magi c'
8. использует соответствующий горизонтальный интервал для удобочитаемости
9. должным образом NUL завершает входной буфер
10. обратите внимание на функцию: atoi() не указывает, когда происходит ошибка, поэтому рекомендуем использовать strtol() Вместо этого
11. правильно очистил структуру address перед настройкой некоторых полей

и теперь предложенный код:

#include <unistd.h> 
#include <stdio.h> 
#include <sys/socket.h> 
#include <stdlib.h> 
#include <netinet/in.h>  



#define PORT 8080 
#define MAX_BUFFER_LEN 1025

int main( void ) 
{ 
    int server_fd, new_socket;

    struct sockaddr_in address; 

    if ( ( server_fd = socket( AF_INET, SOCK_STREAM, 0 ) ) < 0 ) 
    { 
        perror( "socket failed" ); 
        exit( EXIT_FAILURE ); 
    } 

    int opt = 1; 
    if ( setsockopt( server_fd, SOL_SOCKET, SO_REUSEADDR , &opt, sizeof(opt) ) ) 
    { 
        perror("setsockopt failed" ); 
        exit( EXIT_FAILURE ); 
    } 

    memset( &address, 0, sizeof( address ) );
    address.sin_family      = AF_INET; 
    address.sin_addr.s_addr = htonl( INADDR_ANY ); 
    address.sin_port        = htons( PORT ); 

    socklen_t addrlen = sizeof( address );
    if ( bind( server_fd, (struct sockaddr *)&address, addrlen ) < 0 ) 
    { 
        perror( "bind failed" ); 
        exit( EXIT_FAILURE ); 
    } 

    if ( listen( server_fd, 3 ) < 0 ) 
    { 
        perror( "listen failed" ); 
        exit( EXIT_FAILURE ); 
    } 

    if (( new_socket = accept(server_fd, (struct sockaddr *)&address,  
                       (socklen_t*)&addrlen ) ) < 0 ) 
    { 
        perror( "accept failed" ); 
        exit( EXIT_FAILURE ); 
    } 

    ssize_t valread; 
    char buffer[ MAX_BUFFER_LEN ] = { 0 };

    if( ( valread = read( new_socket, buffer, MAX_BUFFER_LEN -1 ) ) > 0 )
    {  // then read successful
        buffer[ valread ] = '\0';
        int value = atoi( buffer );
        printf( "The client sent the string: %s, which began with integer value: %d\n", buffer, value );
    }

    else if( ! valread )
    {
        puts( "client hungup" );
    }

    else
    {
        perror( "read failed" );
    }

    return 0; 
}