Speicher allokieren fuer Objekte [Geloest]

[heinz]

Hallo Zusammen,

ich programmiere ab und zu "Dinge" fuer mich die ich gerade brauche.
Und wenn ich etwas mit Objekten mache (was sehr oft der Fall ist) dann mach ich das meist so: (Pseudo-Code aus dem Kopf...)

Code: Alles auswählen

class objeKt
{
 public:
  void Mach_was();
 private:
  int iregendwas;
};

int objektmenge=0;
objeKt** objektliste = NULL;

void Neues_objekt()
{
 objektmenge++;
 objektliste = (objeKt**) realloc(objektliste , objektmenge * sizeof(objeKt*));
 objektliste[objektmenge-1] = new objeKt();
}

for(int o=0;o<objektmenge;o++)
{
 delete(objektliste[o]);
}
free(objektliste);

Das funktioniert wunderbar in sehr vielen meier Projekte.

Allerdings lese ich immer wieder, "Man soll malloc, calloc, realloc und free nicht mit new und delete verwenden."

Warum nicht?
Und wie soll man es sonst machen, wenn vorher nicht bekannt ist wieviele Objekte es mal werden?
Mit new kann ich mir zwar Speicher allokieren aber danach die Groesse nicht mehr aendern.

Wie macht man es denn "richtig"?
Bzw. Was ist an "meinem" Code so falsch/gefaehrlich/unzureichend wasauchimmer...

Gruss,
heinz

[MSfree]

Allerdings lese ich immer wieder, "Man soll malloc, calloc, realloc und free nicht mit new und delete verwenden."

Grundsätzlich sollte man in C++ gar kein malloc/calloc und free verwenden.

Ausserdem kann man in C++ die new und delete Operatoren überschreiben. Delete führt ausserdem den Destruktor der jeweiligen Klasse aus. Objekte, die man mit (m|c)alloc anlegt und mit delete freigibt, könnten im Destruktor landen, was dann in einem Segmentationfault enden kann.

Ich würde dir aber vorschalgen, dir man die STL reinzuziehen. std::vector ist eine praktische Templateklasse, bei der du dir um Speicher anfordern und Freigabe gar keine Gedanken mehr zu machen brauchst.

[heinz]

Hallo MSfree,

erstmal Danke fuer Deine Antwort.

Ich hab mal ein wenig im Netz gesucht und ein paar Beschreibungen ueber vector gelesen.
Dann hab ich versucht das Gelesene umzusetzen und habe prompt ein Problem.

Hier ist mein Testcode:
41220

Und hier die Ausgabe die er erzeugt:

Code: Alles auswählen

---------------------------------------- Objekte erstellen
Objekt 0 erzeugt.
Objekt 0 vernichtet.
Objekt 0 macht was...
 Objektmenge= 1
Objekt 1 erzeugt.
Objekt 0 vernichtet.
Objekt 1 vernichtet.
Objekt 1 macht was...
 Objektmenge= 2
Objekt 2 erzeugt.
Objekt 0 vernichtet.
Objekt 1 vernichtet.
Objekt 2 vernichtet.
Objekt 2 macht was...
 Objektmenge= 3
Objekt 3 erzeugt.
Objekt 3 vernichtet.
Objekt 3 macht was...
 Objektmenge= 4
Objekt 4 erzeugt.
Objekt 0 vernichtet.
Objekt 1 vernichtet.
Objekt 2 vernichtet.
Objekt 3 vernichtet.
Objekt 4 vernichtet.
Objekt 4 macht was...
 Objektmenge= 5
----------------------------------- Objekte was tun lassen
Objekt 0 macht was...
Objekt 1 macht was...
Objekt 2 macht was...
Objekt 3 macht was...
Objekt 4 macht was...
--------------------------------------- Objekte vernichten
Objekt 0 vernichtet.
Objekt 1 vernichtet.
Objekt 2 vernichtet.
Objekt 3 vernichtet.
Objekt 4 vernichtet.

Warum wird beim Erzeugen der Objekte immer der Destructor aufgerufen?
Und warum werden immer die Destructoren aller anderen Objekte ebenfalls aufgerufen? (Ausser bei Objekt 3???)

Warum kann ich die Zeilen 50 und 51 einfach austauschen ohne eine Veraenderung festzustellen?

Was mach ich falsch, wo liegen meine Denkfehler?

Gruss,
heinz

[MSfree]

Code: Alles auswählen

for(int o=0;o<5;o++)
{
       objektliste.push_back(o);
//     objektliste.push_back(objeKt(o)); // Warum geht das auch?
       objektliste[o].Machwas();
       printf(" Objektmenge= %li\n",objektliste.size());
}

Code: Alles auswählen

objektliste.push_back(o);

erwartet als Argument für push_back ja eigentlich ein Objekt der Klasse objeKt. Du fütterst es aber mit dem int "o". Also wird ein temporäres Objekt erzeugt, dann gepusht und anschließend wird das temporäre Objekt wieder zerstört.

Code: Alles auswählen

objektliste.push_back(objeKt(o)); 

Hier erzeugst du das Objekt objeKt explizit als Übergabeparameter.

[heinz]

Hallo nochmal,

ich glaube, ich hab es hinbekommen:

Code: Alles auswählen

int main()
{
printf("---------------------------------------- Objekte erstellen\n");

	std::vector<objeKt*> objektliste;


	for(int o=0;o<5;o++)
	{
		objeKt* ob=new objeKt(o);
		objektliste.push_back(ob);
		objektliste[o]->Machwas();
		printf(" Objektmenge= %li\n",objektliste.size());
	}

printf("----------------------------------- Objekte was tun lassen\n");

	for(int o=0;o<5;o++)
	{
		objektliste.at(o)->Machwas();
	}

printf("--------------------------------------- Objekte vernichten\n");

	objektliste.clear();
}

Das erzeugt eine Ausgabe wie sie sein soll.

Ich verstehe es so:
Der vector "objektliste" enthaelt eine Liste (Array) von Pointern auf Objekte im Speicher.
Und das bedeutet auch, delete nicht zu verwenden sondern z.B. objektliste.pop_back(); und somit das loeschen der Objekte dem vector zu ueberlassen.

Ist das so korrekt?

Gruss,
heinz

[heinz]

Nochmal Danke fuer Deine Hilfe.

Also wird ein temporäres Objekt erzeugt,

Wenn die Objekte nur temporaer waren, warum konnte ich dann spaeter die Methode "objektliste[o].Machwas();" aufrufen? (Und beim ".clear();" auch die Destructoren...)

[MSfree]

Wenn die Objekte nur temporaer waren, ...

Das temporäre Objekt ist nur das, welches sich in der Parameterliste von push_back befindet. Im std::vector wird dieses temporäre Objekt per Copy-Contructor in den Vektor eingefügt.

[heinz]

Alles klar.

Ich Denke/Hoffe/Glaube ich habs jetzt verstanden.

Vielen Dank fuer Deine Zeit.

Gruss,
heinz