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

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

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

22 ответов

23 просмотра

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

И если в 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 в теле, которую то решение не исправит

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

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

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта