Как правильно изменить реализацию шаблона репозиторий подгружаемого с помощью интерфейсов для инициализации модели?

Question

[Алексей Марков]

Доброго времени суток.

Был создан магазин на основе https://github.com/Laracommerce/laracom на ларавел.
В процессе работы было замечено что наряду с подтягиванием реализации под интерфейс вызовом типа:
use App\Shop\Products\Repositories\Interfaces\ProductRepositoryInterface;

связывание которых объявлено в RepositoryServiceProvider (app\Providers\RepositoryServiceProvider.php),

$this->app->bind(
            ProductRepositoryInterface::class,
            ProductRepository::class
        );

используются прямые вызовы типа

use App\Shop\Products\Repositories\ProductRepository;

(например здесь app/Shop/Orders/Repositories/OrderRepository.php)

В коде можно найти несколько подобных примеров и чаще всего прямое указание адреса необходимо для вызова конструкции new Repository(ОбъектМодели). Например

public function buildOrderDetails(Collection $items)
    {
        $items->each(function ($item) {
            $productRepo = new ProductRepository(new Product);
            $product = $productRepo->find($item->id);
            if ($item->options->has('product_attribute_id')) {
                $this->associateProduct($product, $item->qty, [
                    'product_attribute_id' => $item->options->product_attribute_id
                ]);
            } else {
                $this->associateProduct($product, $item->qty);
            }
        });
    }

Не нашел однозначного выхода из данной ситуации, прошу совета тех кто сталкивался с примером качественной реализации паттерна Репозиторий на laravel.

Comments

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

Опишите подробнее ваши сомнения. Чем не подходит репозиторий, созданный под сущность?

[Алексей Марков]

Максим Федоров, я так понимаю для ослабления связности все подключения репозиториев делаються через интерфейсы, но также в коде встречаются прямые вызовы для использования репозиториев с заданной моделью.
Также смущают вызовы репозиториев в репозиториях, опять же напрямую без использования интерфейса.

Насколько я понимаю вариант использования интерфейсов для связывания должен исключать подобные вызовы.
Также в процессе поиска примеров в интернете, чаще встречал простую реализацию где для работы с моделью в метод репозитория передается её id .
В вариантах с ларавел репозитории в примерах указывались явно.

Хотелось бы иметь возможность исправить ситуацию с прямым вызовом (иначе зачем биндинг репозиториев с интерфейсами) в конкретном случае с сеттингом модели в репозиторий.

То есть задать модель в примере как здесь app/Shop/Orders/Repositories/OrderRepository.php указав лишь интерфейс

[Евгений Ромашкан]

Алексей Марков,

я так понимаю для ослабления связности все подключения репозиториев делаються через интерфейсы

Если реализация одна, нет никакой разницы, дёрнете ли вы репозиторий напрямую или через интерфейс.

Также смущают вызовы репозиториев в репозиториях, опять же напрямую без использования интерфейса.

Один репозиторий не должен вызывать другой, ему это по определению делать незачем.

[Алексей Марков]

Евгений Ромашкан, "

По поводу количества реализаций, я рассматриваю случай когда их может понадобится менять. Для этого все они объявлены через интерфейсы.

Не могу понять как правильно отказаться от $productRepo = new ProductRepository(new Product);

[Gip]

Ветеран Вьетнамской войны Джон Рэмбо в надежде отыскать своих боевых товарищей путешествует по США. Узнав о скоропостижной кончине своего последнего товарища из элитного военного подразделения «зелёных беретов», он приходит в тихий провинциальный городок. Там его встречает задиристый местный шериф Уилл Тизл, который принимает Рэмбо за бродягу и, отвезя его за пределы города, говорит ему, чтобы тот не возвращался, иначе его ждут неприятности. Но, не проехав и полмили, шериф замечает, что Рэмбо идёт в направлении «его» города

немного не о том о чем вы?

[Алексей Марков]

Если вопрос задан непонятно, напишите конкретно чем. Шутка, даже хорошая, мне не поможет в улучшении формулировки

[Gip]

Алексей Марков, я вам привел пример текста, который - адекватно интерпретируется даже школьником.

То, что написали вы - сложно....очень. НЯНЯЯУНИХПОЕЛ!

[Gip]

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

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

public function buildOrderDetails(Collection $items)
{
$items->each(function ($item) {
$productRepo = new ProductRepository(new Product);
$product = $productRepo->find($item->id);
...

Как все плохо, конечно репозиторий должен быть собран уже в сервис-контенйере и быть готовым... Детали не подскажу — сами ищите, либо фабрики, либо что-то аналогичное — то, что соберет готовый сервис репозитория, чтобы можно было его заинжектить через параметра конструктора/метода.

$items->each(function ($item) {
$productRepo = new ProductRepository(new Product);

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

$items->each(function ($item) {
//...
$product = $productRepo->find($item->id);

Зачем под каждую итерацию дергать репу, когда можно было одним разом достать необходимые объекты из БД
Даже в рамках вашей реализации

[Алексей Марков]

1. Пакет как и код не мой, там много всяких странных решений, пока ищу как исправить нелогичность с вызовом.
2. Первая мысль тоже была изменить вызов на фабричный но к своему удивлению так и не нашел как заменить просто связь на фабричную.

Решений было несколько: фабрика/сеттер модели/отказаться от инстанцирования модели в конструкторе(передача id в методах)/отказаться от слабой связности.

В итоге сделал повторное присваивание модели для всех репозиториев на все возвращающие модель методы репозитория.

/** Псевдокод **/

public function findBySlug($slug): Product
{
    $product = $this->model->where('slug', $slug)->first();
    $this->model = $product;
    
    return $product;
}

[Алексей Марков]

Решений было несколько: фабрика/сеттер модели/отказаться от инстанцирования модели в конструкторе(передача id в методах)/отказаться от слабой связности.

В итоге сделал повторное присваивание модели для всех репозиториев на все возвращающие модель методы репозитория.

/** Псевдокод **/

public function findBySlug($slug): Product
{
$product = $this->model->where('slug', $slug)->first();
$this->model = $product;

return $product;
}

Answer 1

[PQR]

Чтобы конкретно этот метод buildOrderDetails не вызывал внутри new ProductRepository, код нужно переписать так:

class OrderRepository extends BaseRepository implements OrderRepositoryInterface
{
    use OrderTransformable;

    public function __construct(Order $order, ProductRepositoryInterface $productRepo)
    {
        parent::__construct($order);
        $this->model = $order;
        $this->productRepo = $productRepo;
    }

    public function buildOrderDetails(Collection $items)
    {
        $items->each(function ($item) {
            $product = $this->productRepo->find($item->id);
            if ($item->options->has('product_attribute_id')) {
                $this->associateProduct($product, $item->qty, [
                    'product_attribute_id' => $item->options->product_attribute_id
                ]);
            } else {
                $this->associateProduct($product, $item->qty);
            }
        });
    }