[c++] destructor problemen

[erkje]

Ik heb een Node die een child en een sibling heeft. Als ik een child wil setten, moet ie dus de ouwe weggooien, alleen krijg ik een error.

child & sibling zijn van het type Node *

De error die ik krijg zit bij de delete in setChild(...)
_BLOCK_TYPE_IS_VALID(pHead->nBlockUse)

deze error zegt me niet zoveel, maar als ik de delete weghaal, werkt ie wel... of als ik this->child = c weghaal, werkt ie ook

Hier de relevante code inclusief de destructor die (mijns inziens) aangeroepen wordt door de delete:

C++:

void Node::setChild(Node * c)
{
    if( child ) {
        delete child;
    }

    this->child = c;
}

Node::~Node()
{
    if( child ) {
        delete child;
    }

    if( sibling ) {
        delete sibling;
    }

    if( c ) {
        delete c;
    }
}

ow ja, die c is van het type Component (class) wat dus de inhoud van een Node is

[ Voor 13% gewijzigd door erkje op 26-03-2004 13:04 ]

[devvy]

Wat ik zou kunnen bedenken is, dat je de childNode na de setChild() ook nog ergens anders ge-delete wordt.
Dus iets in de geest van:

C++:

obj.setChild(child);
delete child;

[Zoijar]

Het kan ook goed fout gaan als er ergens een cykel in je graaf zit. Ik weet niet precies wat voor structuur je hebt maar als het zo iets is:

node 
  |
 node    -> node(3)
  |             |
 node(1) -> node(2)

En je wist node (1) dan wordt 2 ook gewist, maar dat weet 3 niet. Als die dan gewist wordt gaat die voor de tweede keer 2 wissen...crash.

Misschien moet je alleen children wissen, en geen siblings. Of reference counting smart pointers...

[ Voor 25% gewijzigd door Zoijar op 26-03-2004 13:30 ]

[MSalters]

Rule of three:
Als je een destructor, copy ctor of copy assignment operator hebt, dan heb je ze waarschinlijk alle drie nodig.

[cobratbq]

in dit stukje:

C:

if( c ) {
        delete c;
} 

is de 'c' wel een globaal?
of is dit alleen een tijdelijke lokale var zoals je die daarboven gebruikt hebt?

[ Voor 5% gewijzigd door cobratbq op 26-03-2004 14:04 ]

[.oisyn]

Geef eens wat meer info, hier kunnen we niets mee. Laat bijvoorbeeld eens zien hoe je de boomstructuur in elkaar zet

[farlane]

Owhja, en dit

C++:

if( blaat )
    delete blaat;

is een beetje onzinning, delete van een 0 pointer is een no-op.

[riezebosch]

TS is bekende van me, heb in z'n code gekeken en volgens mij delete hij een pointer die helemaal niet aangemaakt is met new...