Проектирую апи для сервиса пользователь загружает картинку и

Question

Проектирую апи для сервиса пользователь загружает картинку и

есть 3 функции

изменить яркость, изменить контрастрость, инвертировать цвета

сейчас я для всех 3 функций обработки изображения сделал единый url (пост запрос приходит на /image/ и содержит в себе 3 поля

— id файла,

—тип операции (яркость/контрастность/инверсия),

— и json строку со всеми параметрами которые специфичны для каждой операции)

потом это сохраняется в бд, приходит воркер и каскадом ифов решает что делать

как это правильно реализовать?

#oop #programming #russian #software #web-design

0

11.07.2022

11 ответов

18 просмотров

Michel Angelo

Андрей Рудин
OpenClose принципом надо пользоваться вместо if'ов...

это точно мне? ))

0

12.07.2022

Sergey P

ну это может быть на уровне сериализации, или еще где. тут вопрос какие правила ты хочешь изолировать. но особенност контроллера в том что он задает порядок действий но не содержит в себе правил - все проверки или трансформации делигируются другим участникам процесса. можно на это смотреть с другой стороны - тестирование. Если у тебя контроллер не имеет логики - то его можно покрыть одним позитивным приемочным тестом + статическим анализом. Мол штуки типа... if (something === null) { doSomethingWith(something); } такую логику проверит стат анализ а потому нам пофигу. так вот, мало логики - можно пушить больше зависимостей. Много зависимостей - надо уменьшать количество тест кейсов (за счет того что ты убираешь логику в зависимости).

0

12.07.2022

Michel Angelo

не вижу профита, вот где для меня профит actionAdmin(){ return action1(); } actionUser(){ return action2(); } и все это в разных классах контроллерах

0

12.07.2022

Sergey P

а оно тебе надо?)

0

12.07.2022

Павел Г.

Michel Angelo
не вижу профита, вот где для меня профит actionAd...

У меня есть контроллер, и переиспользование всего того, что внутри контроллера, но просто с другим адресом - очень маловероятный кейс

0

12.07.2022

Alex

Sergey P
а оно тебе надо?)

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

0

12.07.2022

Павел Г.

Sergey P
ну это может быть на уровне сериализации, или еще ...

Ну вот обертка для простой логики сервиса, а есть ли смысл вообще такое тестами покрывать? Это тривиальный кейс. Опять таки, это будет с моками и прочим, что делает тест несколько непростым.

0

12.07.2022

razumovsky r

Alex
ну как минимум я могу использовать этот сервис в д...

тебе понадобится ягни

0

12.07.2022

Sergey P

razumovsky r
тебе понадобится ягни

понадобится "You Aint gonna need it" звучит интересно

0

12.07.2022

Michel Angelo

Павел Г.
У меня есть контроллер, и переиспользование всего ...

а почему вот для тебя, скажем именование переменных и классов по ddd имеет смысл, а именование тех же ендпоинтов не имееет смысла чтобы они также были читаемы и понятны?

0

12.07.2022

Андрей Рудин · Accepted Answer

Андрей Рудин

OpenClose принципом надо пользоваться вместо if'ов.

0

12.07.2022

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

Проектирую апи для сервиса пользователь загружает картинку и

11 ответов

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