C ++从对象向量中删除

时间:2013-08-28 00:10:54

标签: c++ c++11 vector crash

我有一些包含对象的向量,我有一系列嵌套循环来获取我想要的对象

if (_dir->Instance()->isDebug())
{
    Utils::LogTextWithInt("growing timers size: ", _glo->Instance()->getGrowingTimers().size());
    for (int i=0; i < _glo->Instance()->getGrowingTimers().size(); i++)
    {
         GrowingTimer _growingTimer = _glo->Instance()->getGrowingTimers().at(i);

        std::cout << "growing timer Field id and plant id: " << _growingTimer.getFieldID() << " - "
                    << _growingTimer.getPlantID() << std::endl;
    }
}

std::vector<GrowingTimer>::iterator _gtIterB = _glo->Instance()->getGrowingTimers().begin();

for (_gtIterB; _gtIterB != _glo->Instance()->getGrowingTimers().end(); ++_gtIterB)
{       
    for (std::vector<Field>::iterator _fIterB = _glo->Instance()->getFields().begin();
         _fIterB != _glo->Instance()->getFields().end(); ++_fIterB)
    {          
        if (_gtIterB->getFieldID() == _fIterB->getFieldNumber())
        {
            for (std::vector<Plant>::iterator _pIterB = _fIterB->getPlants().begin();
                 _pIterB != _fIterB->getPlants().end(); )
            {
                if (_gtIterB->getPlantID() == _pIterB->getPlantID())
                {
                    Utils::LogText("gt and plant ID's match");
                    Utils::LogTextWithInt("Plant ID after matching: ", _pIterB->getPlantID());

                    // Wiggle our plant.
                    Utils::wiggleNode(_pIterB->getPlantSprite(), 10.0f, 5.0f);

                    _pIterB->setPlantStoppedGrowing(true);

                    _gtIterB = _glo->Instance()->getGrowingTimers().erase(_gtIterB);

                    ++_pIterB;
                }
                else
                {
                   ++_pIterB;
                }

            }
        }
    }
}

输出结果为:

growing timers size:  2
growing timer Field id and plant id: 7620 - -2130819608
growing timer Field id and plant id: 7620 - -2130802800
gt field id:  7620 
gt plant id:  -2130819608
gt and plant ID's match
Plant ID after matching:  -2130819608
deleted
gt and plant ID's match
Plant ID after matching:  -2130802800
deleted
gt field id:  7620
gt plant id:  -2130802800
gt and plant ID's match
Plant ID after matching:  -2130802800

在所有这些之后我崩溃了:

0x209eff0:  addb   %ah, %gs:115(%ecx,%ebp,2)
0x209eff5:  jo     0x209f063                 ; "nitWithCondition:"
0x209eff7:  popal  
0x209eff8:  jns    0x209f048                 ; "'v'"
0x209effa:  popal  
0x209effb:  insl   
0x209effc:  incl   %esi
0x209effe:  outsl  
0x209efff:  jb     0x209f04c                 ; "ckBeforeDate:"
0x209f001:  jns    0x209f03e                 ; "Name:"
0x209f004:  jbe    0x209f067                 ; "ithCondition:"
0x209f006:  insb   
0x209f007:  jne    0x209f06e                 ; "ition:"
0x209f009:  cmpb   (%eax), %al
0x209f00b:  popl   %edi
0x209f00c:  jo     0x209f080                 ; "lSinceNow"

基本上现在我有一个匹配并且我采取行动我想删除_gIterB中的位置,因为它不再需要。如果你想要运行的定时事件,这个向量会继续作为“跟踪器”添加,这样当定时器完成时我们知道定时器的对象是什么,并且可以返回它以执行后续步骤。

编辑:GrowingTimer的向量存储了fieldID和正在“增长”的PlantID,因此我们可以回到它。

Fields的矢量存储用户拥有的所有字段,并且作为成员,每个Field Object都有一个字段中植物的矢量。 8个植物到1个田地。

用户创建的每个字段都有一个唯一的ID,下面的输出是有一个字段,其中包含2个植物。

1 个答案:

答案 0 :(得分:0)

for (auto _gtIterB=grow_timers.begin(); _gtIterB!=grow_timers.end(); ++_gtIterB) {     
    ...
    _gtIterB = _glo->Instance()->getGrowingTimers().erase(_gtIterB);
    .... 
}

基本上,这里有两个问题。

  • 当您擦除_gtIterB时,为其指定 next 增长计时器的位置。这意味着您已经跳过将此种植者与许多植物进行比较。我认为这很糟糕,但可能不会导致你的崩溃。
  • 如果你删除了_gtIterB,它是最后一个有效的增长器,它被设置为结束迭代器,那么当你到达for循环的下一部分时,你再次++_gtIterB,导致它超过结束,导致谁知道发生了什么。这绝对是坏事。

我不确定其中任何一个都会导致你的崩溃。

但是,我对你的代码的解释更快,深度嵌套:http://ideone.com/zcfnv8