Thread: help w/ classes

>i tried something like this
First, your code isn't legal C++. Second, your inheritance hierarchy could be improved. Rather than focusing on equipment, you can go higher up and go with items. Then weapons, armor, potions, poisons and such can derive from the item class. But since an item is abstract, it would be little more than an interface. Likewise with anything that isn't a concrete item. You want your hierarchy to be wide and flat rather than thin and tall. It's easier to manage that way:

Code:

// Top of the tree, base interface
class Item {
  int buy() = 0;
  int sell() = 0;
};

// All weapons derive from this interface
class Weapon: public Item {
  int attack() = 0;
};

// All armor derives from this interface
class Armor: public Item {
  int defense() = 0;
};

// All potions derive from this interface
class Potion: public Item {
  int charges() = 0;
};

This gives you a workable tree of behaviors that you can make concrete without any intermingling of unnecessaries:

Code:

                                                      Item

               Weapon                                 Armor                           Potion

 Sword          Axe          Bow         Chest        Head        Legs          Cure         Antidote

Muramasa     OgresBane   SilverWing    IronCuirass  SteelHelm  LeatherPants

The trick is to imagine the amount of work it would take to add new items or change existing ones. And don't forget, abstract classes are your friends.

>"missing ';' befor '}'" (it gives this on every line where a class is declared)
A class definition is terminated with a semicolon. The first error will usually cause other errors that don't really exist. Fix them from the top down and you'll have an easier time of things. Also keep in mind that one problem can cause tons of errors, so even if you have 30 or so errors, they might boil down to a only single bug. So don't be overwhelmed.

thank you.

i think i have a better understanding of how classes and inheritance work now. the book i am using was hard to follow and used examples that i couldn't change at all to fit what i was tring to do so it was hard to figure out the right way to do it. after looking at the examples you gave i think i could tweek it enough to fit the game. i do have 1 qustion to ask still. Was i correct in how I created the spacific items?

ie.

Code:

Short_sword.cost = 50;
Short_sword.bat_power = 1

>Was i correct in how I created the spacific items?
Not for the name, but the other members were okay. However, it's better to make any data members private and then provide a public member function if users need to access them. That way you can restrict access as necessary. Kind of like pants. When you wear them properly, you have complete control over who can play with what's inside them.

For initialization, a constructor is preferable, especially if you use proper data hiding and make your data members private.

I suggest setting the name as part of the constructor. And as Daved said, use std::string.