Как в реквесте валидировать id сущности?

Question

[Слава]

Как валидировать поля в requst при отправке post запроса - это понятно - Создаётся Request с правилами валидации по всем полям, которые нужно проверить.

А если еще нужно провалидировать ID подели, то как это сделать красиво, желательно внутри того же requst чтоб не писать лишний код в контроллере?

Например, роут такой:

Route::post("article/{id}", [ActicleController::class, 'update')->name('article.update');

и нужно чтоб помимо title, desctiption, user_id мы провалидировали еще id самого поста.

Comments

[Дмитрий]

используйте model binding и получите 404 если модели нет

[Владимир Кохан]

На предмет чего ты собираешься валидировать id? Наличия? Правильности написания? Или еще чего-нить неизвестного?

[iljaGolubev]

есть routing#parameters-regular-expression-constraints
Но это не валидация. Так что, действительно, уточните зачем и на предмет чего вы хотите валидировать {id}?

И, кстати, зачем символ $ в роуте Route::post("article/{$id}"

Answer 1

[Сергей delphinpro]

При использовании биндинга вы получите 404 при некорректном ID
Кроме того, вам не нужно получать модель в контроллере, она сразу туда попадет.

Route::post("article/{article}", [ArticleController::class, 'update')
    ->name('article.update');

class ArticleController extends Controller
{
    public function update(Article $article, ArticleUpdateRequest $request)
    {
        $article
            ->fill($request->validated())
            ->save();

        return redirect(route('article.edit', $article));
    }
}

Comments

[Слава]

попробовал так сделать как вы сказали. но выяснилось, что 404 выдается только при id несуществующей записи. А если в качестве id передать строку символов, то будет ошибка.

[Дмитрий]

Слава, упал с каким эксепшеном? роут нот фаунд?

[Слава]

Дмитрий, нет. там уже ошибка базы данных. типа select * from article where id = ?

из-за того что id типа int а передали строку. но решил это через RouteServicePrivider.

Route::bind('article', function (string $value) {
            if (! is_numeric($value)) {
                throw new NotFoundHttpException();
            }
            return Article::query()->findOrFail($value);
        });

[Сергей delphinpro]

Слава, не может быть. Тот же самый 404 должен быть.

У меня по крайней мере так

[Слава]

Сергей delphinpro, а у меня вот так:

Answer 2

[JhaoDa]

Во-первых, id поста не относится к данным для валидации.

Во-вторых, если таки хочется велосипедить на квадратных колёсах и валидировать через реквест — ответ есть в документации.

В-третьих, чем не устраивает в контроллере что-то типа Article::query()->findOrFail($id), что более правильно?

Comments

[Слава]

тестировщики передали в качестве id строчку с символами и сайт упал. а проверять через findorfail не хо телось. думал в самом валидаторе как то правильнее будет входные данные проверять.

[JhaoDa]

Слава, зря думал.

[Слава]

но вообще в контроллере не нужно обращаться к данным. к данным обращаются через репозиторий или через Query. но не в контроллере.

[JhaoDa]

Слава, я привёл простой пример, обращайся как угодно.