运行时错误：映射/设置迭代器不兼容

Question

我在第 8 行遇到运行时错误“映射/设置迭代器不兼容”。

void Manager::Simulate(Military* military, Shalishut* shalishut,char* args[]){
    Simulation* simulation = Simulation::GetInstance();
    Time* time = Time::GetInstance();

    multimap<int,Task*>::iterator itTasks;
    itTasks = simulation->GetTasks().begin();
    while(itTasks != simulation->GetTasks().end()){
      while (itTasks->second->GetTimeStamp() == time->GetTime()){ /*line 8 - ERROR*/
            TaskExecute(itTasks->second,military,shalishut,args);
            itTasks++;
        }
        // Unit take car of vehicles
        time->TimeIncrease();
    }

}

Simulation 被声明为 multimap。问题是什么？

Answer 1

我将大胆猜测一下 Simulation::GetTasks() 签名如下所示：

multimap<int,Task*> GetTasks() const;

每次您创建一个新的多重映射（一个副本）调用它。

比较迭代器时，两个 multimap 迭代器必须来自同一个容器；由于每次调用 GetTasks() 时都会获得一个新副本，因此违反了此​​约束，这就是错误的根源。您还有另一个问题 - 临时多重映射副本在创建它们的语句后被销毁，因此您的迭代器立即失效。

你有两个选择；一种是在本地捕获副本并一致地使用该副本：

multimap<int,Task*> tasks = simulation->GetTasks();
multimap<int,Task*>::iterator itTasks;
itTasks = tasks.begin();
while(itTasks != tasks.end()){
  while (itTasks->second->GetTimeStamp() == time->GetTime()){
        TaskExecute(itTasks->second,military,shalishut,args);
        itTasks++;
    }
    // Unit take car of vehicles
    time->TimeIncrease();
}

另一种是让 GetTasks() 返回对持久多重映射的引用，确保每次都使用相同的多重映射：

multimap<int,Task*> &GetTasks();

或常量引用：

const multimap<int,Task*> &GetTasks() const;

这具有避免复制multimap的（可能很大的）开销的优点。

请注意，使用 const 引用需要使用 const_iterator 来逐步执行多重映射。我建议同时定义 const 和非常量访问器（C++ 将根据 Simulation 指针或引用是否为 const 选择正确的访问器），除非您想禁止直接修改底层 multimap 完全，在这种情况下，您只能定义 const 变体。