[Вопрос С++: Наследование] Создание пустых классов.

[MooNDeaR]

Вопрос такой. В общем понадобилась вот такая конструкция:

...

class Man
{
   ...
};

class Player : public Man
{
   int health;
...
public:
   int get_health(){return health;}
   int change_health(int dam){ health -= dam; return health;}
...
};

class Enemy : public Man
{
   int health;
...
public:
   int get_health(){return health;}
   int change_health(int dam){ health -= dam; return health;}
...
};
...

Man *pl, *en;
pl = new Player;
en = new Enemy;

int strike(Man* M,int damage)
{
  ...
   M->change_health(damage);
  ...
   return M->get_health();
}

Дык вот вопрос: Это всё будет работать?
Или необходимо описать get_health() и change_health() как virtual? А потом переопределять их в потомках?

[veniside]

работать не будет, да и самому разве не очевидно, что health (и всё, что с ним связано) надо поместить в class Man, тогда и Player и Enemy его унаследуют, и дублирование пропадёт.
А virtual понадобится только тогда, когда появится некоторый объект, у которго change_health() будет работать по-другому.

[MooNDeaR]

Т.е. health и пр. перенести в секцию public класса man и наследовать?

P.S.

А если мне необходимо будет обращатся к методам, которые не являются методами класса Man?
Тогда надо будет определить класс Man как абстрактный и добавить в него чисто виртуальных методов?

P.P.S.

А если я создам 2 чисто виртуальных метода get_bullets() и get_knifes(). Причем get_bullets() - будет использоваться в классе Player,но не будет использоваться в классе Enemy, а get_knifes() с точностью да наоборот? Что делать с "ненужными генами" ?

[netrino]

health в private, а get_health/change_health в public. Man должен содержать в себе всё общее, что имеют между собой классы Player и Enemy.

А если я создам 2 чисто виртуальных метода get_bullets() и get_knifes(). Причем get_bullets() - будет использоваться в классе Player,но не будет использоваться в классе Enemy, а get_knifes() с точностью да наоборот? Что делать с "ненужными генами" ?

сделать get_ammo и не париться)
А вообще, не получится определить только один из двух нереализованных (чисто виртуальных) методов. Для создания объекта класса, должны быть определены все методы, объявленные в нём.

[MooNDeaR]

Эт я знаю. Ну я тогда просто возвращаю ноль, ибо их действительно у него нету. В принципе я разобрался. Спасибо. Только пихать health в private нельзя. Иначе при создании конструктора для класса Player вида:

Player(int he = 100){health = he;}

компиль ругается на то, что нету доступа к private элементам

[Пепел Феникса]

class Man
{
private:
    int health;
public:
    Man(int _health):health(_health)
    {

    }
    virtual ~Man()
    {

    }
};

class Player:public Man
{
public:
    Player(int _health):Man(_health)
    {

    }
    virtual ~Player()
    {

    }
};

[MooNDeaR]

Хммммм Как полезно иногда задать глупый вопрос Спасибо.

[Пепел Феникса]

еще можно сделать такое:

class Man
{
private:
    int __health;
public:
    Man(int _health):__health(_health)
    {

    }
    virtual ~Man()
    {

    }
    int health()//get health
    {
        return __health;
    }
    void health(int _health)//set health, если надо
    {
        __health=_health;//тут можно сделать еще проверочку на maxhealth
    }
    void heal(int amount)
    {
        __health+=amount;
    }
    void injure(int amount)
    {
        if(__health<=amount)
        {
            __health=0;
        }
        else
        {
            __health-=amount;
        }
    }  
};

[An1ka]

Сообщение от MooNDeaR

Т.е. health и пр. перенести в секцию public класса man и наследовать?
Тогда надо будет определить класс Man как абстрактный и добавить в него чисто виртуальных методов?

Не обязательно, абстрактный класс нужен только для того, чтобы для него нельзя было создать объектов. Да и вообще виртуальные методы нужны только, если тип указателя не будет соответствовать типу объекта и еще при наследовании можно virtual указывать, тогда дублироваться не будет класс при множественном наследовании.
Можно переменную health в секцию protected и она будут использоваться наследниками.

[MooNDeaR]

Да и вообще виртуальные методы нужны только, если тип указателя не будет соответствовать типу объекта

В принципе у меня там так и получалось.

еще можно сделать такое:

Это да Просто вопрос был на образном коде