Можно ли упростить реализацию создания заказа на laravel 9?

Question

[lexstile]

Делаю метод создания заказа, вроде все работает, но перед глазами какое-то *овно.
Можно ли как-то упростить реализацию данного метода? (очень не нравятся циклы в цикле)
Но иначе тут, похоже, не сделать?

public function create(OrderCreateRequest $request, Project $project)
    {
        $order = $project->orders()->create($request->all());

        foreach ($request->get('dishes') as $dishe) {
            for ($i = 0; $i < $dishe['quantity']; $i++) {
                $order->dishes()->attach($dishe['id'], Arr::except($dishe, ['id']));
            }
        }
        
        return $this->baseJson(message: ResponseHelper::success(__('orders.messages.created')), status: 201);
    }

# OrderCreateRequest

    public function rules()
    {
        return [
            'table_id' => ['required', 'numeric', 'exists:tables,id',
                Rule::unique('orders')->where(function ($query) {
                    return $query->where('table_id', $this->table_id)->whereIn('status', [Order::NEW, Order::WORK, Order::PAID]);
                }),
            ],
            'dishes' => ['required', 'array', 'min:1'],
            'dishes.*.id' => [
                'required',
                'numeric',
                function ($attribute, $value, $fail) {
                    $exists = Dishe::where('id', $value)->whereHas('menu', function ($query) {
                        return $query->where('project_id', $this->project->id);
                    })->exists();

                    if (!$exists) {
                        $fail(__('validation.exists'));
                    }
                },
            ],
            'dishes.*.price' => ['required', 'numeric', 'between:0,1000000'],
            'dishes.*.quantity' => ['required', 'numeric', 'digits_between:1,99'],
            'dishes.*.comment' => ['nullable', 'string', 'min:1', 'max:255'],
        ];
    }

Comments

[Дмитрий]

А что у вас за дишес и почему вы один и тот же атачите несколько раз

[lexstile]

Дмитрий, dishes - список блюд для добавления к заказу (массив объектов).
1 блюдо = 1 запись.
В заказе может быть несколько одинаковых блюд.

[Дмитрий]

lexstile, ну и вы в связке делаете поле quantity в релейшн указываете withPivot(['quantity']) после чего

$order->dishes()->attach($dishe['id'], ['quantity' => количество]);

здесь есть пример pivot fields и работы с ними

и у вас минус один цикл. Более того можно использовать sync и пачкой херачить. Тогда в вашем коде будет 0 циклов.

[Дмитрий]

lexstile, ну а если совсем упарываться. то создать какой нибудь OrderService куда перенести код создания заказа, в качестве агрумента передавать DTO с каким нибудь методом fromRequest

[lexstile]

Дмитрий, я вас понял, благодарю.
Но quantity в БД нет, т. к. оно всегда равно 1, потому что каждая позиция, даже если она совпадает с предыдущей - это отдельная запись (логика разделения нужна для отказа, статистики и частичной оплаты позиций в заказе)

[Дмитрий]

lexstile, ну я не готов лезть в ваш огород и утверждать категорично, но частичный отказ решается уменьшением quantity. а поле можно и добавить.

[lexstile]

Дмитрий, я понимаю, просто не очень удобно, когда один заказ оплачивают несколько человек

Answer 1

[Vfreelance]

можно без for. attach сразу массив. примеры https://stackoverflow.com/questions/23226802/larav...

Comments

[lexstile]

Без for нельзя, к сожалению, он дублирует количество одной позиции на несколько записей в БД.

Если только подготовить данные для foreach.
Но тогда не совсем будет понятно, что делать с for.

Если только задублировать позиции, где quantity > 1.

[Vfreelance]

lexstile, может, подготовить массив и потом просто в pivot таблицу insert через DB::table() или модель создать для pivot и через нет

Answer 2

[Adamos]

Вообще-то единственное число от dishes - dish, а не dishe.
И добавлять блюда в таблицу совсем не обязательно через $order->attach.
Достаточно создавать их с полем order_id = $order->id, чтобы при выборке они зацепились.
Ну, а как создавать записи пакетом - думаю, вы либо знаете, либо легко нагуглите.

Comments

[lexstile]

Учту, спасибо.

Вообще-то единственное число от dishes - dish, а не dishe.

Не очень понял, если вы про таблицу dishes, то там не нужен order_id, либо я неверно понял.
Т. е. есть заказ, есть блюда, при создании заказа мы их просто связываем в таблице order_dishe.

Достаточно создавать их с полем order_id = $order->id, чтобы при выборке они зацепились.

[Adamos]

lexstile, вот в этой order_disheS и будут поля order_id да dish_id, и создавать эти записи можно напрямую, без всяких многоступенчатых обращений по зависимостям.

[lexstile]

Adamos, поля такие есть, да.
Интересно... Не намекнете, пожалуйста, как хоть способ называется?

order_disheS - ларавел не подхватывает из коробки почему-то, приходилось вручную прописывать таблицу и поля в связях.
А dishe_order подхватил (чтобы вручную не прописывать в каждой relation).

UPD: может, я неверно relation выставил... (belongsToMany в Order)

[Adamos]

lexstile, название order_dishes верно отражает отношение один-ко-многим. Может, у вас в модели оно прописано иначе?
Называется это - работа с БД. Eloquent реализует Active Record, в ней удобно тянуться по связям типа ->dishes()->, когда они уже есть. При создании же записей бывает удобнее писать напрямую, без накладываемых этой логикой загибов.

[Adamos]

lexstile,

belongsToMany в Order

hasMany же.

[lexstile]

Adamos, эм, я не уверен, один ко многим, как я понимаю, можно реализовать с помощью двух таблиц моделей. А тут без третьей уже никак.

[Adamos]

lexstile, да, многие ко многим, конечно. Это я оговорился.
И таблица, соответственно, orders_dishes.