My guess is that you have a bad "player" pointer. You should check with your debugger to be sure.
You may need to either allocate a new player object using "new Player;". The alternative would be to allocate the player on the stack (or by value in an object).
The first step is to stop using pointers unless you want "optional" arguments. Use references instead.
void Inn(Player &thePlayer){ // ... if (answer == 'y') { //Exception occurs here if (thePlayer.GetGold() >= 10) { thePlayer.SpendGold(10); cout << "You find a nice bed and fall fast asleep."; Wait(); cout << "\nHP and MP restored\n"; cout << "HP: " << thePlayer.GetMaxHP() << " MP: " << thePlayer.GetMaxMana() << "\n"; thePlayer.SetHP(thePlayer.GetMaxHP()); thePlayer.SetMana(thePlayer.GetMaxMana()); Wait(); } // ...}
I must also point out that your manipulation of private member variables is bad style. Instead of manually calling Set*() and Get*() on the player, give the
actions you are doing names and implement these instead as member functions.
class Player{public: void restore(); // or sleep, or whatever void loseHp( int amount ); void castSpell( const Spell & ); void printStats( std::istream & );};
In a well written program, you will have very few "set" functions. "get" functions aren't quite so bad, as sometimes you need to inspect an object. Still, replacing these kind of functions with domain specific functions is far better.
