spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Timo, der Fehler kommt, weil Du das "+1" vergessen hast. Ausserdem darfst Du nicht sizeof benutzen, sondern strlen. Die Fehlermeldung sagt ja auch genau was passiert: Du schreibst hinter das Ende des von Dir (per new) allozierten Speichers.

Die string.h bewirkt keine Wunder. Es liegt wohl eher an deinem Code. In der string.h sind nur die String-Operationen Deklariert, sonst nichts. Vieleicht hat dein Compiler auch nur bei deinen Änderungen nicht alles neu gemacht. Das kann schon mal vorkommen. Daran sind schon viele verzweifelt ;D

Noch ein Tipp zu deiner Klasse. Du solltest noch eine zusätzliche Member einbringen. Diese speichert dann die länge der Zeichenkette (mit oder ohne 0Byte).

Sachen wie diese sollte man nicht machen: Damit ist ein Fehler vorprogrammiert. Denn strlen liefert die länge des Strings und dazu gehört nicht das 0Byte.

@realloc:
Stimmt mit realloc kann man einen Speicherbereich in seiner Größe verändern. Allerdings sind new und realloc nicht Kompatible zu einander. Benutzt man beide in Kombination kann[/b es zu nicht Definierten Fehlferhalten kommen. Wenn man Speicher reservieren will, sollte man sich am besten immer entscheiden ob man die C++ Variante nimmt, oder halt die C Variante.

@realloc:

Natürlich (vergaß ich zu schreiben) muss man die C-Variante zur Allozierung benutzen ( ... = (char*)alloc(X)) ).
Allerdings frage ich mich, ob es in den C++ Standartbibliotheken auch etwas "realloc-Ähnliches" gibt, das kompatibel zu "new" ist...

In Deinem Beitrag befinden sich noch Fehler.
Bitte bearbeite die rot markierten Stellen und lösche diesen Text.

Keine Ahnung. Gefunden hab ich bis jetzt jedenfalls noch nichts. Solange man eine Kombination auf die Std. Typen (char, int, etc.) beschränkt, sollte nichts weiter passieren. Nur sobald man mit Klassen arbeitet, dürfte es Probleme geben.

Hallo

Bei klassen, deren Hauptmember man wahlweise setzen kann, und die dann dynamisch Speicher reserwieren lohnt es sich (vor allem wenn die Klasen größer und unübersichtlicher werden) einige allgemeine Definitionen einzuhalten.

1. zeigt ein Pointer auf Mist, sollte man ihn auf NULL setzen (z. B. nach dem Löschen mit delete oder delete[])

2. wird für nen Pointer (z. B. mit new) Speicher angefordert so sollte man immer prüfen, ob der Zeiger auch NULL ist. Ist er es nicht, so sollte man ihn zuerst löschen, oder ne Fehlermeldung ausgeben (es gibt Fälle, in denen das logisch sinnvoll wäre).

Dadurch erübrigt sich die Verwendung von new char[0] im Konstruktor. Du setzt das Feld erst mal NULL.

PS: ich würd nur Speicher für das char array reservieren, wenn das notwendig ist, und nur soviel wie notwendig ist.

Samuel G.

Realloc:
realloc muss wahrscheinlich auch öfter einfach neuen speicher anlegen und den alten speicher hineinkopieren. Wenn allerdings hinter dem reservierten speicher noch genügen speicher frei ist, dann könnte realloc diesen zu vergrößerung nutzen(was natürlich deutlich schneller gehen würde).
Ich weiß nicht genau wie realloc funktioniert. Das hier sind also nur vermutungen.

Zu malloc und new:
Man kann afaik auch den speicher an sich mit malloc reservieren und dann mit placement new sein objekt darin platzieren. Wie man die objekte dann wieder zerstört weis ich nicht mehr genau, aber ich denke, dass man einfach nur den destructor aufrufen muss.