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

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

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

22 ответов

22 просмотра

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

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

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

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

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...
~
13
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
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
program test; {$mode delphi} procedure proc(v: int32); overload; begin end; procedure proc(v: int64); overload; begin end; var x: uint64; begin proc(x); end. Уж не знаю...
notme
6
Карта сайта