Как правильно оптимизируется код в контроллерах Laravel?

Question

[usdbuy]

Всем привет, подскажите пожалуйста как правильно оптимизируется код в контролерах Laravel

Нагромоздил работающий код в контролере и теперь хотелось бы понять как правильно такие вещи распределяются

public function store(Request $request)
    {
        $id = $request['userId'];
        $start_date = $request['startDate'];
        $end_date = $request['endDate'];

        $user = User::find($id);

        // Проверка на корректность дат
        $check_date_correct = $user->checkDateForm($start_date, $end_date);

        if ($check_date_correct === false) {
            return view('timetable', ['data' => 'Неверно указана дата']);
        }

        // Период дат с формы получение в массив
        $period_date_form = $user->getPeriodDateForm($start_date, $end_date);

        // Получение в массив дат отпуска сотрудника
        $holidays_employee = $user->getHolidays($id);
        $date_holidays_employee = $user->getDateHolidayArr($holidays_employee);

        // Получение колличества дней запрошенного периода
        $end_date_time = strtotime($end_date);
        $start_date_time = strtotime($start_date);
        $days_period = $end_date_time - $start_date_time;
        $quantity_days = $days_period / 86400;

        // Получение праздников компании
        $parties_company = $user->getParties();
       /* dd($user->getDatePartiesArr($parties_company)); */

        // Получение по API праздников
        $arr = [];

        for($i=0; $i <= $quantity_days; $i++)
        {
            $add_day_unix = strtotime("+$i day", strtotime($start_date));
            $add_day = date("Y-m-d", $add_day_unix);
            $api_date = json_decode(file_get_contents('https://api.sm.su/v1/calendar/business/?day='.$add_day),1);

            if($api_date['work'] == 0) {
                $arr[] = $add_day;
            }
        }
}

Comments

[JhaoDa]

Для начала стоит научиться форматировать код по общепринятым стандартам — PSR-2/12.

[Дмитрий]

Ну например цикл в конце можно в метод модели положить
Да и прочие вычисления можно в модели делать

[vism]

Дмитрий, в сервис, в сер-вис.

[Константин Б.]

JhaoDa, ещё у него прикольно что переменные snake, а ключи массива camel)

[Alex Wells]

Константин Б., потому что фронтэнд обычно просит в camel)

[usdbuy]

Константин Б., благодарю за замечание! Ознакомился по данным понятиям поправил код по camel

Answer 1

[Alex Wells]

Получение праздников по АПИ? Это можно делать без АПИ, на крайняк дергать это АПИ регулярно и сохранять результаты в кэш - это по оптимизации.

Остальное по code style и качеству кода - PSR стандарты, laravel validation, carbon, логику за пределы контроллера.

Answer 2

[antimind]

Вынести логику в сервис, а в контроллере уже оперировать результатами выполнения этих операций. Ни в коем случае не в модель, как советовали выше в комментах. Слой сервисов как раз и существует для того, что бы всю логику писать там. А контроллер совсем не об этом.

Comments

[Евгений Ромашкан]

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

Слой сервисов существует для application-specific business-rules, а критичная для бизнеса логика и работа с состоянием моделей должна быть в сущностях.
А помещение всей логики в сервисы называется "Анемичная Модель" и является антипаттерном(как и AR, но его можно выпилить или использовать как DAO/row data gateway).

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

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

[antimind]

Евгений Ромашкан, https://m.habr.com/ru/post/346016/ вот например, почему это не антипаттерн.
И я полностью согласен, что

анемичная модель предметной области ближе соответствует принципам SOLID, чем богатая модель

[Евгений Ромашкан]

antimind,
Я знаю про эту статью, и её автор просто не умеет готовить Рич модель и декомпозировать систему, и не пытался этого делать/научиться)
А аргументы про SOLID - чушь. SOLID не идеал и не самоцель.
Лучше уж смотреть на все виды Coupling/Cohesion в совокупности, они значительно более информативны, и SOLID тоже инструмент для улучшения вышеупомянутых принципов.

P.s.

https://m.habr.com/ru/post/346016/ вот например, почему это не антипаттерн.

И в целом, один лишь факт что какой-то неизвестный Вася написал на Хабре что "X" не антипаттерн ни о чем не говорит и не должно говорить.

[antimind]

Евгений Ромашкан, это ваше видение ситуации, и я не собираюсь это оспаривать. Но, анемичную модель признают не только "Васи с хабра". Это был всего лишь первый попавшийся пример.
ИМХО, я никогда в модели не вношу никакой логики. Модель - это описание структуры данных с геттерами, сеттерами и конструктором. Больше ничего туда пихать не надо. Ну это моё ИМХО.

[Евгений Ромашкан]

antimind,

Модель - это описание структуры данных с геттерами, сеттерами и конструктором

Структуры данных отдельно, логика отедельно - процедурный код прям как пол века назад.
Связность выше, зависимостей больше, юнит тестов в такихах обычно нет, стандартная ситуация.

[antimind]

Евгений Ромашкан, ну да, и весь код пишу в index.php. На этом я прекращу наш бесполезный диалог. Всего доброго.