c++ - C++ Segfault，我不知道为什么？

Question

我在头文件中声明了一个数组，如下所示：

private:
   int frames[10];

并在类构造函数中赋值，如下所示：

file.open(File);
if(file.is_open())
{
    std::string line;
    getline(file, line);
    std::string param[10];
    std::stringstream stream(line);
    int n=0;
    while(!stream.eof())
    {
        getline(stream, param[n], '$');
        frames[n] = atoi(param[n].c_str());
        n++;
    }
    file.close();
}

稍后在函数中使用此数组：

currentFrame++;
if(frames[currentAnimation] <= currentFrame)
{
    currentFrame = 0;
}

当我运行我的代码时，我得到分段错误，并且 gdb 返回这个：

Program received signal SIGSEGV, Segmentation fault.
0x0000000000402c22 in Sprite::update (this=0x7ffff6efe678 <main_arena+88>) at Sprite.cpp:93 93              
if(frames[currentAnimation] <= currentFrame)
(gdb) bt
#0  0x0000000000402c22 in Sprite::update (this=0x7ffff6efe678 <main_arena+88>) at Sprite.cpp:93
#1  0x0000000000401fcb in main (argc=1, argv=0x7fffffffeb88) at main.cpp:146

我不确定我要去哪里错了，我认为错误就在这里。我不能真正发布所有代码，但如果您需要更多具体信息，请询问。

非常感谢您提前。

Answer 1

尝试这个

private:
   std::vector<int> frames;


file.open(File);
if(file.is_open())
{
    std::string line;
    getline(file, line);
    std::string param;
    std::stringstream stream(line);
    while(getline(stream, param, '$'))
        frames.push_back(atoi(param.c_str()));
    file.close();
}

currentFrame++;
if( currentAnimation < frames.size() && frames[currentAnimation] <= currentFrame)
{
    currentFrame = 0;
}

请参阅 Loki 的回答，了解为什么 while(!stream.eof()) 不好

Answer 2

写

int n=0;
while(!stream.eof() && n < 10)
{

...

和

currentFrame++;
if(currentFrame < 10 && frames[currentAnimation] <= currentFrame)
{
    currentFrame = 0;
}

或使用类似的东西

currentFrame = (currentFrame + 1) % 10;

Answer 3

几个问题：

您这里只有 10 项：

std::string param[10];

但是这里没有检查 10：

while(!stream.eof())

所以这可能会增加 10 个以上，这肯定会导致问题。

此外，这种形式的循环几乎总是错误的：

while(!stream.eof())
{
    getline(stream, param[n], '$');
    frames[n] = atoi(param[n].c_str());
    n++;
}

如果您的输入中有任何错误数据，这将进入无限循环。否则，当您到达 EOF 时，std::getline()无法读取数据并设置 eof 标志，但您仍然分配给帧（并增加 n）。atoi()错误数据返回 0，因此最后一个元素Frames将为零（不确定这是否是预期的行为（但它的草率）。

正确的风格是将 read 放入 while 条件中。所以将这两件事加在一起，你的循环应该是这样的。

while(n < 10 && getline(stream, param[n], '$'))
{
    // loop is only entered if the read succeed.

    // Not sure about the type of `frames` so hard to talk about
    // if this loop is still correct. The meaning has changed slightly.
    frames[n] = atoi(param[n].c_str());
    n++;
}
if (n < 10) {/*We have an issue with not enough data!*/}