c++ - 使用 stringstream 在内存中超出范围的问题

Question

我刚开始第一次使用 stringstream，我喜欢这个概念，但我很难找到我的 stringstream 函数到底在哪里超出了内存范围。

我的函数所做的是接受一个字符串，例如，“N02550 G3 X16.7379 Y51.7040 R0.0115”这是我工作中的 CNC 机器的机器代码。我将字符串传递给字符串流，以便找到旁边有 X、Z、Y 的字符串，这些是坐标。然后它去掉了开头的字符，以便将 float 保存到我的结构“坐标”中(有 3 个 double ，x、y、z)。

当我运行一个包含 33 行机器代码的文本文件时，我的程序可以运行。当我用 718 行的机器代码运行它时，它达到 718，然后因内存超出范围而崩溃。然后另一个奇怪的部分是当我运行 118,000 行的机器代码时，它上升到大约 22,000 行然后崩溃。所以我很难弄清楚为什么它能够做到这一点以及导致问题的原因。

函数如下:

   void getC(string& line, Coordinates& c)//coordinates holds 3 doubles, x, y, z
{
    //variables
    string holder;
    stringstream ss(line);

    while(ss)
    {
        ss >> holder;
        if(holder.at(0) == 'X')
        {
            holder.erase(0,1);//get rid the the character at the beggining
            stringstream sss(holder);
            sss >> c.x;
            sss.clear();
        }
        if(holder.at(0) == 'Y')
        {
            holder.erase(0,1);
            stringstream sss(holder);
            sss >> c.y;
            sss.clear();
        }
        if(holder.at(0) == 'Z')
        {
            holder.erase(0,1);
            stringstream sss(holder);
            sss >> c.z;
            sss.clear();
        }
        if(ss.eof()) // to get out of the ss stream
            break;

    }
    ss.clear();
}

如果您想查看整个应用程序(该应用程序有详细的文档)，请询问您是否需要包含机器代码的 txt 文件。谢谢!

Answer 1

尝试改变:

while(ss)
{
    ss >> holder;
    ...
    if(ss.eof()) // to get out of the ss stream
        break;
}

简单来说:

while(ss >> holder)
{
    ...
}

而且您可以在每个分支 (X/Y/Z) 中摆脱那些对 clear 的调用，因为它实际上并没有做任何事情，因为 sss 是一个暂时的，你不会用它做更多的事情(没有必要在你将要丢弃的东西上设置标志)。我怀疑您的超出范围问题是由于 ss >> holder 失败后尝试访问 holder.at(0) 造成的。

您通常希望在读取 token 后立即检查输入失败，同时尝试输入和检查失败的便捷方法是简单地检查 ss >> token 是否计算为真的。所以我们可以这样写代码:

if (ss >> token)
{
    ...
}
else
{
    // handle failure if necessary
}

我通常发现以这种方式编写代码比手动检查错误标志更容易避免遇到麻烦。

简化版:

void getC(string& line, Coordinates& c)
{
    stringstream ss(line);   
    for (string holder; ss >> holder; )
    {
        const char ch = holder.at(0);
        stringstream sss(holder.substr(1));
        if (ch == 'X')
            sss >> c.x;
        else if (ch == 'Y')
            sss >> c.y;
        else if (ch == 'Z')
            sss >> c.z;
    }
}