Такс по пунктам: 1. Никто нигде ничего не обязан. Делайте как вам удобно, но не вводите людей в заблуждение что это SOLID, DDD или что-то еще. 2. "мне нельзя в AR добавить ООП" ну тут я даже не знаю как комментировать... 3. "что так никак нельзя делать в AR или чём то ещё и можно лишь в их православном DM" внезапно PI никоим образом не говорит использовать DM или что-то еще, будь то AR, R(TD)G и иже с ними. Просто ваш репозиторий не несет никакой пользы. Он ничего не инкапсулирует "от слова совсем" (с) Максим. PI поверх AR пилится вполне себе легко и доступно. Собсно сам так жил с Propel лет так 5 назад. Достаточно: a) Выделить интерфейс репозитория и положить его рядом с доменной моделью. А уже в Yii будет лежать имплементация аля ActiveRecordUserRepository. б) Разделить доменную модель и AR модель и гидрировать доменную из AR (и наоборот) в том же репозитории. Для примера ваш UserRepository::findByEmail() будет выглядеть как-то так: ` function findByEmail(string $email): Shop\User { // В AR классах оставляем всю yii муть $userRecord = UserRecord::getByEmail($email); $user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); // Если мы не хотим вызывать явно UserRepository::save(); return $user; } 3. "И готовы бить морду за то что "нет контекстов, значит код говно": дело не в том что их нет, дело в том когда они должны появится. И вроде вы на это уже ответили. Зачем возвращаться? 4. "Ну и не читают диагональщики ..." кто и что читает личное дело каждого. Поверьте ничего нового вы мне не открыли. ЗЫ. В статье годно вполне расписано все.

Ничего он не инкапсулирует, как можно раньше было вызвать напрямую User::save() в обход него так и сейчас можно. Смысл от этого UserFinderAndSaveCaller?

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

function findByEmail(string $email): Shop\User { // В AR классах оставляем всю yii муть $userRecord = UserRecord::getByEmail($email); $user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); // Если мы не хотим вызывать явно UserRepository::save(); return $user; } Я так понимаю здесь создается пользователь, если он не нашелся в бд по email? В этом суть репозитория?

по идее методы с названиями find ничего создавать не должны

так в чем суть репозитория?

репозиторий это абстракция которая должна представить для нас базу данных как коллекцию сущностей

Он и не создает

$user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); А это что происходит?

$user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); А это что происходит?

Это в идеальном мире

какая-то чернь

почему вы не оспариваете этот пример от @zloyuser ? это же чернь в чате про нормальное ООП

$user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); А это что происходит?

@zloyuser это ты типа заворачиваешь данные в модельку и менеджет ее делаешь?

function findByEmail(string $email): Shop\User { // В AR классах оставляем всю yii муть $userRecord = UserRecord::getByEmail($email); $user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); // Если мы не хотим вызывать явно UserRepository::save(); return $user; } Я так понимаю здесь создается пользователь, если он не нашелся в бд по email? В этом суть репозитория?

мы тут создаем "инстанс" пользователя и гидрируем его данными из базы. Попутно мы регистрируем нашу модельку в UoW с пометкой что мы только что ее достали

А если нет пользователя с таким email, как мой код должен выглядеть?

кинуть исключение, вернуть null

В каком месте я здесь это должен сделать? В каком из этих методов?

я как-то накидывал Макарову на тему "а почему никто не делает AR с инвертированной иерархией наследования", ну мол с прокси классами и т.д.

Хороший UoW это не быстро в PHP. Много нюансов с клонами, сериализацией и прочее. Монстр доктрины тому подтверждение.

Ммм, не понял насчет инвертирования

ну если тебе не нужен универсальный UoW то не особо то и сложно. ДОктрина монстр потому что хотят все проблемы решить и причем универсально

function findByEmail(string $email): ?Shop\User { // В AR классах оставляем всю yii муть if (!$userRecord = UserRecord::findByEmail($email)) { return null } $user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); // Если мы не хотим вызывать явно UserRepository::save(); return $user; }

А каким методы еще должен иметь $this->uow, помимо registerClean($user) ? Я не пойму для чего это место

Это нужно чтобы не вызывать вручную User::save(). В конце запроса (команды, бизнес транзакции, etc.) UoW проверяет какие объекты изменились и сохраняет изменения в базу.

UoW проверяет какие объекты изменились и сохраняет изменения в базу Доктрина разве не так работает?

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

Претензия только одна: я пришел в ваш код и вызвал User::save() напрямую и пропали наши доменные ивенты. Ваш "репозиторий" (вообще то это скорее фасад если его задача только скрыть внутренние вызовы) никак не защищает от этого. Вариант с отдельной моделью решает такой кейс.

я пришел в ваш код и вызвал User::save() напрямую и пропали наши доменные ивенты А зачем ты так сделал? Как индус поступил)))

function findByEmail(string $email): ?Shop\User { // В AR классах оставляем всю yii муть if (!$userRecord = UserRecord::findByEmail($email)) { return null } $user = new Shop\User($userRecord->name, $userRecord->email, ...); $this->uow->registerClean($user); // Если мы не хотим вызывать явно UserRepository::save(); return $user; }

Вот я прям предвидел такой наброс. Так и изменения агрегата не было с чего вы взяли что должы быть ивенты?

Я дописал. Юзер email поменял

я пришел в ваш код и вызвал User::save() напрямую и пропали наши доменные ивенты А зачем ты так сделал? Как индус поступил)))

Тогда я не пойму притензий к Дмитрию, что в любом месте можно вызвать $some->save() Претензия только одна: я пришел в ваш код и вызвал User::save() напрямую и пропали наши доменные ивенты