„Vector Iteratoren unvereinbar“, wenn sie in einem Selbstversorger-Vektorschleife Löschen

Question

kann ich um Hilfe bitten, um zu bestätigen, ob mein Problem von einem Design-Problem kommt, oder wenn es würde eine mögliche saubere Lösung für die folgenden sein:

Entity.h

class CLEntity3D 
{ 
public: 
    CLEntity3D(); 
    virtual ~CLEntity3D(); 
    virtual void update() = 0; 

    static std::vector<CLEntity3D*> vecEntity; 
}; 

Entity.cpp

int CLEntity3D::nbrEntity = 0; 
std::vector<CLEntity3D*> CLEntity3D::vecEntity; 

CLEntity3D::CLEntity3D() 
{ 
    vecEntity.push_back(this); 
} 
CLEntity3D::~CLEntity3D() 
{ 
    vecEntity.erase((std::remove(vecEntity.begin(), vecEntity.end(), this)), vecEntity.end());  
} 

verschiedene abgeleitete Klasse erstellen/löschen verschiedene Entitäten durch das Programm-Objekt, das alles funktioniert gut.

In einer Szene Klasse, ich habe die folgenden Methoden:

void CLScene::Update() 
{ 
    for (auto& iter : CLEntity3D::vecEntity) { 
     iter->update(); 
    } 
} 

void CLScene::ClearScene() 
{ 
    for (auto& iter : CLEntity3D::vecEntity) { 
     delete(iter); iter = nullptr; 
    } 
    CLEntity3D::vecEntity.clear(); 
} 

-Update ist in Ordnung, das Problem mit ClearScene ist(). Ich bekomme eine "Vector Iterators Inkompatible" Debug-Assertion.

Aus meiner Forschung scheint das allgemeine Problem zu sein, weil die Iteratoren von verschiedenen Vektoren sind, die ich glaube nicht, ist das Problem hier. Ich denke, das Problem ist, wenn ClearScene() aufgerufen wird, jedes delete (iter) ändert die Größe von vecEntity durch den CLEntity3D-Destruktor und macht daher den Iterator in der ClearScene-Schleife ungültig. Habe ich recht?

Meine Frage wäre dann: Gibt es eine Möglichkeit, alle Clentity3D-Objekte aus CLScene mit diesem Design zu löschen?

Ich glaube, ich könnte CLScene die vecEntity halten, die das Problem beseitigen würde, aber dies würde bedeuten, dass CLScene müssten alle Schöpfung/Löschen von Einheiten verwalten, also nicht so vielseitig sein ...

PS: Ich weiß, dass dieses Beispiel nicht zu kompilieren ist, aber da meine Frage mehr nach Konzept ist ... bitte rate mir, ob ich etwas anderes bereitstellen soll.

Answer 1

Das Problem ist, Sie können nichts aus dem zugrunde liegenden Vektor entfernen, während in einem Bereich für Schleife basiert.

Die Schleife in Ihrer ClearScene Methode löscht CLEntity3D Instanzen, die in seinem Destruktor den gleichen Vektor ändert, den Sie in Ihrer for-Schleife verwendet haben.

Eine relativ einfache Lösung wäre Ihre ClearScene zu so etwas zu ändern:

void CLScene::ClearScene() 
{ 
    auto vectorCopy = CLEntity3D::vecEntity; 
    for (auto& iter : vectorCopy) { 
     delete iter; 
    } 
} 

Das funktioniert, weil die Schleife auf einer Kopie arbeitet, und das Entfernen geschieht auf dem Original.

Beachten Sie, dass der ursprüngliche Vektor nach der Schleife nicht gelöscht werden muss, da die Destruktoren sicherstellen, dass der Vektor nach dem Löschen aller Elemente leer ist.

Oder, wie durch einen Kommentar vorgeschlagen, könnten Sie die Kopie unter Verwendung einer while-Schleife vermeiden:

while (!CLEntity3D::vecEntity.empty()) 
{ 
    delete CLEntity3D::vecEntity.begin(); 
}