1

Всем привет Интересует такой вопрос:

Есть класс User, в нём реализованы методы add (добавляет юзера в БД), get(получает юзера из бд).

С вызовом проблем нет: создаю экземпляр класса, далее $user->add(параметры); и тд

Возник вопрос: если мне нужно получать всех пользователей из БД, то как это сделать так, чтобы не нарушить ООП?

Если внутри User прописать getAll, то как-то странно, что юзер получает всех юзеров

Возможно, нужно использовать новый класс UserList? Или какие рекомендации в таком случае?

10
  • сделайте статический метод
    – splash58
    Commented 9 нояб. 2020 в 17:02
  • Пока писал понял, что и метод add в классе User не корректен скорее всего. Мы же добавляем юзера в список юзеров. А в классе по логике должны быть только изменение параметров и работа с ними? Commented 9 нояб. 2020 в 17:02
  • У меня весь класс был написан в статике, но прочитал, что такой подход вызывает слишком много проблем с зависимостями и хочу передать всё под чистое ООП Commented 9 нояб. 2020 в 17:04
  • 2
    чтобы не нарушить ооп, нужно разделить класс на два - собственно юзер, который вообще ничего не знает про базу данных, и userMapper - который занимается получением и сохранением объектов класса юзер в БД. Вот в этом втором классе иметь метод взятьВсех очень даже уместно. надо почитать про паттерн Data Mapper Commented 9 нояб. 2020 в 17:05
  • @Ипатьев спасибо за наводку с Data Mapper. Ушел копаться :) Commented 9 нояб. 2020 в 17:11

1 ответ 1

1

Если хотите прям совсем по ООП, то как вариант:

class User{

}

class UserCollection
{
    private $users = [];
    public function __construct(... $users)
    {
        foreach ($users as $user)
        {
            if(is_array($user))
            {
                array_walk($user,[$this,'add']);
            }
            else{
                $this->add($user);
            }
        }
    }

    private function add(User $user)
    {
        $this->users[] = $user;
    }
}

interface UserRepositoryInterface{
    public function get(): User;
    public function getAll(): UserCollection;
}

3
  • 1
    какая-то странная магия с is_array(). С одной стороны строгая типизация, а с другой у нас метод не знает, какие данные обрабатывает Commented 9 нояб. 2020 в 18:54
  • @Ипатьев предложите свой вариант Commented 9 нояб. 2020 в 20:37
  • ну вообще я подумал что наверное да, в отсутствие дженериков тут действительно, других вариантов особо и нет Commented 9 нояб. 2020 в 20:46

Ваш ответ

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

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