c++ - 擦除期间std容器迭代器无效[重复]

原文 标签 c++ g++ std composite-component

可能的重复:
vector erase iterator
Problem with std::map::iterator after calling erase()
我担心我的一段代码。我有组件和一个存储组件的对象。问题是在更新过程中,组件可以指示从对象中移除组件。但它是从另一个函数调用的。

void Object::update() { //using std::map here
   for(ComponentMap::iterator i = components.begin(); i != components.end(); ++i) {
      (*i).second->update();
   }
}

void HealthComponent::update() {
   if(health <= 0) object->removeComponent("AliveComponent"); //this is wrong logic. but its just an example :D
}

void Object::removeComponent(string component) {

  ComponentMap::iterator i = components.find(component);
  if(i == components.end()) return;

  components.erase(i);

}

假设我有很多组件——健康、活动、图形、物理、输入等等。
我尝试了类似的方法(使用一些测试组件),在更新期间没有错误。但我真的很担心。以后会不会给我一个错误?如果是,如何修复?
提前谢谢你,
卡西姆

最佳答案

++i可能不再有效时(因为您删除了它),您不能在容器中循环并说i。典型的擦除循环如下:

for (it = x.begin(); it != x.end(); /* nothing here! */)
{
  if (must_erase(*it))
  {
    x.erase(it++); // advance it while still valid, return previous and erase
  }
  else
  {
    ++it;
  }
}

用这种精神重写你的代码。
要阐明您的问题:在Object::update()中,您调用HealthComponent::update()使迭代器无效,然后调用i,这是未定义的行为。

相关文章:

c++ - 我可以在不使用原始指针的情况下在std :: initializer_list初始化期间避免复制吗?

c++ - C到C ++表内联定义

c++ - 函数g ++ 6.2的自动类型推导

c++ - 标识符“ a”在c ++中未定义

c++ - 模板中的静态变量未定义

c++ - 无法打开包含文件:“ ntddk.h”:没有此类文件或目录

c++ - 用元素加载std :: queue <uint8_t *>的更有效方法?

c++ - #include <bits / stdc ++。h>如何在C ++中工作? [重复]

c++ - C ++ 03中C ++枚举的基本类型

c++ - 将成员复制到std :: vector