Löschen von Vektor von Zeigern

Question

Ich brauche Zeiger der Instanzen einer Klasse zu erstellen, und das Programm weiß nicht zur Kompilierzeit, wie viele Zeiger ich erstellen werde. Zum Löschen dachte ich darüber nach, die Zeiger in einem Vektor zu speichern und dann einzeln zu löschen. Würde die Verwendung von Smart Pointers einen saubereren Weg zu gehen? Und wenn man nicht intelligente Zeiger verwenden möchte, würde diese Verwendung von Vektor als sauber angesehen werden?

Mindestcode:

#include <vector> 
using namespace std; 

class Foo { 
public: 
    Foo(); 
}; 
Foo::Foo(){} 
void createFooVector(int nb, std::vector<Foo*> &v){ 
    for(int i=0;i<nb;i++){ 
     Foo* f = new Foo(); 
     v.push_back(f); 
    } 
} 
int main(int argc, char *argv[]){ 
    std::vector<Foo*> v; 
    createFooVector(5,v); 
    while (!v.empty()){ 
     Foo* f = v.back(); 
     v.pop_back(); 
     delete f; 
    } 
} 

Answer 1

Ich würde vorschlagen, entweder eine boost::pointer_vector verwenden, eine std::vector<std::unique_ptr<Foo>>, oder rollen Sie Ihre eigenen Foo Manager-Klasse aus, die eine vector<Foo*> hält und kümmert sich um Löschungen in den Konstruktor (Sie sollten dies als „Experte“ Lösung sehen, und nur Versuch wenn Sie die Ausnahmesicherheit vollständig verstehen). Sie möchten die Löschung nicht manuell vornehmen, was leicht zu Fehlern führen kann.

Answer 2

Der Code ist in Ordnung. Die Verwendung von Smartpointern sollte jedoch die bevorzugte Wahl sein (weniger Code zum Schreiben und viel weniger Möglichkeiten für Speicherfehler).

Answer 3

Wäre die Verwendung von Smart Pointers ein sauberer Weg zu gehen?

Ja.

Und wenn man nicht intelligente Zeiger verwenden möchte, würde diese Verwendung von Vektor als sauber angesehen werden?

Ich habe keine Ideen, warum jemand nicht verwenden intelligente Zeiger in C++ will, wenn es keine Hausaufgaben sind ... Aber ich denke, es ist besser, so etwas wie boost::pointer_containers in diesem Fall zu verwenden.

Answer 4

Wenn Sie keine Klassen verwenden, die von Foo abgeleitet sind, und Foo relativ kostengünstig zu kopieren ist, verwenden Sie einfach vector<Foo>.

Wenn Ihr Compiler die Bewegungssemantik unterstützt, sollte es kein Problem geben.