在 for 循环中擦除向量中的对象时，如何防止发生分段错误？

Question

[编辑 2] 这可能是因为表面正在被释放，而其他对象仍在使用它......所以我想写它，所以只有当屏幕上没有更多对象时它才会被释放。

[编辑][更新] 我修复了使用迭代器的代码，但我仍然遇到同样的问题......我刚刚确认这是对象的析构函数的问题。在类中，析构函数调用 SDL_FreeSurface(image);从内存中释放对象的图像...但是使用复制构造函数创建的对象显然不能这样做。我需要做什么才能使其与使用复制构造函数创建的对象正常工作？我似乎在 Google 上找不到与此相关的任何内容。

数字是指给定时间屏幕上的子弹数...我有一个对象“子弹”的向量（向量子弹；）...代码检查触发按钮（z）是否按下，并在向量中添加一个新项目符号（如果是）。然后它更新（移动，blits），然后检查子弹是否在屏幕的末端......一切正常，直到子弹到达应该被摧毁的屏幕末端。

if (trigger)
{
  bullet.push_back(Bullet(position.x, position.y));
  ++number;
}

for(int i = 0; i < bullet.size(); i++)
{
  bullet[i].update();
}

问题出在下面...当删除项目符号对象时，程序会因分段错误而退出...现在，我明白为什么会发生这种情况，因为矢量大小在循环中发生了变化，但是我不知道如何解决它......我添加了休息，认为它可以解决问题，因为无论如何在给定时间只有一个子弹可以

for(int i = 0; i < bullet.size(); i++)  
{
  if (bullet[i].position.y <= 0)
  {     
    bullet.erase(bullet.begin() + i);
    --number;
    break;
  }
}

我是向量的新手，所以如果我有不明白的地方或者我犯了 n00b 错误，请多多包涵。

Answer 1

序列容器的标准擦除循环如下所示：

for (std::vector<Bullet>::iterator it = v.begin(); it != v.end() /* not hoisted */; /* no increment */)
{
    if (delete_condition)
    {
        it = v.erase(it);
    }
    else
    {
        ++it;
    }
}

使用迭代器比使用计数索引更可取，因为它将您的代码从计数器算术中解放出来，这不会增加代码的清晰度并且相当麻烦，而这里介绍的迭代器版本是相当自我的-解释性的。

关键是在出现erase 时不要增加循环变量。

您可能还想查看remove/erase 成语：

v.erase(std::remove_if(v.begin(), v.end(),
                       [](Bullet & b) -> bool { return delete_condition }),
        v.end());