Где реализовать такую логику?

Question

[Эдуард Баженов]

Здравствуйте. Добрался в разработку приложения до нестандартной бизнес-логики. Вот такой кусок вставил в контроллер:

public function update(Request $request, $id)
    {
        $part = RollPartsStorage::find($id);
        $part->lenght = $part->lenght - $request->lenght;
        $part->save();

        $roll_plus = new RollPartsStorage;
        $roll_plus->roll_storage_id = $request->roll_storage_id;
        $roll_plus->price = $request->price;
        $roll_plus->provider_id = $request->provider;
        $roll_plus->status_id = 1;
        $roll_plus->type_id = 2;
        $roll_plus->width = $part->width - $request->width;
        $roll_plus->lenght = $request->lenght;
        $roll_plus->save();

        $action = new RollActionsStorage;
        $action -> roll_storage_id = $request->roll_storage_id;
        $action -> type_id = 2;
        $action -> user_id = Auth::id();
        $action -> reason = $request->reason;
        $action -> width = $request->width;
        $action -> lenght = $request->lenght;
        $action->save();
        return redirect(route('roll.show', ['id' => $request->roll_storage_id]))->with(['part_status' => 'Изменения сохранены!', 'part_color' => 'success']);
    }

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

Comments

[Daria Motorina]

Если нет четкого понимания, зачем нужен DI Container и сервис провайдеры - не используйте их, пока не разберетесь.
Здесь стоит доработать читаемость кода (создать осмысленные константы для status_id = 1, type_id = 2 и т.д.), завернуть вычисляемые свойства в осмысленные методы ($roll_plus->width = $part->width - $request->width;), и только потом решать, нужны ли отдельные классы для хранения этой логики. Эти методы больше тянут на изолированную логику для методов create для моделей RollPartsStorage, RollActionsStorage.

[Эдуард Баженов]

Daria Motorina, ,благодарю за ответ! Что значит "изолированная логика для методов"? Первый раз такое вижу :(

[Daria Motorina]

Эдуард Баженов, "изолированная логика" - имеется в виду, чтобы метод и названием, и содержанием отвечал за один процесс) Здесь в методе update видно три явных шага - сначала обновляете length у $part, потом создаете и сохраняете $roll_plus, потом создаете и сохраняете $action. Если можете обычными словами объяснить, почему делаете это в такой последовательности, то можете придумать, как эти шаги выделить в методы с осмысленными названиями и перенести код в них)

[Эдуард Баженов]

Daria Motorina, т.е. создать эти методы в моделях и вызывать где нужно?

[Daria Motorina]

Эдуард Баженов, да, можно в моделях, если логика будет сильно разрастаться - можно будет вынести в сервис

[JhaoDa]

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

Сервис-провайдеры добавляют сервисы в сервис-контейнер, поэтому фраза «Хотел сделать сервис-провайдеры для трех этих моделей» не имеет никакого смысла. Предлагаю подумать и убрать из неё часть символов, чтобы она этот смысл обрела.

Кроме того, вопрос «где» не имеет отношения к ларавел — принципы проектирования приложения общие для разных языков.

[JhaoDa]

Daria Motorina, Эдуард Баженов, не «изолированная логика для методов», а «инкапсуляция».

[Эдуард Баженов]

JhaoDa, благодарю

[Эдуард Баженов]

Daria Motorina, спасибо за помощь!

[JhaoDa]

Daria Motorina,

"изолированная логика" - имеется в виду, чтобы метод и названием, и содержанием отвечал за один процесс

Но не забываем про бизнес-логику, сущности-агрегаты, транзакционность и прочее.

Эдуард Баженов, это должен быть один метод, который сделает всё необходимое, делегировав нужные вызовы кому-то ещё. Псевдокод:

public function writeOff(int $rollPartId, WriteOffAction $action)
{
    $part = RollPartsStorage::find($rollPartId);
    $cut = RollPartsService::cutFrom($oldPart, $action);
    RollActionService::createFor($cut, $action)
}

И не забываем про транзакции.

[NubasLol]

JhaoDa,Транзакции можно вставлять везде? Или есть какой минус в них?

Answer 1

[Alex Wells]

Вынеси логику куда захочешь.

В сервис, например.