0

Обычный пример с юнитом.

class Unit {
private:
    int health;

public:
    Unit(int health);
    virtual ~Unit();

protected:
    virtual void setHealth(int newHealth);

далее делаем наследника

class Healer : public Unit {
public:
    Healer(int health);

    virtual void healing(Unit* other);
};

В реализации лечения:

void Healer::healing(Unit* other) {
    other->setHealth(100);
}

И вот инкапсуляция не разрешает так делать. other->setHealth(100);

'setHealth' is a protected member of 'Unit' other->setHealth(newHealth);

can only access this member on an object of type 'Healer' virtual void setHealth(int newHealth);

При этом для this вызвать этот метод можно.

5
  • пробовал переопределять setHealth в наследнике. но это так же не помогло
    – Johnny K.
    Commented 16 нояб. 2018 в 13:23
  • сделать публичным
    – Саске
    Commented 16 нояб. 2018 в 13:38
  • 1
    внутри юнита сделать метод для работы с приватным полем
    – Саске
    Commented 16 нояб. 2018 в 13:39
  • Вообще можно сделать Healer другом(friend) Unit, но я вам этого делать не рекомендую. Ваши "победы" над инкапсуляцией приведут к тому что в код станет сложно воносить изменения. Если юниту может менять хп кто-то, кроме самого юнита, то делайте этот метод публичнм, и не водите в заблуждение того, кому придется читать этот код(например будущего себя) Commented 16 нояб. 2018 в 13:56
  • Ну а если сделать метод публичным, тогда из мэйна изменять ХП смогут все кому не лень. ведь не зря сеттеры делают приватными. а если сделать публичный метод лечения у Юнита, то это тоже архитектурно не правильно, так как лечением занимается только один наследник юнита, а не все его наследники
    – Johnny K.
    Commented 16 нояб. 2018 в 14:36

1 ответ 1

-1

Мне, кажется, надо добавить вам интерфейс типа IHealable например. И, далее, действовать через него. Например сделать ещё один класс HealableUnit: Unit, IHealable и в методе healing работать с интерфейсом, а не с методом.

0

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.