//This is not a compilable example, just bits and pieces

class CPlayer
{
public:
   CWeapon WeaponWielded;
   CArmor  ArmorWorn;
   CShield ShieldEquipped;
   void*   Inventory[10]; // This is where I lose it
};

CPlayer::Createchar()
{
   WeaponWielded = WeaponList[0];
   Inventory[0] = (CWeapon) WeaponList[1];
//Also tried these
   (CWeapon) player.Inventory[0] = WeaponList[1];
   Inventory[0] = WeaponList[1];
   *Inventory[0] = WeaponList[1];
   &Inventory[0] = (CWeapon) WeaponList[1];
   (CWeapon) &Inventory[0] = WeaponList[1];
}
 

quote: Original post by siaspete
I'd try the CItem base class, not void pointer. Though I'd recommend you finish Tetris first.

Helpful links:
How To Ask Questions The Smart Way | Google can help with your question | Search MSDN for help with standard C or Windows functions

    class CItem{public:   char* name;   int weight;   int value;};class CWeapon : public CItem{public:   int Damage;   int Hitbonus;   int dmgtype;};  

        CWeapon WeaponList[2] = {{"Shortsword", 3, 100, 6, 0, 1},{"Longsword",  5, 200, 8, 0, 1}}//This is assuming that the variable order on this class will go//Name, Weight, Value, Dmg, HitBonus, DmgType.////Is this a correct assumption?  

quote: Original post by siaspete
I wouldn''t count on that working. But since you''re doing it in code anyway, why not just have constructors that take all those parameters?

Helpful links:
How To Ask Questions The Smart Way | Google can help with your question | Search MSDN for help with standard C or Windows functions

  CWeapon::CWeapon(char* thename, int thedmg, int theweight, int thevalue){   name = thename;   dmg = thedmg;   weight = theweight;   value = thevalue;}//Then in Main, assuming WeaponList is similar to aboveint main(){     Cplayer player;     Weapon weapon(WeaponList[0]);     player.weaponwielded = weapon;}  

Weapon myweapon("Shortsword", 3, 100, 6, 0, 1); 

Weapon::Weapon(Weapon the_template){   name = the_template.name;   dmg = the_template.dmg;   weight = the_template.weight;   value = the_template.value;    } 

quote: Original post by Mezz
Using your example you''d call it like this:

Weapon myweapon("Shortsword", 3, 100, 6, 0, 1);  

The idea is you put parameters in for it''s creation when you actually instantiate it.

Bear in mind you can actually make multiple constructors, so if you wanted to call it as you have (with the weaponlist[0] parameter) you''d have to make one like:

Weapon::Weapon(Weapon the_template){   name = the_template.name;   dmg = the_template.dmg;   weight = the_template.weight;   value = the_template.value;    }  

You''d probably prefer to use a few pointers (or better in C++, use references) in there to avoid copying everything in the parameter.
Hopefully that''ll help you out a bit.

-Mezz

WeaponList[MAXWEAPS] = {{"Shortsword", 5, 3 ,2},etc.} 

quote: Original post by Radagar
Any Suggestions?

    #include <iostream>#include <vector>#include <boost/smart_ptr.hpp>class base{public:	virtual void f() const =0;	virtual ~base()	{}};class derived1 : public base{public:	void f() const	{		std::cout << "derived1::f()\n";	}};class derived2 : public base{public:	void f() const	{		std::cout << "derived2::f()\n";	}};int main(){	typedef boost::shared_ptr<base> itemPtr;	typedef std::vector<itemPtr> itemVect;	itemVect items;	items.push_back(itemPtr(new derived1));	items.push_back(itemPtr(new derived1));	items.push_back(itemPtr(new derived2));	items.push_back(itemPtr(new derived1));	items.push_back(itemPtr(new derived2));	itemVect::const_iterator it = items.begin();	for(; it != items.end(); ++it)	{		(*it)->f();	}}