2014-03-03 66 views
1

我得到了一些客戶端服務器程序的代碼,我自己修改了它。但我找不到爲什麼我的代碼無法正常工作。編譯很好,但在客戶端中出現中止錯誤?我已經嘗試了自己,並且對編程還很陌生。所以,如果您可以看看下面給出的客戶端服務器代碼,並檢查問題是什麼。我的客戶端服務器代碼無法正常工作

//Client side code 

#include <iostream> 
#include <winsock2.h> 
#include <vector> 
#include <algorithm> 
#include <ctime> 
#include "client.h" 
#pragma comment(lib, "Ws2_32.lib") 

using namespace std; 
tcp_client::tcp_client() 
{} 

tcp_client::~tcp_client() 
{} 
int tcp_client::start_listening() 

{ 

    WSADATA wsaData; 
int iResult = WSAStartup(MAKEWORD(2,2), &wsaData); 
    SOCKET ConnectSocket; 
    ConnectSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); 

    if (ConnectSocket == INVALID_SOCKET) 
    { 
     cout<<"Client: Error at socket(): %ld.\n"<< WSAGetLastError(); 
     WSACleanup(); 
     return 0; 
    } 

    sockaddr_in clientService; 
    clientService.sin_family = AF_INET; 
    clientService.sin_addr.s_addr = inet_addr("127.0.0.1"); 
    clientService.sin_port = htons(55555); 

    if (connect(ConnectSocket, (SOCKADDR*)&clientService, sizeof(clientService)) == SOCKET_ERROR) 
    { 
     cout<<"Client: Failed to connect\0"; 
     WSACleanup(); 
     return 0; 
    } 
    return 0; 
} 

int tcp_client::start_sending() 
{ 

    int bytesSent; 
    int bytesRecv = SOCKET_ERROR; 

    std::vector<double> m_vector; 
    uint32_t nlength =0; 
    std::vector<double> m_vector1(nlength/sizeof(double)); 


    for(int i = 0; i <100; i++) 
    {  
     m_vector.push_back(i); 
    } 

    uint32_t siz = (m_vector.size())*sizeof(double); 



    int total_bytes = 0; 
    int count=0; 


    for(int j=0; j<10; j++) 
    { 

     bytesSent = send(ConnectSocket,(char*)&siz, 4, 0); 
     assert (bytesSent == sizeof (uint32_t)); 
     std::cout<<"length information is in:"<<bytesSent<<"bytes"<<std::endl; 

     bytesSent = send(ConnectSocket,(char*)m_vector.data(), siz, 0); 
     total_bytes = total_bytes+bytesSent; 

     std::cout<<"Client: Bytes sent:"<<total_bytes<<std::endl; 
    } 

    closesocket (ConnectSocket); 
    return 0; 
} 


int main() 
{ 

    std::clock_t c_start = std::clock(); 

    tcp_client a; 
    a.start_listening(); 
    a.start_sending(); 

    std::clock_t c_end = std::clock(); 
    std::cout << "CPU time used: "<< 1000.0 * (c_end-c_start)/CLOCKS_PER_SEC<< " ms\n"; 

    WSACleanup(); 
    system("pause"); 
    return 0; 
} 

// Client.h

#include <string.h> 
#include <winsock2.h> 
class tcp_client 
{ 

public: 
    tcp_client(); 
    virtual ~tcp_client(); 
    int start_listening(); 
    int start_sending(); 

protected: 

    SOCKET ConnectSocket; // client Socket 
    }; 

//服務器代碼

#include <iostream> 
#include <winsock2.h> 
#include <vector> 
#include <algorithm> 
#include <ctime> 
#pragma comment(lib, "Ws2_32.lib") 
#include "server.h" 

using namespace std; 
tcp_server::tcp_server() 
{} 

tcp_server::~tcp_server() 
{} 

int tcp_server::start_listening() 
{ 

    WORD wVersionRequested; 
    WSADATA wsaData; 
    int wsaerr; 
    wVersionRequested = MAKEWORD(2, 2); 
    wsaerr = WSAStartup(wVersionRequested, &wsaData); 


    if (wsaerr != 0) 
    { 
     std::cout<<"server starting...."<<std::endl; 
    } 


    m_socket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); 

    if (m_socket == INVALID_SOCKET) 
    { 
     cout<<"Server: Error at socket(): "<<WSAGetLastError()<<std::endl; 
     WSACleanup(); 
     return 0; 
    } 


    sockaddr_in service; 
    service.sin_family = AF_INET; 

    service.sin_addr.s_addr = inet_addr("127.0.0.1"); 
    service.sin_port = htons(55555); 

    if (bind(m_socket, (SOCKADDR*)&service, sizeof(service)) == SOCKET_ERROR) 
    { 
     cout<<"Server: bind() failed:"<< WSAGetLastError(); 
     closesocket(m_socket); 
     return 0; 
    } 


    if (listen(m_socket, 10) == SOCKET_ERROR) 
    { 
     cout<<"Server: listen(): Error listening on socket "<< WSAGetLastError()<<std::endl; 
    } 

    SOCKET AcceptSocket; 

    while (1) 
    { 
     AcceptSocket = SOCKET_ERROR; 
     while (AcceptSocket == SOCKET_ERROR) 
     { 
      AcceptSocket = accept(m_socket, NULL, NULL); 
     } 

     cout<<"Server: Client Connected"<<std::endl; 
     m_socket = AcceptSocket; 
     break; 
    } 

    return 0; 
} 

int tcp_server::start_receiving() 
{ 

    int bytesSent; 
    int bytesRecv; 
    uint32_t nlength =0; 
    int total_br =0; 
    std::vector<double> m_vector(nlength/sizeof(double)); 


    std::clock_t c_start=0; 

    int count =0; 

    while(1) 
    { 

     int length_received = recv(m_socket,(char*)&nlength, 4, 0); 
     m_vector.resize(nlength/sizeof(double)); 
     bytesRecv = recv(m_socket,(char*)m_vector.data(), nlength, 0); 


     if(bytesRecv > 0) 
     { 
      total_br = total_br + bytesRecv; 

      cout<<"Server: Received bytes are"<<total_br<<std::endl; 
     } 
      else 
     { 
      std::cout<<"Data Receiving has finished"<<std::endl; 
      break; 
     } 

} 
closesocket (m_socket); 
WSACleanup(); 
return total_br; 
} 

int main() 

{ 
    std::clock_t c_start = std::clock(); 
    tcp_server a; 
    a.start_listening(); 
    int byte= a.start_receiving(); 
    std::cout<<"total byte received are"<<byte; 

    std::clock_t c_end = std::clock(); 
    std::cout << "CPU time used: "<< 1000.0 * (c_end-c_start)/CLOCKS_PER_SEC<< " ms\n"; 
    system("pause"); 
    return 0; 
} 

//server.h

#include <string.h> 
#include <winsock2.h> 
class tcp_server 
{ 

public: 
    tcp_server(); 
    virtual ~tcp_server(); 
    int start_listening(); 
    int start_receiving(); 

protected: 

    SOCKET m_socket; // Server Socket 
    }; 
+3

您的服務器代碼中存在一個小問題。如果我連接到服務器端口併爲nLength發送0xffffffff,則代碼將嘗試爲隨後的m_vector.resize調用分配4GB。另外,recv()可以並將返回部分數據。您應該循環調用recv(),直到您收到所有字節。或使用MSG_WAIT標誌。 – selbie

+0

你如何運行該程序? – jfly

+0

@selbie我試着用MSG_WAITALL標誌,但結果是一樣的。我的問題是,如果我每次從客戶端發送長度和數據,那麼爲什麼服務器每次都不接收帶有前綴長度的數據?爲什麼我應該只保留recv調用循環爲什麼不長度recv以及?因爲它們在讀取長度時不是分隔的,而只能接收到大量的數據。請問如果可能,請糾正錯誤?這段代碼在沒有類的情況下工作正常,但在實現類之後我有問題。 – user3369727

回答

1

在client.cpp刪除這條線,它將工作:

SOCKET ConnectSocket; 

我之前運行代碼後刪除了這一行,所以它工作正常。

這條線是指定義的功能start_listening()這個局部變量屏蔽成員變量範圍內的局部變量ConnectSocket。看起來你將一個有效的文件描述符分配給成員變量,但成員變量ConnectSocket始終未初始化,因此send()失敗。

+0

@ user3369727你能告訴我你的.h文件嗎? – jfly

+0

請參閱編輯的代碼。 – user3369727

+0

我發現了問題,請參閱我的編輯。 – jfly

相關問題