Как избежать дублирования кода при валидации данных в методах store и update?

Question

[ildar-meyker]

Если бы логика валидации была абсолютно идентичная, казалось бы, все просто. Можно вынести в метод rules и FormRequest. Но как быть, если проверка незначительно отличается?

store:

request()->validate([
            'name' => 'required',
            'poster' => 'image|mimes:jpeg,png,jpg,gif|max:2048',
            'name_en' => [
                'required',
                Rule::unique('authors', 'name_en'),
                new AuthorUrl
            ]
        ]);

update:

request()->validate([
            'name' => 'required',
            'poster' => 'image|mimes:jpeg,png,jpg,gif|max:2048',
            'name_en' => [
                'required',
                Rule::unique('authors', 'name_en')->ignore($author->id),
                new AuthorUrl
            ]
        ]);

Answer 1

[JhaoDa]

В методе rules() можно корректировать список правил в зависимости от каких-то условий — того же http verb.

Comments

[ildar-meyker]

Еще возник вопрос в процессе переделки разбухшего контроллера. Куда вынести функцию загрузки файла? То есть загружается и в store, и в update. Где этому место?

[JhaoDa]

ildar-meyker, в сервисе.

[ildar-meyker]

JhaoDa, вы вроде бы и отвечаете и вроде бы и нет. На деле ответы бесполезны, потому что не содержательны.

[JhaoDa]

ildar-meyker, может, мне за тебя код написать?

[ildar-meyker]

JhaoDa, Смешно) А смысл постить полуответы?)

[NubasLol]

В методе rules() можно корректировать список правил в зависимости от каких-то условий — того же http verb.

Насколько это оправдано, вместо того что бы писать под каждый метод класс?
Типо это ведь галимый код, когда один мето/класс делает слишком разные вещи, в зависимости от параметров. Или как?

[ildar-meyker]

NubasLol, получилось не сильно стремно.

public function rules()
{
    return [
        'name' => 'required',
        'name_en' => [
            'required',
            ( $this->isMethod('patch') ) ? 
                Rule::unique('authors', 'name_en')->ignore($this->author->id) :
                Rule::unique('authors', 'name_en'),
            new AuthorUrl
        ],
        'website' => 'nullable|url',
        'vkontakte' => 'nullable|url',
        'instagram' => 'nullable|url',
        'poster' => 'image|mimes:jpeg,png,jpg,gif|max:2048',
    ];
}

[NubasLol]

ildar-meyker, Да, нормально выглдяит. Кроме самого тернарного оператора)

[ildar-meyker]

NubasLol, вкусовщина.