解析字符串并尝试查找“\0”时出错特点

Question

我试图让一侧向客户端发送错误消息，但客户端无法正确解析它。

我的错误是>>>>>在我的 parseString 函数中，它让 index = 0，因此我的“substr”调用超出范围。

服务器端:::

   #define ERRBUFSIZE 51

    string error = "Error - Already Registered: ";
                error.append(name);
                char err[ERRBUFSIZE];
                err[0] = 0;
                std::copy(error.begin(), error.end(), err+1);

                err[error.size() + 2] = '\0';

                if (send(sock, &err, ERRBUFSIZE, 0) < 0)
                {
                DieWithError("send() failed");
                }

客户端（谁正在接收）::

   char msgBuffer[ERRBUFSIZE];
        int bytesRcvd = 0;
        int totalRcvd = 0;
        while(totalRcvd != ERRBUFSIZE)
        {   
            // Get Message from Server
            if ((bytesRcvd = recv(sock, msgBuffer, ERRBUFSIZE, 0)) < 0)
            DieWithError("recv() failed or connection closed prematurely");

            totalRcvd += bytesRcvd; 
        }

        cout << "Bytes received: " << totalRcvd << endl;
        msgBuffer[totalRcvd] = '\0';

        string rcvMsg( msgBuffer , msgBuffer + totalRcvd);
        cout << rcvMsg << endl;
        rcvMsg = parseString(rcvMsg);

        return rcvMsg;

在哪里....

   string TCPClient::parseString(string message)
    {
        cout << "parsing new string" << endl;
        int index = message.find('\0');
        string result = message.substr(0, index);
        cout << "THE RESULT :: " << result << endl;
        return result;
    }

Answer 1

为什么不将 parseString 更改为以下内容：

string TCPClient::parseString(const string& message, int strLength )
{
    cout << "parsing new string" << endl;
    string result = message.substr( 0, strLength );
    cout << "THE RESULT :: " << result << endl;
    return result;
}

然后将调用代码更改为：

rcvMsg = parseString(rcvMsg, totalRcvd );

如果失败，您甚至可以使用字符串的“size()”函数。

编辑：还值得注意的是， find 将在字符串中前进，直到找到 '\0' 字符，但实际上不会查找它。将字符串放入字符串对象（存储长度），然后尝试通过缓慢的迭代过程自己计算长度，这似乎有点奇怪。

Edit2：如果您只想使用字符数组来执行此操作，您可以按如下方式进行手动搜索。

int index = 0;
while( *charString++ != '\0' ) index++;

就这么简单:)

Answer 2

从（非常难以阅读）代码来看，我认为您可能会遇到“索引地狱”。

检查 totalRecvd 实际上是您想要写入 \0 的索引，并且在创建 rcvMsg 时添加 >\0 对其进行索引（当您将字符串结尾标记放在 totalRcvd 字节上时，msgBuffer + TotalRcvd 看起来很糟糕，通常您会得到您编写的 (start, end-1) 索引之间的字符串）。

另外 - 搜索 \0 没有任何意义，因为它只是一个标记。

另外 (2) - 如果 totalRcvd == ERRBUFSIZE 您无法访问 msgBuffer [totalRcvd]，您只能访问范围 (0, ERRBUFSIZE -1)

Answer 3

当您想要管理缓冲区时，请使用 std::vector 而不是 std::string。 '\0' 和 std::basic_string 结合得不好。

Answer 4

首先，协议很糟糕。当您想要发送以 null 结尾的字符串时，您正在发送固定大小的块...将字符串长度放在消息的前面会更简单。

其次，您没有从套接字正确读取数据，只要管道中没有任何其他消息，它就会正常工作。
如果您的 while 循环曾经真正被执行过，您将用任何后续读取覆盖第一个片段。

无论如何，该错误的直接原因是您在字符串前面添加了一个 nul 终止符（分隔符？）。这难道不能解释为什么你总是在索引 0 处找到 nul 吗？
如果看不到它，请尝试写出发送时缓冲区的内容。

#define ERRBUFSIZE 51

void server(string const &name)
{
    string error = "Error - Already Registered: ";
    error.append(name);
    char err[ERRBUFSIZE];
    // you sure you want a NUL terminator at index 0??
    err[0] = 0;
    // can replace both lines with strcpy(err, error.c_str())
    std::copy(error.begin(), error.end(), err+1);
    err[error.size() + 2] = '\0';

    if (send(sock, &err, ERRBUFSIZE, 0) < 0)
        DieWithError("send() failed");
}

string parseString(string const &message) // no need to copy
{
    // ... this is redundant anyway, see below
}

string client()
{
    char msgBuffer[ERRBUFSIZE];
    int bytesRcvd = 0;
    int totalRcvd = 0;
    while(totalRcvd != ERRBUFSIZE)
    {       
        // Get Message from Server
        // ... recv(sock, msgBuffer+totalRcvd, ERRBUFSIZE-totalRcvd) ...
        if ((bytesRcvd = recv(sock, msgBuffer, ERRBUFSIZE, 0)) < 0)
            DieWithError("recv() failed or connection closed prematurely");

        totalRcvd += bytesRcvd; 
    }

    cout << "Bytes received: " << totalRcvd << endl;
    //sizeof(msgBuffer)==ERRBUFSIZE==totalRcvd, so this overflows
    msgBuffer[totalRcvd] = '\0';

    // assuming you skip the leading \0, msgBuffer is already nul-terminated
    // so use string rcvMsg(msgBuffer) and skip the parseString/substr
    string rcvMsg( msgBuffer , msgBuffer + totalRcvd);
    cout << rcvMsg << endl;
    // rcvMsg = parseString(rcvMsg);

    return rcvMsg;
}