这些通过套接字发送/接收字符串的函数有什么问题?

What is wrong with these functions for sending/receiving a string over a socket?

本文关键字：函数 字符串 什么 问题 套接字      更新时间：2023-10-16

我试图写两个方法发送和接收一个套接字字符串。这是为了清理Windows的遗留代码，它使用了到处都是的send和recv，所以我想把它放到函数中来清理一下。你可以在下面看到我的想法。不幸的是，一旦我在代码中像这样使用这个函数:

std::string buffer = receiveStringFromSocket();

和

rc = sendStringToSocket(buffer);

我的代码被锁住了，甚至没有碰到断点。

谁能告诉我我做错了什么?我有一个困难的时间调试这个，因为我不是很熟悉C/c++。错误可能在代码库的其他地方。

sendStringToSocket : 

int sendStringToSocket(std::string sendString)
{
    char sendBuffer[1024];
    memset(sendBuffer, 0, sizeof(sendBuffer)); //Clear the buffer
    //sendString.copy(sendBuffer, sendString.length());
    strcpy(sendBuffer, sendString.c_str());
    cout << "string buffer content:" << endl << cout.write(sendBuffer,sendString.length()) << endl;
    rc = send(acceptsocket, sendBuffer, sizeof(sendBuffer)-1, 0);
    return rc;
}

receiveStringFromSocket : 

std::string receiveStringFromSocket()
{
    char receptionBuffer[1024];
    memset(receptionBuffer, 0, sizeof(receptionBuffer)); //Clear the buffer
    rc = recv(acceptsocket, receptionBuffer, sizeof(receptionBuffer)-1, 0); 
    string str(receptionBuffer);
    return str;
}

对于发送:主要问题是您不需要sendbuffer:您可以将sendString.c_str()传递给send()函数。使用sendbuffer带来的问题在注释中有描述。

接收:必须检查recv()的返回值。谁也说不准网络会出什么问题。

组合:在TCP中(假设你正在使用)不能保证每个send-call的字节数与每个recv-call接收的字节数相同。您必须实现一些检查，以查看是否接收到所有需要的字节(最简单的:发送并检查空字节)。对于小于50字节的字符串，这可能不会被注意到，但在某些时候，您将丢失字符串的一部分。

主要问题是send和recv都不能保证发送/接收的数据与您期望的大小相同。所以你需要基于这个事实的函数。例如:

void sendBuffer( const char *buff, size_t size )
{
     for( size_t sent = 0; sent != size; ) {
         int rc = send( socket, buff + sent, size - sent );
         if( rc <= 0 ) { // handle error here
         }
         sent += static_cast<size_t>( rc );
     }
}

你需要一个类似的接收端。有了这样的函数，可以更容易地在字符串之前发送数据大小，它减少了数据使用并简化了代码:

 uint32_t size = str.length();
 sendBuffer( reinterpret_cast<char *>( &size ), sizeof( size ) );
 sendBuffer( str.data(), size );

接收端相似。

虽然如果你决定或必须仍然发送字符串作为固定长度填充缓冲区1023字节长，这个函数可以用来，因为它是。

相关文章：