2016-03-31 2 views
-4

Код не работает. Я искал день для решения, но на удивление ничего не получил. Когда я отправляю слово «привет» на сервер tcp, я бы хотел, чтобы он печатал текст «он работает». Проблема находится в строке 97. Я отредактирую заголовок сообщения, как только узнаю, что происходит.Сравнение двух строк/буферов в C возможно

#include<stdio.h> 
#include<string.h> //strlen 
#include<stdlib.h> //strlen 
#include<sys/socket.h> 
#include<arpa/inet.h> //inet_addr 
#include<unistd.h> //write 
#include<pthread.h> //for threading , link with lpthread 
void *connection_handler(void *); 

int main(int argc , char *argv[]) 
{ 
    int socket_desc , new_socket , c , *new_sock; 
    struct sockaddr_in server , client; 
    char *message; 

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

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

    //Bind 
    if(bind(socket_desc,(struct sockaddr *)&server , sizeof(server)) < 0) 
    { 
     puts("bind failed"); 
     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((new_socket = accept(socket_desc, (struct sockaddr *)&client, (socklen_t*)&c))) 
    { 
     puts("Connection accepted"); 

     //Reply to the client 
     message = "Hello Client , I have received your connection. And now I will assign a handler for you\n"; 
     write(new_socket , message , strlen(message)); 

     pthread_t sniffer_thread; 
     new_sock = malloc(1); 
     *new_sock = new_socket; 

     if(pthread_create(&sniffer_thread , NULL , connection_handler , (void*) new_sock) < 0) 
     { 
      perror("could not create thread"); 
      return 1; 
     } 

     //Now join the thread , so that we dont terminate before the thread 
     //pthread_join(sniffer_thread , NULL); 
     puts("Handler assigned"); 
    } 

    if (new_socket<0) 
    { 
     perror("accept failed"); 
     return 1; 
    } 

    return 0; 
} 
/* 
* This will handle connection for each client 
* */ 
void *connection_handler(void *socket_desc) 
{ 
    //Get the socket descriptor 
    int sock = *(int*)socket_desc; 
    int read_size; 
     char *message , client_message[2000]; 
     //char *contents; 
     //contents = "hello"; 
     //strcpy(mess,contents); 
    //Send some messages to the client 
    message = "Greetings! I am your connection handler\n"; 
    write(sock , message , strlen(message)); 

    message = "Now type something and i shall repeat what you type \n"; 
    write(sock , message , strlen(message)); 

    //Receive a message from client 
    while((read_size = recv(sock , client_message , 2000 , 0)) > 0) 
    { 
     write(sock , client_message , strlen(client_message)); 

Проблема ниже этой линии

 char mess[] = "hello\n"; 
     if(strcmp(client_message, mess) != 0){ 
      printf("it works"); 
     } 
    } 
    if(read_size == 0) 
    { 
     puts("Client disconnected"); 
     fflush(stdout); 
    } 
    else if(read_size == -1) 
    { 
     perror("recv failed"); 
    } 

    //Free the socket pointer 
    free(socket_desc); 

    return 0; 
} 
+3

Помещенный комментарий в строке, которая терпит неудачу и точно объясняет, в чем проблема. Невозможно пересчитать 97 строк. – sabbahillel

+0

Здесь мы идем с обычными подозреваемыми: неспособность правильно обработать результат, возвращенный recv(). Невозможность обработать потоковый поток TCP-потока и предположение, что TCP может передавать сам по себе сообщения размером более одного байта. Неправильное использование вызовов strlen() для буферов символов, которые не гарантируются с нулевым завершением. –

+0

Вы также предполагаете, что сообщение клиента завершено с NUL. –

ответ

3

strcmp возвращает ноль, если строки равны. Ваше состояние неверно.

Вы должны сделать if (! strcmp(...)) printf("it works"); вместо if (strcmp(...) != 0) ...

+0

Спасибо, что это было действительно. Я попробовал несколько утверждений, которые заявили, что если strcmp был равен 0, чтобы отобразить текст, он все равно не работал. Я вижу, что мое условие strcmp неверно выше, не знаю, как я этого не видел. но главная проблема заключалась в том, что strcmp не работал со сравнением двух строк. Кроме того, ваш ответ правильный. –

1

Top привет это "Привет"

Хотя привет вблизи линии 97 является "Привет \ п"

Либо:

1) редактировать один шлейфов

2) использование stricmp()

+0

__WHAT ??? __ Не могли бы вы объяснить это? – ForceBru

+0

@ForceBru: это достоверное наблюдение. Хотя я считаю, что это не проблема * только *, это, безусловно, хороший улов. – usr2564301

+0

Я не вижу, откуда приходит ** client_message **, но ** сообщение ** содержит «Привет», за которым следует пробел, ** mess [] ** содержит «привет», за которым следует строка перевода строки. Лучший способ получить совпадение - использовать ** strnicmp (client_message, mess, 5); ** –

1
while((read_size = recv(sock , client_message , 2000 , 0)) > 0) 

recv Linux

Все три функции возвращают длину сообщения об успешном завершении.

В этом случае вы не гарантируете, что client_message имеет символ окончания строки. Многие соединения будут отправлять только фактические символы сообщения, а не символ NULL в конце. Вы должны добавить (при условии, что это будет соответствовать)

client_message[read_size] = '\0'; 

этак strlen(client_message) будет несомненно верным. В качестве альтернативы вы можете использовать

write(sock , client_message , read_size); 

Поскольку вы повторяете сообщение, размеры одинаковы.

Обратите внимание, что если вы используете строку client_message в другом месте кода, вы должны убедиться, что символ '\ 0' был добавлен, чтобы быть в безопасности.

+0

Спасибо за подсказку :) –