170 похожих чатов

Привет, есть ли какое-то решение разбить метод контроллера, когда в

нем много параметров? Один метод из-за этого создает спагетти код с кучей if else, где идет обращение к разным сервис методам в завимости от переденных request param:)

22 ответов

15 просмотров

Ну можно создать класс, который будет содержать в себе все поля и смапить реквест на него

И если в if/else идет какая-то валидация, то ее вынести в отдельный класс

Javist- Автор вопроса
Yana
И если в if/else идет какая-то валидация, то ее вы...

там идет проверка на существование определенных параметров

Javist- Автор вопроса
Yana
А параметры по смыслу как-то связаны?)

ну например такие параметры: offset/limit, dateFrom/dateTo, idList1, idlist2, в некоторых есть валидация по типу @Positive, @Max, есть где-то дефолтные значения и тд

По-моему проблема тут в том, что у вас метод "швейцарский нож")

Javist
ну например такие параметры: offset/limit, dateFro...

Валидатор и @AssertTrue методы валидации на различные параметры

Сделать разные эндпоинты или реализовать стратегию

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

Javist- Автор вопроса
Etki
это значит что у вас один контроллер там, где долж...

ну извините, не я решал сколько будет ендпоинтов :) И в чем плохого в гибком ендпоинте?

Javist- Автор вопроса
Etki
ну решите теперь сейчас сами

сменить контракт ендпоинта которым пользуется куча сервисов?

Javist- Автор вопроса
Javist
?

Выкати новую версию сервиса и обозначь для потребителей сроки переезда

Javist- Автор вопроса
Sergey Demidov
Выкати новую версию сервиса и обозначь для потреби...

так все просто?) Может еще и бизнесу сказать что они лохи и нужно по-другому?😁

Javist
так все просто?) Может еще и бизнесу сказать что о...

Объясни что поддерживать старое говно для них будет куда дороже

Javist- Автор вопроса
Sergey Demidov
Объясни что поддерживать старое говно для них буде...

а как стоит такой ендпоинт разбить? Типо отдельно offset/limit, отдельно idList/offset/limit, отдельно idsList1/idList2/offset/limit, а там еще и limit/offset/dateFrom/dateTo?

Javist
а как стоит такой ендпоинт разбить? Типо отдельно ...

Надо смотреть что конкретный параметр делает. Может там подойдёт и вариант как выше писали в объект с валидацией засунуть

Javist- Автор вопроса
Sergey Demidov
Надо смотреть что конкретный параметр делает. Може...

это про аргумент метода, но ведь еще есть проблема ifelse в теле, которую то решение не исправит

Похожие вопросы

Обсуждают сегодня

Типа вызывать GetParent и проверять на соответствие GetModuleHandle?
The Bird of Hermes
67
Do any of you guys have interesting projects one could join? I'm a Middle Full-Stack developer (JS/TS, React & Node)
Lev Shapiro
40
$res = json_decode($наша строка из респонса); $res1 = array_map(fn($o) => $o->name, $res->breadcrumbs[0]->entities); Как такое будет на Хаскеле?.. В начале весь джейсон, в ко...
Хаскель Моисеевич Гопник
27
В чем сила брат, в NASM или FASM?
Isaac Kleiner
18
Вопрос по диагностике ошибок (я знаю в чем, в данном конкретном примере, я знаю, как исправить, пример модельный, понятно, что в реальности бывает намного запутаннее). module...
ⰄⰎⰋⰐⰐⰑⰛⰤⰧⰧⰩⰄ ⰊⰑⰁⰓⰡⰛⰦⰕⰫ
11
Хтось використовував Vapor на Windows?
Jaroshevskii
15
А чем вам питонисты не угодили?😂
.
79
Есть какой-нибудь для Delphi/FPC T*Compression(Decompression)Stream на базе LZ4/Zstd/любой другой быстрый(и хорошо сжимающий) алгоритм А ещё лучше в pure pascal А ещё лучше од...
notme
52
Тут кста кто-нибудь NeoVim использует?
Simple Sorcerer
13
Оба варианта в целом подходят, но скорее создание конфликтной ситуации (на пустой основе). В прошлый раз он сказал мне про ESP32, я забыл про этот микроконтроллер, ошибся, я п...
КТ315
4
Карта сайта