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

Question

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

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

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

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

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

下面是问题所在...当一个子弹对象被删除时，程序退出时出现分段错误...现在，我明白为什么会发生这种情况，因为矢量大小在循环中被改变了，但我可以' t弄清楚如何修复它...我添加了休息时认为它可以解决问题，因为无论如何只有一个子弹在给定时间<= 0但是它没有...我最初有if语句在更新函数之后的最后一个for循环中，但我把它放在它自己的for循环中，所以我可以使用break。

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());