ich hab eine Frage die allein die korrekte Speicherreservierung betrifft.
Und zwar geht es um folgendes:

struct TEST
{
    char* text;
};

int main()
{
    TEST test;
    test.text = new char[6];
    test.text = "Hallo";

    // mache irgendwas

    delete[] test.text;
}

Erzeugt einen Speicherfehler.

Folgendes nicht:

int main()
{
    TEST* test;
    test = new TEST;
    test->text = "Hallo";

    // mache irgendwas

    delete[] test;
}

Mich wurmt allerdings, dass test->text nun nicht explizit Speicher zugewiesen wurde und frage mich, ob das korrekt ist.
Mach ich für test->text auch noch ein new und delete so erhalte ich wieder einen Fehler.

Ich würd mich freuen, wenn mich jemand darüber aufklären könnte, was korrekt ist und wieso - und ob ich zwingend einen Zeiger auf meinen struct benutzen muss.

lg Max

Beides ist nicht korrekt. Ob du das struct nun auf dem Stack oder auf dem Heap anlegst ist dabei nicht wichtig. Denn der Zeiger text wird in beiden Fällen zweimal gesetzt : Zuerst weist du ihm eine Adresse eines manuell reserviertem Speichers zu, und danach weist du ihm die Adresse eines konstanten StringLiterals zu. Der vorher reservierte Speicher geht dabei verloren und wird nicht für das StringLiteral genutzt (wie du wahrscheinlich annimmst).
Deshalb ist das danach folgende delete auch ungültig, String-Literale dürfen nicht gelöscht werden. Das bei den beiden Varianten einmal eine Speicherverletzung und einmal nicht angezeigt wird ist Zufall, denn nicht korrekter Code erzeugt undefiniertes Verhalten.

Also zwei Hinweise
- benutzt strncpy um den Inhalt des String-Literals in einen anderen Speicherbereich zu kopieren. C-Strings werden nicht kopiert wenn man zwei Pointer zuweist.
- benutzt lieber std::string...

bis bald
akari

Hast Du einen ordentlichen Link für die Speicherverwaltung?

lg Max

Du speicherst dir in der Klasse ja lediglich einen Zeiger auf char. Wenn du da jetzt aber speicher zuweist und dann nachher per = "Hallo" einen neuen Speicher zuweist, dann geht der Speicher verloren. Das ist bis jetzt so noch kein Problem. Das Problem kommt erst, wenn du du den Speicher freigeben willst. (Wenn du das delete auskommentierts, dann gibts auch keinen Fehler, der Memory Leak bleibt aber).
Das untere Beispiel macht etwas ganz anderes. Du fordrst das Objekt test an und machst nacher das richtige mit einem Zeiger.Jedoch solltest du das mit delete freigeben und nicht mit delete[], da es ja kein Array ist.

Ich war mir sicher im zweiten Beispiel auch ein

test->text = new char[6];

zu sehen, wurde es schon wegeditiert?

Tutorials zum Thema C-Strings wirst du eher im Ansi-C Forum finden, hier im C++ Forum vermeidet man rohe Speicherverwaltung lieber und benutzt std::string.

bis bald
akari

Hallo

Ich war mir sicher im zweiten Beispiel auch ein

test->text = new char[6];

zu sehen, wurde es schon wegeditiert? Aber auch ohne das sind beide nicht gültig, durch das delete auf ein String-Literal.

Tutorials zum Thema C-Strings wirst du eher im Ansi-C Forum finden, hier im C++ Forum vermeidet man rohe Speicherverwaltung lieber und benutzt std::string.

bis bald
akari

Wenn er delete test; macht, dann pasts wieder. (Er fordert ja das Objekt an und nicht den string).

Ich merk schon wir kommen hier schon zeitlich durcheinander... bleibt gar nicht genug Zeit seine Beiträge zu korrigieren

So wie das zweite Beispiel jetzt dasteht hat drakon natürlich Recht.

bis bald
akari

Danke erstmal, d.h. also folgendes wäre korrekt:

struct TEST
{
    char* text;
};

int main()
{
    TEST* test;
    test = new TEST;
    test->text = "Hallo";

    // mache irgendwas

    delete test;
}

Was wiederum bedeutet, dass mit "new TEST" auch für test->text Speicher reserviert wird? Aber wieviel ist das?

lg Max

struct TEST
{
    const char* text;
};

Was wiederum bedeutet, dass mit "new TEST" auch für test->text Speicher reserviert wird? Aber wieviel ist das?

Genau so viel wie ein Zeiger auf deinem Rechner benötigt (vermutlich 4 oder 8 Byte).

lg Max

das beispiel, das du suchst, um zu verstehen, was man dir schreibt:

char* text = new char[6];
strncpy (text, "Hallo", 6);
delete[] text;

dieses beispiel macht genau das, was du in deinem allerersten beispiel versuchtest du zu erreichen; nur dass es funktioniert.

Funktioniert! Danke!

lg Max

Gibt es eigentlich einen guten Grund den ganzen Kram auf den Heap zu schaufeln?

Ich denke mal, dass er das mehr als Übung gemacht hat und nicht als reeles Beispiel.

Der eigentliche Zweck soll sein eine beliebige Anzahl an structs dieser Art (d.h. mit enthaltenen Zeigern) erstellen zu können, ohne bei Nichtnutzung überflüssigen Speicherplatz reserviert zu haben.

lg Max