Как избавится от повторений в коде Laravel 8?

Question

[Павел Кончич]

Подскажите, как правильно избавиться от повторений вот такого вот кода в каждой функции могу контроллера?

public function home(Request $request)
{
    $agent = new Agent();
    $device = $agent->device();
    $viewer = New Viewers();
    $ip = $request->ip();
    $data = Location::get($ip);

    $viewer->request_uri = $request->getRequestUri();
    $viewer->ip_address = $ip;
    $viewer->country = $data->countryName;
    $viewer->city = $data->cityName;
    if($agent->isMobile()){
        $phone = $agent->isPhone();
        $viewer->device = $agent->platform();
    } else if($agent->isTablet()) {
        $viewer->device = $agent->device();
    } else if($agent->isRobot()) {
        $robot = $agent->robot();
        $viewer->device = $robot;
    } else {
        $platform = $agent->platform();
        $viewer->device = $platform;
    }

    $viewer->save();

   }

Вот эта часть будет повторяться в каждой функции - а это уже слишком большое загромождение кода.

Comments

[Константин Толмачев]

Вынесите в приватный метод, а из публичных методов вызывайте. В чем проблема то? Учите основы

[Павел Кончич]

Константин Толмачев, где именно приватный метод создать нужно? в базовом контроллере? или внутри этого контроллера?

[Сергей delphinpro]

Павел Кончич, нигде. Миддлвэр сделайте.

[vism]

fullstack developer; founder of IT solution

и правда

[jazzus]

Павел Кончич, в базовом создать приватный метод и запрашивать из всех методов контроллеров

[Павел Кончич]

jazzus, напишите пример кода, я поставлю как решение

[jazzus]

Павел Кончич, это будет геморрой с ошибками, а не решение. копировать в каждый метод вызов да еще приватного метода. видимо здесь такое предлагают с учетом одного котроллера и то плохо.

[Павел Кончич]

jazzus, мне бы еще кто то объяснил почему на главной странице метод этот tracker() записывает данные в базу 2-жды хотя на все страницах проэкта один раз

[jazzus]

Павел Кончич, в программировании чудес не бывает. если два раза записывается, значит два раза записывается

[larik laravue]

В контроллере не должно быть много кода. Раз говорят в Мидлваре выносить. значит в мидлваре.
В контроллере максимум можно условия прописать что куда, а дальше уже отдельные классы сделают то что нужно.

Answer 1

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

Логирование данных запроса — это работа для middleware.

Comments

[Павел Кончич]

то есть создать web middleware в котором просто вписать эту часть кода, и дальше в karnel добавить его в web?

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

Конкретные шаги зависят от ваших требований. Варианты описаны по ссылке.

[Павел Кончич]

Алексей Уколов, Мне бы еще кто то объяснил почему у меня с главной страницы 2-жды в базу пишутся данные, как будто у меня 2-жды страница перезагружается

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

Вот тут полно желающих подебажить ваш код за вас: https://freelance.habr.com