habe ein Problem mit der dynamischen Speicherzuweisung mittel calloc bzw. realloc. Hier ein kleines Beispiel, was das Problem verdeutlichen soll :p

#include <cstdlib>
#include <iostream>
#include <string>

using namespace std;

int main (void)
{
	string* strPtr = NULL;
	string str = "hallo";
	strPtr = (string*) calloc(1, sizeof(string));
	strPtr[0] = str;
	strPtr = (string*) realloc(strPtr, 2 * sizeof(string));
	strPtr[1] = str;
	cout << strPtr[1] << endl;

	return 0;
}

Bis zum calloc und 'strPtr[0] = str; ' funktioniert alles super. Die realloc anweisung scheint er auch noch auszuführen. Bei der Zuweisung bekomme ich dann jedoch eine Fehlermeldung:

Eine Ausnahme (erste Chance) bei 0x004221ac in List.exe: 0xC0000005: Zugriffsverletzung beim Schreiben an Position 0xbaadf00d.
Unbehandelte Ausnahme bei 0x004221ac in List.exe: 0xC0000005: Zugriffsverletzung beim Schreiben an Position 0xbaadf00d.

Kann mir jemand sagen, was den Fehler verursacht?
Danke schon mal im Voraus.

PS.:Mit int, double, etc. funktioniert das super...

Diese Funktionen rufen nämlich weder Konstruktor noch Destruktor auf. Benutze new und delete bzw. new[] und delete[] , wenn es nötig ist, oder arbeite gleich auf dem Stack.

Das ich das ganze mit
string str = new string[x];*
lösen kann, das weiß ich ja. Es gibt aber beispielsweise keine möglichkeit eine vergrößerung vorzunehmen. Ich weiß auch wohl, dass sich das mit einer for(;;;) Schleife recht schnell lösen lässt, aber das ist nicht Sinn der Sache. Ich möchte viel mehr verstehen warum realloc nicht funktioniert und was ich machen kann, damit es funktioniert.

Das Hauptproblem bei Dir ist, dass die string Objekte gar nicht richtig initialisiert wurden. In deinem Array steht Müll (ungültiger Zustand eines string -Objektes) drin. Das zweite Problem ist, dass realloc ggf die Daten bitweise zu einem neuen Block kopiert und den Kopier-Konstruktor und Destruktor der Klasse übergeht.

Gruß,
SP

Das ich das ganze mit
string str = new string[x];*
lösen kann, das weiß ich ja. Es gibt aber beispielsweise keine möglichkeit eine vergrößerung vorzunehmen.

--> std::vector<std::string>

(Es kann so einfach sein)

Gruß,
SP

Ich möchte viel mehr verstehen warum realloc nicht funktioniert und was ich machen kann, damit es funktioniert.

Gar nichts. Es geht nicht. realloc ist unberechenbar und für non-PODs einfach nicht zu gebrauchen. Das Problem mit realloc ist, dass es sich die Freiheit nimmt, den alten Speicherblock freizugeben und einen neuen zu reservieren -- ohne dass man als Nutzer die Gelegenheit bekommt, die Objekte ordnungsgemäß zu kopieren/zerstören.

Und selbst wenn Du eine weniger freche realloc Funktion hättest -- eine, die vorher zurückfragt, wie die Daten denn zu kopieren sind, falls der Block nicht einfach vergrößert werden kann -- wäre es den Aufwand gegenüber std::vector einfach nicht wert. Da haben sich schon ein paar Leute die Zähne dran ausgebissen.

Gruß,
SP

Sebastian Pizer schrieb:

In deinem Array steht Müll

Meiner Kenntnis nach steht in den Speicherstellen, die mit calloc / realloc reserviert werden Nullen. Weil die Funktionen jedes Bit der Speicherstellen auf Null setzen. Zumindest laut Referenz.

Dann schreibe ich mir die Funktion zum Array vergrößern eben doch selbst. Das ist schade.

Nimm doch, wie schon erwähnt, std::vector<T>!!!!
Der geht für alle Typen (solange sie kopierbar sind) und wenn nötig, kannst Du mit &vec[0] direkt auf den Speicher zugreifen (z.B. für C Interoperabilität).
Simon

ich weiß doch
Habe euch wohl etwas verwirrt Aber zu Übungszwecken schreibe ich mir gerade meine eigene vector Klasse. Dann wäre es ja blöd, wenn ich sie benutzen würde. Über Sinn und Unsinn kann man natürlich streiten.
Auf jeden Fall ist das der Grund, warum ich den speicher selber verwalten möchte

Setze das jetzt gerade mit new[] um!

Meiner Kenntnis nach steht in den Speicherstellen, die mit calloc / realloc reserviert werden Nullen. Weil die Funktionen jedes Bit der Speicherstellen auf Null setzen. Zumindest laut Referenz.

nur calloc initialisiert mit 0

Sebastian Pizer schrieb:

In deinem Array steht Müll

Meiner Kenntnis nach steht in den Speicherstellen, die mit calloc / realloc reserviert werden Nullen. Weil die Funktionen jedes Bit der Speicherstellen auf Null setzen. Zumindest laut Referenz.

Nullen sind Müll, wenn es um einen std::string geht**!** Du darfst Objekte von Non-POD-Klassen nicht manuell initialisieren, dafür gibt es Konstruktoren.

kronos_m schrieb:

Setze das jetzt gerade mit new[] um!

Ein Tipp: std::vector reserviert meistens für ein bisschen mehr Elemente Speicher als tatsächlich belegt werden. Wenn du das auch machst, kannst du dir in den meisten Fällen teure Reallokationen sparen, wenn du nur ein Element anhängst.