使用 stringstream 在内存中超出范围的问题
Trouble with Out of range in memory using stringsteam
我刚开始第一次使用 stringstream,我喜欢这个概念,但我很难找到我的 stringstream 函数到底在哪里超出了内存范围。
我的函数所做的是接受一个字符串,例如,"N02550 G3 X16.7379 Y51.7040 R0.0115"这是我工作中的 CNC 机器的机器代码。我将字符串传递给字符串流,以便找到旁边有 X、Z、Y 的字符串,这些是坐标。然后它去掉开头的字符,以便将浮点数保存到我的结构 "Coordinate"(有 3 个双精度数,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 文件。谢谢!
尝试更改:
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)
造成的。
您通常希望在读取令牌后立即检查输入失败,同时尝试输入和检查失败的一种简便方法是简单地检查 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;
}
}
我刚开始第一次使用 stringstream,我喜欢这个概念,但我很难找到我的 stringstream 函数到底在哪里超出了内存范围。
我的函数所做的是接受一个字符串,例如,"N02550 G3 X16.7379 Y51.7040 R0.0115"这是我工作中的 CNC 机器的机器代码。我将字符串传递给字符串流,以便找到旁边有 X、Z、Y 的字符串,这些是坐标。然后它去掉开头的字符,以便将浮点数保存到我的结构 "Coordinate"(有 3 个双精度数,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 文件。谢谢!
尝试更改:
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)
造成的。
您通常希望在读取令牌后立即检查输入失败,同时尝试输入和检查失败的一种简便方法是简单地检查 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;
}
}