Quote:Original post by matthProjectile *projSwap;projSwap = new Projectile();//this is a memory leakprojSwap = (*projNode);Enemy *enemySwap;enemySwap = new Enemy(); //this is a memory leakenemySwap = (*enemyNode);}
// Check for Collisionsfor( std::list<Projectile*>::iterator projNode = ProjectileList.begin(); projNode != ProjectileList.end(); ++projNode ){ Projectile *projSwap = (*projNode); if(SpriteCollisionTest(dynamic_cast<Sprite&>(*projSwap), dynamic_cast<Sprite&>(*playerMothra))) { (*projNode)->Kill(); } else { for( std::list<Enemy*>::iterator enemyNode = EnemyList.begin(); enemyNode != EnemyList.end(); ++enemyNode ) { Enemy *enemySwap = (*enemyNode); if(SpriteCollisionTest(dynamic_cast<Sprite&>(*projSwap), dynamic_cast<Sprite&>(*enemySwap))) { (*projNode)->Kill(); (*enemyNode)->Hit(); } //delete enemySwap; <---remove } } //delete projSwap; <---remove}Quote:Original post by Moss
Keep in mind smart pointers are NOT STL container safe. I normally store
plain old pointers in the container and when I need to move it around I throw
it into a smart pointer.
Quote:Original post by Moss
The following should work:
*** Source Snippet Removed ***
Quote:Original post by matth
Thanks for all the help guys - as usual I am able to find my errors a whole lot easier once I start asking questions.
I changed the following line :
if(SpriteCollisionTest(dynamic_cast<Sprite&>(*projSwap), dynamic_cast<Sprite&>(*enemySwap)))
over to:
if(SpriteCollisionTest(dynamic_cast<Sprite&>(**projNode), dynamic_cast<Sprite&>(**enemyNode)))
And took out all the unneeded enemySwap/projSwap stuff
Quote:Source: me [smile]Quote:if(SpriteCollisionTest(dynamic_cast<Sprite&>(*projSwap), dynamic_cast<Sprite&>(*playerMothra)))
Will not work as you intend, as a bad dynamic_cast on a reference will throw an exception (it cannot return a "null" reference because no such thing exists). It is one of the few times it is preferable to use a pointer over a reference.
