Konstruktor/Destruktor (bei dynamischer Speicherreservierung)

[trudi (Gast)]

Hallo ich habe eine Frage zu einem Konstruktor bzw. zum Destruktor.

Kessel ist mein Class.


Reicht es  wenn ich den Destruktor so wie unten schreibe oder muss ich 
das anderst programmieren da ich ja im Konstruktor den Speicher für 
"Name"  dynamisch anlege

•

Kessel(char*Name,int X, int Y, double Inhalt, double Temperatur)
        { 
        Kessel::Name = new char[strlen(Name)+1];                                    // Name wird in neu reservierten Speicherbreich kopiert...
        strcpy(Kessel::Name, Name);   
        }
       
       
~Kessel ()

grüße Trudi

[manni (Gast)]

In diesem Fall musst Du den Speicher - spätestens im Dekonstruktor - 
selbst wieder freigeben, sonst entsteht ein Speicherleck. D.h., der 
Speicher würde erst beim Beenden des Prozesses freigegeben. Allerdings 
muss man noch weitere Dinge beachten - insbesondere beim Kopieren von 
Objekten des Typs "Kessel".

Generell: Verwende besser die C++-String-Klassen. Und wenn man die 
Grundlagen des manuellen Speicherhandlings verstanden hat, sollte man 
sich die Smart-Pointer-Klassen ansehen.

BTW: Einen leeren Dekonstruktor müsstest Du nicht explizit 
hinschreiben, er wird automatisch vom Compiler generiert.

[Karl H. (kbuchegg)]

trudi schrieb:

> das anderst programmieren da ich ja im Konstruktor den Speicher für
> "Name"  dynamisch anlege

Grundregel:
Für alles, was du im Programm mittels 'new' anlegst, bist auch du 
verantwortlich, dass es ein zugehöriges 'delete' gibt. Und zwar nur ein 
einziges pro Pointer-Wert!

PS: Achtung.
Du verwendest ein 'new []'. Also musst du auch das zugehörige 'delete 
[]' benutzen.

[Rolf Magnus (Gast)]

Und sehr wichtig: Die "Rule of Three" beachten:
Wenn du einen Destruktor, einen Kopierkonstruktor oder einen 
Zuweisungsoperator brauchst, dann mußt du dich immer um alle drei 
kümmern.
Aber wie manni schon schrieb, wäre hier std::string das Mittel der Wahl, 
um sich gar nicht erst manuell um Speicher kümmern zu müssen.