Rozbijanie klas na mniejsze czesci.

Question

Witam, czy takie rozbicie jest poprawne. Muszę użyć rekurencji a reszta w setterach i getterach.

    class Hero
    {
    public:
    int strenght_hero;
	int stamin_hero;
	int sleight_hero;
	
	void set_reszta()
	{
	int intelligence_hero;
	int state_money_beginn;
	int deffence_hero;
	int life_hero;
	int minimal_damage_hero;
	int maximal_damge_hero;
	int damage_hero;
	int hero_lenght;
	int hero_width;
	int lvl_hero;
	int exp_hero;
	int point_for_statystick;
	string klasa_hero;
	}
	void get_reszta()
	{
	   int intelligence_hero;
	int state_money_beginn;
	int deffence_hero;
	int life_hero;
	int minimal_damage_hero;
	int maximal_damge_hero;
	int damage_hero;
	int hero_lenght;
	int hero_width;
	int lvl_hero;
	int exp_hero;
	int point_for_statystick;
	string klasa_hero; 
	}
    Hero(int strenght,int stamin,int sleight,int intelligence,int money,int lenght,int width,int lvl,int exp,int point,string klasa);
    void hero_attributs_description();
	};

 

Answer 1

Tak się zastanawiam.. czy ty zdajesz sobie sprawę z tego co napisałeś (mówię o kodzie).
Z twojego zapisu wynika, że po wywołaniu funkcji set_reszta (tak samo jak get_reszta) stworzysz mnóstwo zmiennych tymczasowych, które po zakończeniu działania funkcji nie będą już istnieć. Zmienne dla klasy muszą się znajdować w klasie, a nie w jakiejś funkcji.
Jeżeli chcesz sobie porozdzielać kod na mniejsze części to zrób po prostu do tego osobne struktury (możesz je umieścić oczywiście w klasie Hero, ale nie wolno zapomnieć o stworzeniu obiektów/struktur).
Przykładowo:

class Hero
{
public:
   struct Type_A
   {
      int var_1, var_2, var_3;
   }
   struct Type_B
   {
      int var_1, var_2, var_3;
   }

   Type_A Armor;
   Type_B Dmg;
   void Set_Hero()
   {
      Armor.var_1=10;
      Armor.var_2=20;
      Dmg.var_2=4;
      Dmg.var_3=123;
   }
};

 

Comments

Dziękuje o coś takiego mi chodziło.