Можно ли изменять String в объект прямо в методе, который делает вызов к API?

Question

[Рус Лан]

Подскажите, плз. является ли нарушением принципа solid, данный код

HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString());
        return new ObjectMapper().readValue(response.body(), responseType);

Лучше вынести превращение из строки в объект в другой класс, чтобы избавится от побочных эффектов в методе get Запроса?

Comments

[Сергей Горностаев]

При чём тут "сервис-ориентированная архитектура"?

[Рус Лан]

Сергей Горностаев, я сервис, вы ориентир, общаемся по поводу архитектуры

Answer 1

[Dmitry Roo]

Я за реактивный клиент, а там все это делается вполне прозрачно:

webClient.get()
                    .uri("https://your.awesome.uri")
                    .retrieve()
                    .bodyToFlux(YourAwesomeObject.class)
                    .log()
// TODO :: map here                    .map(yourAwesomeObject -> )
//  recomended way  :   .subscribe(result ->   ...);
//  depricated way :       .block();

При этом маппинг ответа в свою модель можно выести хоть в другой класс...

Comments

[Рус Лан]

Dmitry Roo Я пока на java кор пишу. Но скорее всего буду на спринг переносить. Подскажи, плз, Если в респонсе будет 400 строк ключ значение, а в YourAwesomeObject.class 3-4 поля нужных. Можно будет как-то на ходу это сделать. Сейчас использую JSONPropertyIgnore

[Dmitry Roo]

Рус Лан, есть несколько путей и JsonIgnore один из них. На вскидку еще варианты:
- сделать внутри большего объекта, который на вход, метод, который будет возвращать заполненный маленький объект, почти по Бугаенко
- Добавить слой, преобразующий входные данные к доменной модели, это по луковой архитектуре будет
- Использовать что-то типа MapStruct

[stas1212]

Рус Лан, есть класс RestTemplate , у него есть много замечательных методов и один их них getForObeject(
String uri, Class type) , это то что ты ищешь и работает прямо из коробки

Answer 2

[Дмитрий Кинаш]

1) SOLID - это 5! принципов, а не только лишь single responsibility

2) Глядя на всего две строки кода, говорить о нарушении принципов ООП-проектирования могут только полные теоретики и далекие от практики люди.

3) Обмен с внешним ресурсом у вас идет с помощью классов библиотеки java.net.http, а преобразование ответа в объект с помощью библиотеки com.fasterxml.jackson . Вы в приведенных двух строках кода только стыкуете результаты работы двух библиотек. Что именно вас в этом смущает? Хотите сделать над каждой библиотекой обвертку? Но уровнем выше вам все равно придётся делать эту стыковку результатов, просто добавите лишний уровень абстракции.