Блог

Я использую запросы формы в Laravel для проверки. Я заметил постоянно возникающую закономерность, и я не смог найти для нее решение в SE (или, по крайней мере, поиск в Google мне не помог).

Допустим, мы создаем API и используем ApiResource от Laravel для создания обычных методов CRUD: store , update и delete

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

Возможно ли справиться с этой ситуацией с помощью одного запроса формы в Laravel? Можем ли мы использовать Laravel required_if разумным способом, чтобы избежать дублирования кода?

Редактировать: это не обязательно должно быть решение Laravel. Решение, использующее PHP, тоже подойдет (если оно чистое и соответствует принципам SOLID).

Я сталкивался с этой проблемой много раз, и я понимаю ваше разочарование…

С моей точки зрения и профессионального опыта, лучшим решением всегда было иметь конкретные FormRequest s для каждого случая:

• Один для store со своими правилами
• Другое для update с аналогичными правилами, но не такими, как store
• И последнее для delete (наверняка намного меньше правил, чем у других, и без дублирования)

Я знаю, что вы сказали «нет дублирования кода», но, как сейчас, это невозможно (но у вас не должно быть дублирования кода, как я уже говорил ранее).

Вы сказали: «до тех пор, пока он чистый и соответствует принципам SOLID», помните SOLID, S = Единая ответственность, поэтому, если вы хотите решить эту проблему с помощью одного FormRequest , вы уже нарушаете S .Я не могу представить a FormRequest с 10 или 15 входами, и это зависит от того, является ли это store , update , или delete . Это не будет чистым и наверняка не будет соответствовать принципам SOLID.

Как насчет проверки метода и последующего возврата набора правил на основе этого метода? Например, у меня есть InvoiceFormRequest со следующими правилами. Таким образом, я могу использовать один запрос формы для двух разных методов.

 /**
* Get the validation rules that apply to the request.
*
* @return array
*/
public function rules()
{
    if ($this->isMethod('post')) {
        return [
           'template' => 'required',
           'due_by_date'   => 'required',
           'description'   => 'required',
           'charge_items'  => 'required',
        ];
    }

    if ($this->isMethod('put')) {
        return [
            'due_by_date'   => 'required',
            'description'   => 'required',
            'charge_items'  => 'required',
        ];
    }
}
 

Вот два возможных решения, которые я придумал

1. Используйте метод контроллера для возврата правильных правил проверки:

     public function rules()
    {
        $method = $this->route()->getActionMethod();

        switch($method){
            case 'store':
                return [
                    \ validation rules
                ]
            ...
        }
    }
 

2. Вместо этого используйте $this->getMethod() вместо $this->route()->getActionMethod() и проверяйте с помощью методов HTTP.

Вы также можете сохранить свои правила проверки в массиве и манипулировать им, чтобы уменьшить дублирование кода.

Я думаю, это в значительной степени решает проблему дублирования кода.