Надо делать наследование или нет?

Question

[Max Ba]

Никак понять не могу. В примерах по ООП часто пишут, есть класс User и наследники UserStandart, UserAdmin, ну примерно так, +-.

А у меня класс User, в котором сразу прописаны все методы управления. А то, каков статус пользователя, записано в свойстве: 1 обычный юзер, 2 админ.

И я вот не пойму, зачем делать наследование в данном случае, ради примера, или тут есть смысл?

Answer 1

[Vasyl Fomin]

Если у вас всегда экземпляр класса User имеет одинаковые методы и поля, то и классов отдельных не нужно делать. Тем более что конкретно в вашем примере функционал юзера, можно отделить ролями/правами, которые могут быть реализованы позднее отдельно.

Comments

[Max Ba]

У меня в некоторых методах есть проверка

if(!$this->admin()){
  return 'Доступ запрещен';
}

Просто непонятно зачем учат разделять на классы ну в сосем нелогичных задачах :)

[Vasyl Fomin]

Макс Васильев, думаю, еще, было бы правильнее сделать метод для проверки роли, и передавать роль как аргумент, а не отдельно писать код проверки роли. А вдруг вы захотите проверить есть ли в пользователя одна из ролей (admin, manager, ...), например:

//....
 protected function checkRole($roleName) {
   if($roleName == 'admin' && $this->admin()){
     return true;
   }
   if($roleName == 'manager' && $this->manager()){
     return true;
   }
   return false;
 }
//...

Можно еще этот метод вынести в trait, и передавать методу еще экземпляр (объект) пользователя. Тогда вы, подключая трей в свои классы, сможете в любом месте проверять роль пользователя.
Но это так, отступление от темы, куда можно углубляться еще далеко....

Answer 2

[Arman]

if($user instanceof UserAdmin) {
//admin
}
public function openWebsite(UserAdmin $userAdmin) {
// 
}

Точно знаем что юзер админ или просим чтоб юзер был только админом, как там реализована проверка на админство уже этот код не касается.

Answer 3

[Александр Гетманский]

Есть хороший способ определить рациональность использования наследования. Для этого нужно ответить на ряд вопросов:
- оба класса из одной предметной области;
- наследник отвечает принципу подстановки Барбары Лисков;
- код наследуемого класса необходим либо хорошо подходит для наследника;
- наследник в основном добавляет логику.
Чем больше положительных ответов, тем больше вероятность, что вы не промахнетесь с выбором наследования.

Конкретно в вашем случае я рискну предположить, что использование наследования излишне, и даже может стать бомбой замедленного действия при дальнейшем развитии системы. Здесь наследование выступает в роли выбора возможных алгоритмов действий пользователя в системе и введение новых ролей может оказаться очень дорогим, особенно если один пользователь должен будет обладать сразу несколькими ролями.

Answer 4

[Yan-s]

Надо или нет определяете вы проектируя архитектуру приложения. Примеры пишутся для объяснения сути, что наследование, это когда один объект - более конкретная разновидность другого.

Comments

[Max Ba]

Понятно. Значит это лишь для объяснения.
Думал неправильно учу)
Спасибо.

[Yan-s]

Макс Васильев, почитайте общие книги по программированию и проектированию. Можете начать с Совершенный Код С. Макконнелл. Будет яснее когда, что, для чего и почему применять.

Answer 5

[Петр]

У вас может быть в классе User описано лишь поле Name и поле LastName.
А в конкретной реализации вы можете дописать для каждого конкретного юзера уже свои методы и дополнительные поля. Все от проекта зависит.

Comments

[Max Ba]

Да не, у меня проект то обычный пока что. да и учусь)
Спасибо. Понял в целом всё.

[Петр]

Макс Васильев, Удачи в учебе:)