Dziedziczenie klas i och wielkosc [C++]

Question

Witam,

Siedzialem wczoraj zaczynajac pisac powoli klasy do gry i gdy doszedlem do klasy Hero i Monster, no to odrazu pomyślalem, ze trzeba tu dziedziczyc z klasy entity. Wpakowalem wszystkie wspolne zmienne i funkcje ta w klase i tak sobie pomyslalem, ze zostala mi klasa NPC, ktora ma 4 takie same zmienne. Zastanawiam sie, czy powinnienem zrobić jeszcze jedna klase i wtedy bedzie to wygladalo tak:

klasa npc i entity dziedzicza z tej nowej klasy, a klasa Hero i Monster dziedzicza z entity.

Nie wiwm, czy rozbijac ta klase, czy zostawic tak jak jest i po prostu do NPC dopisac te 4 zmienne wiecej. Jak wy byacie ro rozwiazali?

Pozdrawiam

DC

Answer 1

A klasa NPC nie może dziedziczyć z Entity?

Comments

Ale wtedy dziedziczyla by funkcje i pola prywatne, ktore sa jej zbedne

---

A to jakie są te zbędne pola? Bo na oko typowy `entity` powinien być w całości potrzebny dla `NPC`.

---

W entity chcialem dac damage, defence, objekt klasy bonus. A npc potrzebuje tylko pozycji, nazwy, i teksty, ktore wymawia.

Dlatego pomyslalem nad zrobieniem jeszcze jednej klasy, z ktorej dziedziczylby entity i npc, a z entity dziedziczylby potwór i bohater

---

To Entity ma informacje dotyczące pozycji. To pod nią byłaby ewentualnie jakaś klasa z damage/defence etc.

(btw po terminologii potwór w zasadzie też jest NPCem)

---

Patrzac na terminologie to tak, ale chodzilo mi o NPC od ktorych dostaje sie misje itp. Moze pokaze jak wyglada klasa entity i inne klasy:

class Entity {
private:

    unsigned int widthEntity_;
    unsigned int lengthEntity_;
    
    std::string nameEntity_;
    unsigned long long targetMoneyEntity_;
    unsigned short minimalDamageEntity_;
    unsigned short maximalDamageEntity_;
    unsigned short damageEntity_;
    unsigned short defenceEntity_;
    unsigned short lifeEntity_;
    unsigned short strengthEntity_;
    unsigned short staminaEntity_;
    unsigned short sleightEntity_;
    unsigned short levelEntity_;
    unsigned short criticalDamageEntity_;
    
    bool livingEntity_;
    bool criticalDamageTrueFalseEntity_;
    
    int lifeEntity_;

};

class Monster : 
    public Entity {

};

class NPC {
unsigned int widthEntity_; 
unsigned int lengthEntity_; 
std::string nameEntity_;
std::vector <std::string> text_;
};

class Hero : 
    public Entity {

};

 

lub zrobie to tak: 

class glowna_klasa {
    unsigned int widthEntity_;
    unsigned int lengthEntity_;
    
    std::string nameEntity_;
};

class Entity : 
    public glowna_klasa{
private:
    unsigned long long targetMoneyEntity_;
    unsigned short minimalDamageEntity_;
    unsigned short maximalDamageEntity_;
    unsigned short damageEntity_;
    unsigned short defenceEntity_;
    unsigned short lifeEntity_;
    unsigned short strengthEntity_;
    unsigned short staminaEntity_;
    unsigned short sleightEntity_;
    unsigned short levelEntity_;
    unsigned short criticalDamageEntity_;
    
    bool livingEntity_;
    bool criticalDamageTrueFalseEntity_;
    
    int lifeEntity_;

};

class Monster : 
    public Entity {

};

class NPC :
    public glowna_klasa {
};

class Hero : 
    public Entity {

};

 

---

No i OK, tylko byłbym za czytelniejszym nazewnictwem: `glowna_klasa` raczej nazwałbym `Entity`, a obecne `Entity` nazwałbym na przykład `CombatEntity`.

I proszę wyrzuć te wszystkie `...Entity_` z nazw pól :(

---

moge wywalic, wydawalo mi sie, ze przez to beda one czytelniejsze