C++ recv() 问题

Question

我想发送一个字符串“Hello there”，但我只收到“re”。这是为什么呢？

void Accept()
{
    SOCKADDR_IN sock;
    int intsock = sizeof(sock);
    remoteSocket = ::accept(desc, (LPSOCKADDR)&sock,  &intsock);
    if(remoteSocket == -1)
    {
        cout << "Error in Accept()" << endl;
    }
    HandleConnection();
}

void HandleConnection()
{
    cout << "You are connected !!!" << endl;
    char* temp = new char[20];
    Recv(temp);
    cout << temp << endl;
}

void Send(const char* buffer)
{
    if((::send(remoteSocket, buffer, strlen(buffer), 0)) < 0)
    {
        cout << "Error in Send()" << endl;
    }
}

void Recv(char* buffer)
{
    int n = 0;
    while((n = ::recv(remoteSocket, buffer, strlen(buffer), 0)) 0)
    {
        buffer[n] = 0;
    }
}

~Server()
{
    WSACleanup();
}

};

int main()
{
    Server s;
    s.Initialize();
    s.Socket();
    s.Bind();
    s.Listen();
    while(1)
    {
        s.Accept();
    }
    return 0;
}

Answer 1

尽管@Billy ONeal 指出了析构函数问题，但您在循环中执行recv()，但每次您都在覆盖接收到的缓冲区。我相信你想要这样的东西：

// Pass buffer and its real size. This function takes care of NULL termination.
size_t Recv(char* buffer, size_t size) {
    size_t total = 0, n = 0;
    while((n = ::recv(remoteSocket, buffer+total, size-total-1, 0)) > 0) {
        total += n;
    }
    buffer[total] = 0;
    return total;
}

int main() {
    char buffer[128];
    // Connect or whatever (and set your global remoteSocket)
    Recv(buffer, sizeof(buffer));
    cout << buffer << endl;
    return 0;
}

Answer 2

我认为问题出在while((n = ::recv(remoteSocket, buffer, strlen(buffer), 0)) 0)这行代码

您正在使用strlen(buffer) 获取不正确的缓冲区大小，您应该将sizeof(buffer) 传递给您的Recv 函数。

如果这不是问题，那么它就是问题之一：P

编辑：

正如 Kitsune 和 Mark 所指出的，sizeof(buffer) 将返回 4 或 8 个字节，因为它是在堆上分配的，并且只是一个指向内存块的指针。如果您选择使用堆栈（char buffer[20] 而不是new char[20]），您可以将 sizeof(buffer) 传递给您的 Recv 函数。否则，只需使用硬编码的 20。

你的代码应该是这样的：

void HandleConnection()
{
    cout << "You are connected !!!" << endl;
    char temp[20]; // <-- now we have an array
    Recv(temp, sizeof(temp)); // <-- sizeof(temp) will give us 20, not 4 anymore
    cout << temp << endl;
}

Recv(char* buffer, size_t buffer_size)
{
    recv(remoteSocket, buffer, buffer_size, 0);
}

Answer 3

您需要指定接收数据的缓冲区有多大——不是strlen(buffer)。

如果缓冲区数组在本地定义为数组（不在参数列表中），或者缓冲区是其定义在函数中可见的全局或文件范围数组，则可以使用sizeof(buffer)。否则，您需要使用传递给Recv() 函数的额外缓冲区大小参数——也就是说，如果缓冲区是在另一个函数中定义的，或者如果它是动态分配的。 （在代码中，数组定义在Recv() 中不可见，因此您需要确保Recv() 以某种方式知道大小 - 作为显式的额外参数，或者因为您将缓冲区包装在一个适当的类中，其中包括一种方法，告诉您为它所拥有的缓冲区分配了多少空间。

当然，显示的代码无法编译，因为buffer 实际上并没有在任何地方定义或声明。