Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > C/C++: Сети > Обработка комманд, полученных через сокеты


Автор: Chupik 5.4.2009, 21:46
Собственно, этот вопрос был в одном сабже ветки "Для новичков", но мне посоветовали обратится сюда. smile 

Итак, пишу что-то вроде небольшого бота для дистанционного управления компьютером.

Вот его клиентская часть:

Код

#pragma comment( lib, "wsock32.lib" )
#include<stdio.h>
#include<iostream>
#include<winsock2.h>
using namespace std;
void main()
{
    WORD wVersionRequested;
    WSADATA wsaData;
    int err;
    wVersionRequested = MAKEWORD( 2, 2 );
    err=WSAStartup( wVersionRequested, &wsaData );
    if ( err != 0 ){return;}
while (true)
{
    SOCKET s=socket(AF_INET,SOCK_STREAM,0);
    sockaddr_in dest_addr;
    dest_addr.sin_family=AF_INET;
    dest_addr.sin_port=htons(1280);
    dest_addr.sin_addr.s_addr=inet_addr("127.0.0.1");
    connect(s,(sockaddr *)&dest_addr,sizeof(dest_addr));
    
    char buf[100];
    char pol[100];
    cout<<"Enter the string:"<<endl;
    fgets(buf,sizeof(buf),stdin);
    send(s,buf,100,0);
    if (recv(s,pol,sizeof(buf),0)!=0)
    {
        cout<<"Received string:"<<endl<<pol<<endl;
    }
    closesocket(s);
}
WSACleanup();
}


Вот серверная:

Код

#pragma comment( lib, "ws2_32.lib" )
#include <iostream>
#include <string>
#include <windows.h>
using namespace std;
DWORD WINAPI ThreadFunc(LPVOID client_socket)
{
    SOCKET s2=((SOCKET *) client_socket)[0];
    char buf[100];
    char res[100], *z;
     strcpy(res, "");     
     std::string receive_string;
    while(recv(s2,buf,sizeof(buf),0))
    {
        receive_string = buf;
        int n=0,k=0;
        k=strlen(buf)-1;
        if (k>=2)
        {
            z=buf;
            sscanf(z+=n," %s%n",res, &n);
            cout<< "Buf. value:" << buf <<endl;
            send(s2,res,100,0);
        }
        else if ( receive_string == "/still_alive" ) {
        send(s2,"Hello, world! Test.",100,0);
    }
        else
            send(s2,"The string lenght is less than 2 symbols",100,0);
    }
    closesocket(s2);
    return 0;
}
int numcl=0;
//************************************************************
void print()
{
    if (numcl) printf("%d client connected\n",numcl);
    else printf("No clients connected\n");
}
//************************************************************
void main()
{
    WORD wVersionRequested;
    WSADATA wsaData;
    int err;
    wVersionRequested = MAKEWORD( 2, 2 );
    err = WSAStartup( wVersionRequested, &wsaData );
    if ( err != 0 ){return;}
    SOCKET s=socket(AF_INET,SOCK_STREAM,0);
    sockaddr_in local_addr;
    local_addr.sin_family=AF_INET;
    local_addr.sin_port=htons(1280);
    local_addr.sin_addr.s_addr=0;
    bind(s,(sockaddr *) &local_addr,sizeof(local_addr));
    int c=listen(s,5);
    cout<<"Server receive ready"<<endl;
        SOCKET client_socket;    
    sockaddr_in client_addr;
    int client_addr_size=sizeof(client_addr);
    while((client_socket=accept(s,(sockaddr *)&client_addr, &client_addr_size)))
    {
      numcl++;   
      print();
            DWORD thID;// thID идентификатор типа DWORD
      CreateThread(NULL,NULL,ThreadFunc,&client_socket,NULL,&thID);
    }
    
}


Проблема в том, что сервер никак не распознаёт команду:

 
Код

else if ( receive_string == "/still_alive" ) {
        send(s2,"Hello, world! Test.",100,0); }


т.е. когда я отправляю /still_alive он мне возращает не Hello, world! Test, а как обычно мою посланную команду(/still_alive).

В чём загвоздка? 

Заранее огромное спасибо. smile 

P.S. Поиск по сайту уже использовал... Гугл тоже.

Автор: Irdis 6.4.2009, 00:57
Код

 char buf[100];
k=strlen(buf)-1;
        if (k>=2)


99>=2 я бы с этим спорить не стал,

Автор: vinick 6.4.2009, 10:56
TCP - потоковый протокол, нет никаких гарантий, что за один вызов recv ты получишь ровно одно сообщение. Нужен внутренний  накопительный буфер.

Добавлено через 10 минут и 17 секунд
Код

    cout<<"Enter the string:"<<endl;
    fgets(buf,sizeof(buf),stdin);
    send(s,buf,100,0);


Ты читаешь строку, например в 10 байт, а на сервер отсылаешь 100.  Сервер получает лишний мусор.
И еще, пройдись отладчиком по серверу, посмотри чему у тебя равен receive_string. Возможно туда этот мусор и попадает.

Автор: Chupik 6.4.2009, 12:30
Irdis, vinick, большое спасибо, сейчас прогоню в отладчике. 

Автор: Chupik 6.4.2009, 12:55
Итак, результаты прогона на отладчике:
Поток: buf
Значение: ¬.6
Поток: receive_string 
Значение: {}

Поток: buf
Значение: tfgf (как раз то, которое я ввёл в клиенте)
Поток: receive_string
Значение: {""}.

Вот так вот, что-то со  receive_string непонятное...


Добавлено позже:

Решил по другому немного попробовать:

 
Код

else if ( strcmp(buf, "/test" ) == 0 ) {
        send(s2,"Hello, world! Test.",100,0);
        cout << "Test test test i td" << endl;
    }


тоже не срабатывает...

Добавленно позже:

А вот результаты последнего прогона:

значение buf  "still_alive"
значение receive_string  {"still_alive"}

Автор: Chupik 8.4.2009, 19:56
Up... smile 

Итак я понял, что кажись простым конвертированием из char * в string или int ничего не получится...

 
Код

std::string receive_string;
 receive_string = buf;


receive_string получается тоже массивом(слово test выглядит [1] = t; [2] = e; [3] = s и т.д...) и при сравнении:

Код

else if (receive_string == "/test") {
        send(s2,"Hello, world! Test.",100,0);
        cout << "GlaDOS mod on..." << endl;
    }


или:

Код

char buf[100];
char ppc[100] = "/test";
.....
else if (buf == ppc) {
        send(s2,"Hello, world! Test.",100,0);
        cout << "GlaDOS mod on..." << endl;
    }


ноль эмоций... smile 


P.S. Сори за 3 пост подряд...

Автор: jonie 8.4.2009, 20:18
конечно, тыже сравниваешь значения указателей, а не сами данные. Используй memcmp или для строк strcmp функции.

Автор: Chupik 8.4.2009, 20:32
Цитата(jonie @  8.4.2009,  20:18 Найти цитируемый пост)
Используй memcmp


Код

char buf[100];
char ppc[100] = "/test";
int result;
result=memcmp(buf,ppc,100);
****
else if (result = 0) 
****


Тоже самое.. Пишу /test - реагирует по прежнему...

Цитата(jonie @  8.4.2009,  20:18 Найти цитируемый пост)
strcmp функции. 


Уже использовал, на два поста выше:

Код

else if ( strcmp(buf, "/test" ) == 0 )


тоже не работает smile 

Автор: vinick 8.4.2009, 20:59
Цитата(Chupik @  5.4.2009,  21:46 Найти цитируемый пост)
 
k=strlen(buf)-1;
        if (k>=2)
        {
            z=buf;
            sscanf(z+=n," %s%n",res, &n);
            cout<< "Buf. value:" << buf <<endl;
            send(s2,res,100,0);
        }
        else if ( receive_string == "/still_alive" ) {
        send(s2,"Hello, world! Test.",100,0);


Пример:
buf == "/still_alive"
k = strlen(buf)-1 == 11;
Выполняется условие k>=2 и под else просто никогда не войдет.

И не пихай размер в 100 байт во все функции.

Автор: Chupik 8.4.2009, 21:32
Спасибо smile Пофиксил, сделал так:
Код

...................
   char buf[10];
    char res[10], *z;
     strcpy(res, "");     
     std::string receive_string;
    while(recv(s2,buf,sizeof(buf),0))
    {
        receive_string = buf;
        int n=0,k=0;
        
         if ( strcmp(buf, "/test" ) == 0 ) {
        send(s2,"Hello, world! Test.",10,0);
        cout << "GlaDOS mod on..." << endl;
    }
         else {
            z=buf;
            sscanf(z+=n," %s%n",res, &n);
            cout<< "Buf. value:" << buf <<endl;
            send(s2,res,10,0);   }
    }
    closesocket(s2);
    return 0;
................

т.е. размеры буферов уменьшил, убрал проверку на наличие символов, но... на команду /test не отзывается как надо...  smile 

З.Ы. Простите, что задаю много глупых вопросов. smile 

Автор: vinick 8.4.2009, 22:17
Цитата(Chupik @  8.4.2009,  21:32 Найти цитируемый пост)
на команду /test не отзывается как надо... 

у тебя в клиенте fgets читает кроме строки "/test" еще и символ перевода строки и дописывает его в buf, который ты отправляешь на сервер.

На сервере у тебя в buf на самом деле лежит "/test\r\n\0" или "/test\n\0", поэтому strcmp(buf,"/test") не работает как ожидалось.


Цитата(Chupik @  8.4.2009,  21:32 Найти цитируемый пост)
размеры буферов уменьшил, 

Я не говорил, что размеры буферов надо уменьшать, я имел ввиду что надо писать реальные размеры строк, а не 100 байт во всех функциях указывать. Теперь у тебя вобще бардак с памятью.

Вместо 
Код

 send(s2,"Hello, world! Test.",10,0); /// <- !!!!
 ...
 send(s2,res,10,0);  


надо 
Код

send(s2,"Hello, world! Test.",20,0); 
 ...
 send(s2,res,strlen(res)+1,0);  

и в других местах так же.

Автор: Chupik 19.4.2009, 21:00
vinick, спасибо, понял свои ошибки. smile 

Автор: J0ker 22.4.2009, 00:32
писание програм методом тыка никогда и ни у кого не получалось

Автор: GrayCardinal 23.4.2009, 09:37
Цитата

писание програм методом тыка никогда и ни у кого не получалось 

Не скажите... Метод самое то...  smile 

Заработало хоть ?

Автор: Chupik 23.4.2009, 21:50
Цитата(GrayCardinal @  23.4.2009,  09:37 Найти цитируемый пост)
Заработало хоть ? 

Пока искал решение проблемы, в гугле нашёл компонент icq для делфи и сделал бота на нём... smile 

Думаю, потом доделаю этот проект(который на с++).

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)