🚀 go-pugleaf

Het probleem ligt aan het statement
_LangeString = new char[1];

Je voorziet slechts plaats voor 1 enkel character. Dit werkt zolang de
string leeg is, want dan heb je enkel een terminating 0 nodig. Wanneer je
echter woordjes begint toe te voegen, dan zal 'strcat' geheugen voorbij deze
ene byte beginnen kapot te schrijven. Daardoor raakt je heap corrupt. Visual
C++ detecteert dit tijdens runtime. De "DAMAGE" waarvan sprake legt uit dat
je heap corrupt is, omdat een geheugenblok beginnend op adres 0x300040 aan
het einde overschreven werd. Als je de waarde van de pointer _LangeString
bekijkt, dan zul je zien dat deze gelijk is aan 0x300040!

Het feit dat het toevallig werkt met andere compilers wil niet zeggen dat
het programma correct is. Indien je deze code zou inlassen in een groter
programma, dan zou dat programma naar alle waarschijnlijkheid om mysterieuze
redenen de mist ingaan op het meest onverwachte ogenblik (althans als we
Murphy mogen geloven)...

Als je de structuur van het programma ongewijzigd wenst te laten, en je wilt
dat de array automatisch groeit, dan zul je de methode setString moeten
aanpassen. Je zult daar een nieuwe array moeten alloceren, bv met

_lengte = strlen(string);

// Nieuwe lengte=oude string+nieuwe string+0 character

int newLength=_lengte+strlen(_LangeString)+1;

if (_LangeString[0]!=0)

newLength++; // reserveer ook plaats voor de spatie als niet eerste woord

char *tmpArray = new char [newLength+1];

strcpy(tmpArray, _LangeString);

if (_LangeString[0]!=0)

strcat(tmpArray, " ");

strcat(tmpArray, string);

if (_LangeString[0]==0)

tmpArray[0] = toupper(tmpArray[0]);

delete[] _LangeString;

_LangeString = tmpArray;

return;



Iets gelijkaardigs moet gebeuren in getLangeString(), want daar wordt ook
een character toegevoegd. Als je dit gedoe wenst te vermijden, dan is het
raadzaam om de class std::string te gebruiken, die achter de schermen al
deze (de)allocaties volledig automatisch uitvoert, inclusief een ganse boel
optimalisaties!

Luc K