Incrementing/Decrementing Object's hitpoints using classes

I can't seem to increment/decrement an object's hitpoints in a game. For example...

Code:

---


#include <iostream>

class Player
{
public:
    int getHitPoints() const { return HitPoints; }
    void setHitPoints(int hit) { HitPoints = hit; }
private:
    int HitPoints;
};

int main()
{
    using namespace std;
   
    Player Bob;
    Bob.setHitPoints(10);

    cout << "Bob gets hit!" << Bob.getHitPoints() - 2 << endl;

    return 0;
}

---

Any help is appreciated.

What is the problem?

Kuphryn

This

Code:

---

Bob.getHitPoints() - 2

---

never actually assigns to anything the value of Bob.getHitPoints() - 2. SOmething along the lines of

Code:

---

Bob.setHitPoints(Bob.getHitPoints() - 2);

---

would, however, work. On the other hand, you may wish to simply have a function that will decrement the hitpoints for you.

Can you post an example?

I tried this, but when I checked his new health status, it went back up to 10.

Code:

---


cout << "Bob gets hit!" << (Bob.getHitPoints() - 2) << endl; // 8
cout << "Bob checks his health!" << Bob.getHitPoints() <<  endl;
              // 10

---

Code:

---

(Bob.getHitPoints() - 2)

---

Isn't assigning anything to your class. It's calling the method getHitPoints and getting the value 10 or whatever and then decrementing 2 from that and displaying it. To actually change it you need to call the method for setting the hitpoints in your class. As it has already been stated, something like

Code:

---

Bob.setHitPoints(Bob.getHitPoints() - 2);

---

You could also make setHitPoints return an int that is the new health value if you wanted to.

You have to make a seperate class function like void decrementhp(int amt) to subtract or add hitpoints. You can't actually subtract from the function itself.

Quote:

---

Originally posted by KneeGrow
You have to make a seperate class function like void decrementhp(int amt) to subtract or add hitpoints. You can't actually subtract from the function itself.

---

But you can... See the examples that Mr Wizard and I posted.

Quote:

---

Originally posted by Zach L.
But you can... See the examples that Mr Wizard and I posted.

---

lol my bad

Having things like GetHP() and SetHP() is not really a very good OOP way of doing things. Your way offers almost no benefit over a struct; you aren't hiding the data or improving access to it, you're only obfuscating access to it.

A good way for your class is to do this:

1) Have a constructor which sets the character's max HP, and sets the current HP to max.

2) Have two functions -- Heal() and Harm(). Heal() increments the HP, Harm() decrements. Both of them do bounds checking (i.e. you can never get above max HP or below 0)

3) Have an isDead() function that tells if the current HP is positive.

4) Have a HP() function to get the current HP, but do NOT allow a direct set of HP.