Was ist der richtige Weg, um Speicher im C++ - Konstruktor zuzuordnen?

Question

Welcher ist der richtige Weg, um Speicher über new im C++ - Konstruktor zuzuweisen. Erste Art und Weise in der Argumentliste:

class Boda { 
    int *memory; 
    public: 
     Boda(int length) : memory(new int [length]) {} 
     ~Boda() { delete [] memory; } 
}; 

oder im Körper des Konstruktor:

class Boda { 
    int *memory; 
    public: 
     Boda(int length) { 
      memory = new int [length]; 
     } 
     ~Boda() { delete [] memory; } 
}; 

Danke, Boda cyclo.

Answer 1

Ich denke, der einfachste Weg, dies zu tun wäre, eine boost scoped array verwenden und lassen Sie jemand anderen gut getestet Bibliothekscode alles für Sie.

So:

class Boda { 
    boost::scoped_array<int> memory; 
    public: 
     Boda(int length) : memory(new int [length]) {} 
     ~Boda() {} 
}; 

Außerdem scoped Arrays können nicht kopiert werden - so vermeiden Sie die fiese Copykonstruktor Deallokation Problem in einer anderen Antwort erwähnt.

Answer 2

Wenn Sie Speicherzuordnungsfehler abfangen möchten (was Sie wahrscheinlich tun sollten), müssen Sie den Aufruf von new im Rumpf des Konstruktors vornehmen.

Answer 3

Ich würde sagen, beide sind gleichwertig in der Wirkung, die sie produzieren und beide sind "der richtige Weg". Ich bevorzuge Initialisierungslisten, aber ich würde mit der zweiten Variante gehen, nur in der Lage sein, auf ungültige Länge Argument zu testen, bevor Sie versuchen, den Speicher zuzuordnen.

Answer 4

Das Problem ist allgemeiner. Siehe C++ FAQ Lite: [10.6] Should my constructors use "initialization lists" or "assignment"?

Answer 5

Sie sollten Ressourcenverwaltungsklassen verwenden, die dies für Sie erledigen. Sonst gibt es einige ernsthafte Probleme mit der Ausnahmesicherheit, abgesehen von unnötiger Duplizierung bestehender Logik und Wartung von Kopier-/Zuweisungsoperatoren.

Answer 6

memory Membervariable ist ein Zeiger, wenn Sie es in Initialisierungsliste zuteilen und es fehlschlägt, Ihre Klasse nicht initialisiert und Sie brauchen es nicht zu befreien später (dank RAH-Entwurfsmuster, das durch C verwendet wird ++ zur Klasseninitialisierung). Wenn Sie seinen Speicher im Körper des Konstruktors zuweisen, wird ähnliches Verhalten auftreten.

Aber wenn Sie etwas behandeln wollen, dann ordnen Sie seinen Speicher im Körper des Konstruktors. Überprüfen Sie etwas oder versuchen Sie, es abzufangen oder einige nützliche Nachrichten auszudrucken, aber Sie müssen zumindest eine weitere Ausnahme auslösen, da Ihre Klasseninitialisierung fehlerhaft ist.

Ich denke, memory Zuordnung im Körper des Konstruktors ist lesbarer als die andere.