Когда стоит наследоваться от класса?

165

26 августа 2017, 01:33

Ведь можно сделать просто:

class DB {
    // все методы работы с БД
}
class Logger extends DB {
    // все методы логирования в БД и файлы
}
class Valirator extends Logger {
    // все методы проверки форм
}
class User extends Valirator {
    // все методы работы с юзером
}

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

Почему так как написано в коде выше это плохо?
По какому принципу в приложении выбирают кто и от кого должен наследоваться?

Answer 1

Плохого в это то, что нарушается принцип единственной ответственности. Например тут "у меня в User досупны все методы всех родителей". Зачем вашему User уметь, да и вообще знать, про какое-то там логирование?

Наследоваться имеет смысл если и родитель и потомок действительно являются "родственниками".

Т.е. например "Авторизованный пользователь" вполне логично отнаследовать от "Пользователь", а вот наследовать "Пользователя" от "Базы данных" - это какой-то оксюморон.

Вот так норм:

AuthorizedUser extends User
PrivilegedUser extends AuthorizedUser

А вот так нет

Table extends User
User extends Autobus
Autobus extends Database

UPD

С точки зрения объектного подхода к проектированию, User у вас - это отражение актора "Пользователь", который в принципе не должен знать о том, как устроена логика вашего приложения. Validator, Logger и Database- это, по хорошему, интерфейсы, а не объекты, с которыми, опять же по хорошему, User взаимодействовать не должен.

Рассмотрим use case - пользователь заходит в систему и по этому факту происходит его проверка и логирование.

Пользователь вводит логин и пароль
Подсистема авторизации передает в объект класса, реализующий интерфейс Validator введенные данные.
Validator проверяет корректность и передает объекту класса, реализующего интерфейс Logger, данные для логирования
Конкретный Logger, наследующий от DatabaseSource или FileSource (соответственно DbLogger или FileLogger) совершает логирование
Подсистеме авторизации возвращается соответствующий объект User, PriveledgedUser, ReadOnlyUser, etc... Либо ошибка авторизации (Exception например)