我正在编写一个简单的客户端套接字应用程序但在连接之后服务器正在接收一个空缓冲区

时间:2018-05-25 08:37:27

标签: c++ sockets server client

这是一个简单的客户端套接字应用程序。当我运行此代码时,服务器在接收数据时,正在接收一个空缓冲区。任何人都可以找到问题吗? 代码用cpp编写,使用套接字传输数据。 在连接之后,客户端应该向服务器发送“Hello im client”,并且服务器打印消息,但是方式有问题。

这是服务器代码

#define IP "127.0.0.1"


int ConnectServer::connectTo()
{

    //----------------------
    // Initialize Winsock
    WSADATA wsaData;
    int iResult = 0;

    _ListenSocket = INVALID_SOCKET;
    sockaddr_in serverAddr;

    iResult = WSAStartup(MAKEWORD(2, 2), &wsaData);
    if (iResult != NO_ERROR) {
        wprintf(L"WSAStartup() failed with error: %d\n", iResult);
        return 1;
    }
    //----------------------
    // Create a SOCKET for listening for incoming connection requests.
    _ListenSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
    if (_ListenSocket == INVALID_SOCKET) {
        wprintf(L"socket function failed with error: %ld\n", WSAGetLastError());
        WSACleanup();
        return 1;
    }
    //----------------------
    // The sockaddr_in structure specifies the address family,
    // IP address, and port for the socket that is being bound.
    serverAddr.sin_family = AF_INET;
    serverAddr.sin_addr.s_addr = INADDR_ANY;
    serverAddr.sin_port = htons(PORT);

    iResult = bind(_ListenSocket, (SOCKADDR*)& serverAddr, sizeof(serverAddr));
    if (iResult == SOCKET_ERROR) {
        wprintf(L"bind function failed with error %d\n", WSAGetLastError());
        iResult = closesocket(_ListenSocket);
        if (iResult == SOCKET_ERROR)
            wprintf(L"closesocket function failed with error %d\n", WSAGetLastError());
        WSACleanup();
        return 1;
    }
    //----------------------
    // Listen for incoming connection requests 
    // on the created socket
    if (listen(_ListenSocket, SOMAXCONN) == SOCKET_ERROR)
    {
        wprintf(L"listen function failed with error: %d\n", WSAGetLastError());
        closesocket(_ListenSocket);
        WSACleanup();
        return 1;
    }

    // Create a SOCKET for accepting incoming requests.
    SOCKET AcceptSocket;
    sockaddr_in client;


    wprintf(L"Waiting for client to connect...\n");

    //----------------------
    // Accept the connection.
    int clientSize = sizeof(client);

    AcceptSocket = accept(_ListenSocket, (SOCKADDR*)&client, &clientSize);
    if (AcceptSocket == INVALID_SOCKET)
    {
        wprintf(L"accept failed with error: %ld\n", WSAGetLastError());
        closesocket(_ListenSocket);
        WSACleanup();
        return 1;
    }
    else
    {

        wprintf(L"Client connected.\n");

    }
    //cout << "whats happening??" << endl;
    char* buffer = "";
    iResult = recv(AcceptSocket, buffer, (int)strlen(buffer), 0);
    if (iResult == SOCKET_ERROR) {
        printf("recv function failed with error: %d\n", WSAGetLastError());
        closesocket(_server);
        WSACleanup();
        return 1;
    }
    else
    {
        printf("recieved data:\n");
        printf(buffer);
    }




    iResult = closesocket(_ListenSocket);
    if (iResult == SOCKET_ERROR) {
        wprintf(L"closesocket function failed with error %d\n", WSAGetLastError());
        WSACleanup();
        return 1;
    }
    else
    {
        std::cout << "Client disconnected" << std::endl;
    }
    WSACleanup();
    return 0;
}

这是客户端代码:

int ConnectClient::connectTo()
{
    int iResult;
    WSADATA WSAData;
    SOCKET _server;
    SOCKADDR_IN addr;
    HBITMAP pic = NULL;
    WSAStartup(MAKEWORD(2, 0), &WSAData);
    _server = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);

    addr.sin_addr.s_addr = inet_addr(IP);
    addr.sin_family = AF_INET;
    addr.sin_port = htons(PORT);

    if (connect(_server, (SOCKADDR *)&addr, sizeof(addr)) == INVALID_SOCKET)
    {
        wprintf(L"connect function failed with error: %d\n", WSAGetLastError());
        system("PAUSE");
        return 1;
    }
    else{
        std::cout << "Connected to _server!" << std::endl;
        char  *Buffer = "hello I'm client";
        printf(Buffer);
        iResult = send(_server, Buffer, (int)strlen(Buffer), 0);
        if (iResult == SOCKET_ERROR) {
            printf("send failed: %d\n", WSAGetLastError());
            closesocket(_server);
            WSACleanup();
            return 1;
        }

    }

    //cout << testBuffer << endl;
    //cout << strlen(testBuffer) << endl;



    //coordinations(_server);
    iResult = closesocket(_server);
    if (iResult == SOCKET_ERROR) {
        wprintf(L"closesocket function failed with error %d\n", WSAGetLastError());
        WSACleanup();
        return 1;
    }
    else
    {
        std::cout << "Client disconnected" << std::endl;
    }

    WSACleanup();
    std::cout << "Socket closed." << std::endl << std::endl;
    system("PAUSE");

    return 0;
}

int ConnectClient::coordinations(SOCKET server)
{

    HANDLE hStdInput, hStdOutput, hEvent;                         //WAIT_ABANDONED   = 128
    INPUT_RECORD ir[128];                                       //WAIT_OBJECT_0    = 0
    DWORD nRead;                                                //WAIT_TIMEOUT     = 258
    COORD xy;
    UINT i;

    hStdInput = GetStdHandle(STD_INPUT_HANDLE);
    hStdOutput = GetStdHandle(STD_OUTPUT_HANDLE);
    FlushConsoleInputBuffer(hStdInput);
    hEvent = CreateEvent(NULL, FALSE, FALSE, NULL);                  //Event is created non-signaled (3rd param).
    HANDLE handles[2] = { hEvent, hStdInput }; 
    //char testBuffer[1024];    
    char a;
    char b;
    //Program loops monitoring two handles.  The
    while (WaitForMultipleObjects(2, handles, FALSE, INFINITE))     //1st handle ( handles(0) ) is an event which
    {                                                           //is initially set to non-signaled.  The 2nd
        ReadConsoleInput(hStdInput, ir, 128, &nRead);                 //handle monitored by WaitForMultipleObjects()
        for (i = 0; i<nRead; i++)                                       //is the standard input handle set up to
        {                                                          //allow access to mouse/keyboard input.  As
            switch (ir[i].EventType)                                //long as neither handle is in a signaled
            {                                                      //state, WaitForMultipleObjects() will block
            case KEY_EVENT:                                       //in an efficient wait state.  If any keypress
                if (ir[i].Event.KeyEvent.wVirtualKeyCode == VK_ESCAPE) //or mouse movement occurs, WaitForMultiple
                    SetEvent(hEvent);                                //Objects will return TRUE and the input will
                else                                                //be read by ReadConsolInput().  If the [ESCAPE]
                {                                                   //key is pressed the event object represented
                    xy.X = 0; xy.Y = 0; 
                    //by hEvent will be set to a signaled state by
                    SetConsoleCursorPosition(hStdOutput, xy);         //the SetEvent() Api function.  This will be
                    printf                                           //picked up by the next WaitForMultipleObjects()
                        (    
                        //call, and the function will return FALSE and
                        "AsciiCode = %d: symbol = %c\n",                //execution will drop out of the while loop
                        ir[i].Event.KeyEvent.uChar.AsciiChar,           //and program termination will occur.
                        ir[i].Event.KeyEvent.uChar.AsciiChar
                        );     
                    a = ir[i].Event.KeyEvent.uChar.AsciiChar;
                    b = ir[i].Event.KeyEvent.uChar.AsciiChar;
                    char testBuffer[1024] = "AsciiCode = %d: symbol = %c\n", a, b;              //execution will drop out of the while loop
                                 //and program termination will occur.

                    send(server, testBuffer, strlen(testBuffer), 0);//It is important to note that if the 3rd
                }                                                   //parameter to WaitForMultipleObjects() is
                break;                                              //set to FALSE, the function will return if
            case MOUSE_EVENT:                                     //either of the handles in the HANDLE array
                xy.X = 0, xy.Y = 1;                                     //represented by handles is signaled.
                SetConsoleCursorPosition(hStdOutput, xy);
                printf
                    (
                    "%.3d\t%.3d\t%.3d",
                    ir[i].Event.MouseEvent.dwMousePosition.X,
                    ir[i].Event.MouseEvent.dwMousePosition.Y,
                    (int)ir[i].Event.MouseEvent.dwButtonState & 0x07   //mask out scroll wheel, which screws up
                    );     //output
                a = ir[i].Event.MouseEvent.dwMousePosition.X;
                b = ir[i].Event.MouseEvent.dwMousePosition.Y;
                char testBuffer[1024] = "AsciiCode = %d: symbol = %c\n", a, b;              //execution will drop out of the while loop
                //and program termination will occur.

                send(server, testBuffer, strlen(testBuffer), 0);
                break;
            }
        }
    };

    return 0;
}

1 个答案:

答案 0 :(得分:2)

问题在于:

char* buffer = "";
iResult = recv(AcceptSocket, buffer, (int)strlen(buffer), 0);

您告诉recv读取零字节,因为buffer为空。

除此之外,代码还存在一些问题。您正在发送这样的字符串:

iResult = send(_server, Buffer, (int)strlen(Buffer), 0);

将发送所有字符,但不发送空终止符。但是,您的服务器代码需要一个null终止符,因为它是通过printf打印代码。

更重要的问题是您如何发送数据。你的服务器不知道客户端有多少数据,所以除非它一次读取一个字节,直到它获得一个空终止符(效率非常低),否则会出现问题。

通常的方法是发送固定大小的邮件头并让服务器读取头。标头中的一个字段将包含消息体的长度,一旦服务器读取了标头,它就可以为消息体分配内存,然后读取它。