Как узнать метод запроса в laravel (если стоит в роуте match)?

Question

[suhuxa1]

Добрый день!
Добавит вот такой роут:

Route::match(['get', 'post'], 'action', ['uses' => 'mainController@action']);

Собственно action может быть любой. Если get запрос - то открыть страницу action. А если post запрос - значит сработала форма на этой странице. Вот как грамотно разделить эти 2 задачи? Правильно ли я вообще сделал, что дал оба метода одной функции контроллера? Раньше я писал 2 разных роута и функции в контроллере для гет и пост, сейчас хочу сократить.

Благодарю!

Answer 1

[Алексей Уколов]

Правильно ли я вообще сделал, что дал оба метода одной функции контроллера?

Если у вас внутри этого метода появилась необходимость различать тип запроса, то нет, что-то у вас не так сделано.

Вот как грамотно разделить эти 2 задачи?

Сгенерируйте при помощи стандартных инструментов контроллер ресурса и посмотрите как это делается идиоматически. Да просто добавьте Route::resource(...) и вызовите php artisan route:list, там вся структура контроллера видна будет.

Раньше я писал 2 разных роута и функции в контроллере для гет и пост, сейчас хочу сократить.

GET - это вывод формы, POST - обработка пользовательсого ввода. Это разная логика, ничего общего в этих методах не бывает обычно, поэтому вместо "сократить" в данном случае правильнее использовать "усложнить себе работу" или "ухудшить код".

Тем не менее, вот ответ непосредственно на ваш вопрос: эта задача решается при помощи объекта запроса:

Route::match(['get', 'post'], 'test', function (\Illuminate\Http\Request $request) {
    return $request->method();
});

Comments

[Александр Шаповал]

чем тогда объясняется возможность объединять типы запросов?

[Алексей Уколов]

Александр Шаповал, честно говоря, не знаю чем руководствовался Тэйлор, когда это делал. Мне ни разу не приходилось этим пользоваться.
Но в схеме "сначала я несколько запросов отправлю на обработку в один метод, а потом внутри него буду определять что это за запрос" явно есть проблема с архитектурой. Если запросы можно объединить на уровне роутера, то в логике никаких различий быть не должно. Если есть различия - нельзя объединять.
То, что хочет автор и то, что вы описали ему в комментарии - это Битрикс какой-то, простите. Так код лучше не писать.

[Александр Шаповал]

Алексей Уколов, удалил сообщение что бы не советовать плохие решения, с вами я полностью согласен