The Entity class looks like this:
Code:class Entity { public: bool hidestate; Entity* parent; };I'm getting a crash on this line:Code:bool Entity::IsHidden() { if (hidestate==true) return true; Entity* e = parent; while (e!=NULL) { if (e->hidestate==true) return true; e = e->parent; } return false; }
Am I doing something obviously wrong? Thanks.Code:if (e->hidestate==true) return true;
You might want to initialize parent to NULL in an Entity constructor -- if you don't, it might well be a random, non-NULL value, which you will then try to dereference in IsHidden(). This constructor might look like e.g.
By the way, if I could suggest a simpler implementation of IsHidden():Code:Entity::Entity() : parent(NULL) { }
Or something along those lines.Code:bool Entity::IsHidden() { if(hidestate) return true; if(parent && parent->IsHidden()) return true; return false; }
dwk
Seek and ye shall find. quaere et invenies.
"Simplicity does not precede complexity, but follows it." -- Alan Perlis
"Testing can only prove the presence of bugs, not their absence." -- Edsger Dijkstra
"The only real mistake is the one from which we learn nothing." -- John Powell
Other boards: DaniWeb, TPS
Unofficial Wiki FAQ: cpwiki.sf.net
My website: http://dwks.theprogrammingsite.com/
Projects: codeform, xuni, atlantis, nort, etc.
Thanks. I am learning a lot really fast!
