推回对象，然后在 std::list 中的先前位置删除它

Question

请注意，顺序可以是任意一种（先擦除然后推回，只是这种方式不需要创建对该对象的本地引用）。

  for ( GameObjItr gameObj = m_gameObjects.begin();
        gameObj != m_gameObjects.end(); gameObj++ ) {
    if ( *gameObj && *gameObj != gameObject ) {
      const sf::FloatRect &otherObj = ( *gameObj )->GetCollisionBox();
      if( mainObj.Intersects( otherObj ) ) {
        m_gameObjects.splice( m_gameObjects.end(), m_gameObjects, gameObj );
        return m_gameObjects.back();
      }
    }
  }

  return NULL;
}

这个想法是，如果函数发现一个对象与传递的对象发生冲突，则会将该对象添加到列表的末尾，将其从当前位置删除，然后返回该对象。我不认为迭代器失效是一个问题，因为我在删除后立即返回？

该代码导致程序崩溃。

PS cppreference 说擦除“删除”对象，但我认为这只是将其从列表中删除。 （如果这是错误的，我如何通过元素的位置删除元素。我正在阅读的删除函数文档仅允许传递对象的值）。

编辑：崩溃的位置并不总是相同，但它是由于此代码而发生的（即，如果我恢复它，则行为正常）。是什么导致了不同时间的延迟崩溃？我不希望发现的物体被毁坏。

GameObject* GameObjectManager::DetectCollision( const GameObject *
  gameObject ) {
  const sf::FloatRect &mainObj = gameObject->GetCollisionBox();

  for ( GameObjItr gameObj = m_gameObjects.begin();
        gameObj != m_gameObjects.end(); gameObj++ ) {
    if ( *gameObj && *gameObj != gameObject ) {
      const sf::FloatRect &otherObj = ( *gameObj )->GetCollisionBox();
      if( mainObj.Intersects( otherObj ) ) {
        m_gameObjects.splice( m_gameObjects.end(), m_gameObjects, gameObj ); //<--------
        return m_gameObjects.back(); //<-------------
      }
    }
  }

  return NULL;
}

编辑 发现错误，当将对象移动到列表末尾时，导致 2 个游戏对象碰撞之间的无限循环。抱歉，无法从发布的代码中破译。

Answer 1

当您说：

m_gameObjects.erase( gameObj );

将调用要删除的列表中包含的内容（如果有）的析构函数。从你的问题中不清楚这个东西的类型是什么，或者是否需要这个析构函数调用。

Answer 2

“m_gameObjects.splice( m_gameObjects.end(), m_gameObjects, gameObj );”
您想将 gameObj 移动到列表中的最后一个位置吗？这就是上面一行所暗示的。如果是，那么
如果 gameObj == m_gameObjects.end() 或 ++gameObj == m_gameObjects.end()
这是一个 NULL 操作。

另一件事是，您在 for 循环中对迭代器进行后增量，使其前增量。但这与崩溃无关。