2010-02-04 4 views
0
C++

Я пытаюсь написать оболочку приложения сервера, как я бы с любым приложением, и я искал в течение недели в течение, по крайней мере порядочное руководства или учебника по асинхронным сокетам (это обертка должна быть асинхронными) и до сих пор, что я мог бы сделать это:Асинхронного обертка Winsock сервера, CPU отстающий -

#ifndef _SERVER_H 
#define _SERVER_H 

#include "asynserv.h" // header file with the important lib includes 
#include <map> 

namespace Connections 
{ 
    DWORD WINAPI MainThread(LPVOID lParam); // Main thread 
    DWORD WINAPI DataThread(LPVOID lParam); // thread that will be created for each client 
    struct ClientServer // struct to keep a server and a client pair. 
    { 
    public: 
     struct Client* Client; 
     class Server* Server; 
    }; 
    struct Client // a struct wrapper to keep clients 
    { 
    public: 
     Client(SOCKET Connection, int BufferSize, UINT ID); 
     ~Client(); 
     SOCKET WorkerSocket; 
     char Buffer[255]; 
     bool Connected; 
     int RecvSize; 
     UINT UID; 
     void Send(char * Data); 
     void Disconnect(); 
    }; 
    class Server 
    { 
    private: 
     SOCKET WorkerSocket; 
     SOCKADDR_IN EndPnt; 
     UINT ID; 
     int CBufferSize; 
    public: 
     Server(int Port, int Backlog, int BufferSize); 
     ~Server(); 
     __event void ClientRecieved(Client* Clientr, char * RecData); 
     bool Enabled; 
     int Port; 
     int Backlog; 
     HWND ReqhWnd; 
     std::map<UINT, Client*> ClientPool; 
     void WaitForConnections(Server*); 
     void WaitForData(Client*); 
     void InvokeClientDC(UINT); 
     void Startup(); 
     void Shutdown(); 
    }; 
} 
#endif 

server.cpp:

#include "Server.h" 

namespace Connections 
{ 
    void Server::Startup() 
    { 
     WSADATA wsa; 
     WSAStartup(0x0202, &wsa); 
     this->WorkerSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); 
     this->EndPnt.sin_addr.s_addr = ADDR_ANY; 
     this->EndPnt.sin_family = AF_INET; 
     this->EndPnt.sin_port = htons(this->Port); 
     this->Enabled = true; 
     this->ID = 0; 
     bind(this->WorkerSocket, (SOCKADDR*)&this->EndPnt, sizeof(this->EndPnt)); 
     printf("[AsynServer]Bound..\n"); 
     listen(this->WorkerSocket, this->Backlog); 
       CreateThread(NULL, NULL, &MainThread, this, NULL, NULL); 
    } 
    void Server::WaitForConnections(Server * Ser) 
    { 
     WSAEVENT Handler = WSA_INVALID_EVENT; 
     while(Ser->Enabled) 
     { 
      Handler = WSACreateEvent(); 
      WSAEventSelect(Ser->WorkerSocket, Handler, FD_ACCEPT); 
      WaitForSingleObject(Handler, INFINITE); 
      SOCKET accptsock = accept(Ser->WorkerSocket, NULL, NULL); 
      Client * NewClient = new Client(accptsock, 255, Ser->ID++); 
      NewClient->Connected = true; 
      printf("[AsynServer]Client connected.\n"); 
      ClientServer * OurStruct = new ClientServer(); 
      OurStruct->Server = Ser; 
      OurStruct->Client = NewClient; 
      CreateThread(NULL, NULL, &DataThread, OurStruct, NULL, NULL); 
     } 
    } 
    void Server::WaitForData(Client * RClient) 
    { 
     WSAEVENT Tem = WSA_INVALID_EVENT; 
     Tem = WSACreateEvent(); 
     WSAEventSelect(RClient->WorkerSocket, Tem, FD_READ); 
     while(RClient->Connected) 
     { 
      WaitForSingleObject(Tem, INFINITE); 
      RClient->RecvSize = recv(RClient->WorkerSocket, RClient->Buffer, 255, NULL); 
      if(RClient->RecvSize > 0) 
      { 
       RClient->Buffer[RClient->RecvSize] = '\0'; 
       __raise this->ClientRecieved(RClient, RClient->Buffer); 
       Sleep(50); 
      } 
     } 
     return; 
    } 
    DWORD WINAPI MainThread(LPVOID lParam) 
    { 
     ((Server*)lParam)->WaitForConnections((Server*)lParam); 
     return 0; 
    } 
    DWORD WINAPI DataThread(LPVOID lParam) 
    { 
     ClientServer * Sta = ((ClientServer*)lParam); 
     Sta->Server->WaitForData(Sta->Client); 
     return 0; 
    } 
} 

Теперь после создания экземпляра сервера и создания основного потока, я могу принимать клиент одновременно и читать данные, которые они отправляют, но после двух соединений мой процессор l AGS до до использования 100% .. Я предполагаю, что мой метод неверен, поэтому мой вопрос кто-то может указать на возможный недостаток в моем коде, или просто указать мне приличное руководство для асинхронных сокетов, при условии, что я уже искал в течение недели без каких-либо результатов (возможно, мое отчаяние мешает мне выбрать правильные ключевые слова: |). Спасибо заранее и извините за огромный кусок кода, подрезал его до тех пор, пока это разрешено.

Mfg, SimpleButPerfect.

ответ

2

1-я ошибка: В сервере :: WaitForConnections вы создаете HEVENT (Handler = WSACreateEvent()) каждый раз, когда цикл while выполняется без его уничтожения. Переместите создание HEVENT вне цикла while.

2-я ошибка: Ваш буфер 255 длинный (char), но вы делаете это: RClient-> Buffer [RClient-> RecvSize] = '\ 0'; , где RClient-> RecvSize может быть исключением размера вашего буфера - это означает, что вы делаете «переполнение буфера» класса.

Надеюсь, это поможет. Dominik

+0

Спасибо за ваш ответ, однако, когда я создаю обработчик вне цикла, цикл просто начинает стрелять как сумасшедший, даже не дожидаясь появления WSAEvent. SimpleButPerfect. – SimpleButPerfect

+1

Это потому, что событие, созданное WSAEvent(), является событием с ручным сбросом, и вы нигде не перезагружаете его; поэтому он устанавливается один раз, а затем остается установленным. –

+0

Осмотрев код, спасибо, что указал на недостаток. – SimpleButPerfect