Является ли это исключительной ситуацией?

Question

[topuserman]

Например есть класс, который в конструктор получает некоторые входные данные, которые необходимы для инициализации объекта.

Например:

class User {
    private ?array $userData;

    public function __construct(int $userId) {
        $this->userData = ...;
        // ...
    }

    // ...

    public function getId(): int {
        return $this->userData["ID"];
    }

    public function getFullName(): string {
        return implode( " ",
            clearArray( [
                    $this->userData["NAME1"],
                    $this->userData["NAME2"],
                    $this->userData["NAME3"]
            ] )
        );
    }

    public function getLogin(): string {
        return $this->userData["LOGIN"];
    }

    public function getEmail(): string {
        return $this->userData["EMAIL"];
    }
}

Что делать, в случаях, когда по входным данным невозможно инициализировать объект ? Например, по заданному Id не удалось найти пользователя ? Выбрасывать исключение ? или создать доп. метод isExist(): bool ?

Я склонен к первому варианту, т.к. во втором случае, нужно будет, каждый раз, перед использование методов класса, писать проверку if($user->isExist()) ..

Как принято решать подобные задачи ?

Comments

[Victor Bomberow]

Если конструктор кидает исключение, то это ООП курильщика.

[Максим Федоров]

Vito Ombero, InvalidArgumentException

[Максим Федоров]

Vito Ombero, а вообще автор ересь городит не понятную

[topuserman]

Vito Ombero, хорошо, а как решаются такие проблемы ?

Класс сущности, может в конструкторе получать Ид, по которому будет получена сущность из БД. Что делать, если запись в бд не найдена, или какие-то важных данных нет, и нельзя инициализировать сущность ?

[topuserman]

Максим Федоров, а как не ересь городить?

[Антон Антон]

topuserman, Не надо передавать id в конструктор, надо туда передавать уже полученную сущность (строку БД, еще что). А то как тестировать будете? А вообще не надо изобретать ORM, пользуйтесь готовыми. Ну или если очень хочется, почитайте про фабричные методы, инверсию зависимостей и прочую теорию.

[topuserman]

Антон Антон, паттерн фабричного метода тут не причем, а IoC хз как тут использовать.

[topuserman]

Скорее всего, нужно делать так:

Создаем репозиторий UserRepository, у которого есть метод getById,
который будет возвращать null || UserEntity

[topuserman]

Антон Антон, а как тестируют репозитории ?)

Answer 1

[Максим]

В сущность должны передаваться конкретные параметры: $id, $login, а не массив $userData, из которого потребуется извлекать эти данные, проверять их тип и существование. При таком подходе вы можете использовать типизацию и будете уверены в переданных данных.

Вы так же можете создавать сущность разными фабричными методами.

Исключение должно быть в методе get() вашего репозитория. И не путайте методы find() и get():

• find() — возвращает сущность или null.
  
• get() — возвращает сущность или исключение.
  

Answer 2

[Сергей П]

А БД у вас глобально торчит в виде синглтона? Вы даёте id конструктору пользователя, и этот конструктор сам (!) лезет в БД (какую-то), что-то оттуда подтягивает, бросает исключения в зависимости от внешнего состояния...

У какой-то другой сущности, к примеру у приложения, должен быть фабричный метод get_user. Он по идентификатору достаёт необходимые данные (этот код можно в классовый метод юзера поместить) и создаёт на основе этих данных инстанс юзера, например передавая все основные данные в конструктор.

Comments

[topuserman]

Не совсем сам лезет в базу, а делегирует работу ORM, методу getRowById()..

Вы так предлагаете:

Создаем репозиторий UserRepository, у которого есть метод getById,
который будет возвращать null || UserEntity

?

[Сергей П]

topuserman, вполне вариант.

[Кирилл Несмеянов]

topuserman, "getById" возвращает только UserEntity (или ошибку NotFoundException), а "findById" уже null || UserEntity. Семантика-с...