Как провалидировать список на laravel в кастомном правиле?

Question

[lexstile]

Нужно проверить 2 кейса:
1. Что блюдо принадлежит тому меню, которое принадлежит текущему проекту
2. Сумма всех блюд должна находиться в диапазоне 1-150000.

Кажется, что что-то я делаю не так.
1. $fail не работает внутри коллекции, я мог бы это исправить переведя на обычный foreach или установить флаг на уровне ф-ции __invoke и менять его, в случае ошибки и проверять уже за пределами коллекции (но кажется, что что-то не то).
2. Стоит ли мешать 2 подобные проверки в одном месте?

<?php

namespace App\Rules;

use App\Models\Dishe;
use App\Models\Order;
use App\Models\Project;
use Illuminate\Contracts\Validation\InvokableRule;
use Illuminate\Support\Arr;

class DishesRule implements InvokableRule
{
    private Project $project;
    private ?Order $order;

    public function __construct (Project $project, Order $order = null) {
        $this->project = $project;
        $this->order = $order;
    }

    public function __invoke($attribute, $value, $fail)
    {
        $dishesRequest = collect($value);

        $dishesRequest->each(function($dishe) use($fail) {
            $disheId = Arr::get($dishe, 'id');

            # проверяем принадлежность блюда и меню к одному проекту
            $exists = $disheId && Dishe::where('id', $disheId)->Active()->whereHas('menu', function ($query) {
                return $query->where('project_id', $this->project->id)->Active();
            })->exists();

            if (!$exists) {
                return $fail(__('validation.exists'));
            }
        });

        # Получаем список блюд по массиву идентификаторов
        $dishes = Dishe::whereIn('id', $dishesRequest->pluck('id'))->get();
        $totalDishes = $dishes->sum('price');
        $total = $this->order ? $totalDishes + $this->order->totalPrice : $totalDishes;

        if ($total < 1 || $total > 150000) {
            return $fail('Сумма заказа должна быть от 1 ₽ до 150 000 ₽');
        }
    }
}

# rules

    public function rules()
    {
        return [
            'table_id' => [
                'required',
                'numeric',
                Rule::unique('orders')->where(function ($query) {
                    # проверка доступности стола (нет ли на нем заказов в указанных статусах)
                    return $query->where('table_id', $this->table_id)->whereIn('status', [Order::NEW, Order::WORK]);
                }),
            ],
            'dishes' => ['required', 'array', 'min:1', new DishesRule($this->table->project)],
            'dishes.*.id' => ['required', 'numeric'],
            'dishes.*.quantity' => ['required', 'numeric', 'digits_between:1,99'],
            'dishes.*.comment' => ['nullable', 'string', 'min:1', 'max:255'],
        ];
    }

Comments

[Дмитрий]

Не существует слов «не работает» в айти- просто стоит забыть. Есть слова - происходит вот это, вот этим может быть ошибкой в логе или на странице, или неправильное поведение. И вот это надо описать

[lexstile]

Дмитрий, понял, учту.

Answer 1

[Дмитрий]

public function __invoke($attribute, $value, $fail)
    {
        $dishesRequest = collect($value);
        $data = Dishe::whereIn('id', $dishesRequest)->Active()->whereHas('menu', function ($query) {
                return $query->where('project_id', $this->project->id)->Active();
        })->selectRaw('count(1) as count, sum(price) as sum');
        if($data->count !=  $dishesRequest->count()){
                return $fail(__('validation.exists'));
        }
        # Получаем список блюд по массиву идентификаторов
        $totalDishes = $data->sum;
        $total = $this->order ? $totalDishes + $this->order->totalPrice : $totalDishes;

        if ($total < 1 || $total > 150000) {
            return $fail('Сумма заказа должна быть от 1 ₽ до 150 000 ₽');
        }
    }

Но вторую проверку я бы наверное вынес. Но не уверен

Comments

[lexstile]

Интересный способ проверить наличие блюд в БД, что количество пришедших должно быть равно количеству полученных.
Спасибо за совет, это явно быстрее, единственное, не очень люблю прямые запросы через *Raw.
Но иначе тут, как я понимаю, никак...

[Дмитрий]

lexstile, 1. ну сделайте через Macroable для QueryBuilder аналог ларавеловских sum() и count() что нибудь типа sumAndCount() или еще как.

2. сделайте через scope и сможете вынести в модель

3. ну или разбейте уже на два запроса count и sum-

Dishe::whereIn('id', $dishesRequest->pluck('id'))->sum('price');

но это как то блин

[lexstile]

Дмитрий, попробовал, похоже, щас все из коробки работает и sum, и count (без использования selectRaw).

$dishes = Dishe::whereIn('id', $dishesRequest->pluck('id'))->Active()->whereHas('menu', function ($query) {
            return $query->where('project_id', $this->project->id)->Active();
        })->get();

        if ($dishesRequest->count() !== $dishes->count()) {
            return $fail(__('validation.exists'));
        }

        $totalDishes = $dishes->sum('price');
        $total = $this->order ? $totalDishes + $this->order->totalPrice : $totalDishes;

        if ($total < 1 || $total > 150000) {
            return $fail('Сумма заказа должна быть от 1 ₽ до 150 000 ₽');
        }

[Дмитрий]

lexstile, ну только в вашем случае база передает вам записи, которые пихаются в массив и потом вы его еще обходите обсчитываете. в моем уже готовый результат. Вы уверены что ваша процедура менее ресурсоемкая, и трата этих ресурсов оправдана тем что вам не нравится selectRaw? Я то вот как то сильно не уверен

[lexstile]

Дмитрий, я думаю, ваш вариант быстрее отработает, но есть момент, что заказов с более, чем 100 позиций либо не будет, либо будут единицы, следовательно, подсчет должен довольно резво отработать.
Грубо говоря, разницу мы заметим только замеряя скорость работы, как я понимаю, визуально, вряд ли.

[lexstile]

Дмитрий, замерил на 1000 раз.
Получились такие результаты, делюсь, вдруг будет интересно:
0.59127497673035 - считаем sum и count динамически (мой последний вариант)
0.54258990287781 - возвращаем в запросе через selectRaw

На 10 000 соответственно:
5.868910074234
5.5112619400024

[Дмитрий]

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