Как пометить геттеры, используемые Jackson, чтобы ревьювер меня понимал?

Question

[JacksonRest]

Как вы знаете, Jackson умеет переводить Java Bean в JSON формат при помощи геттеров.

При прохождении код ревью мне пишет синьор замечания, что добавленный мною геттер нигде не используется.

На проекте REST и фронтенд на JS фреймворке

Что мне можно предпринять, чтобы ревьювер понимал, что геттер неявно используется и это только IntelliJ думает, что method unused?

Comments

[Dmitry Roo]

Напишите мини-пример в котором доказываете свою точку зрения.
Проведите беседу с ревьюером.
Угостите его пивом.

[JacksonRest]

Dmitry Roo, а какой-то общий подход с аннотациями есть?

[JacksonRest]

Dmitry Roo, я вообще писал уже, это вторая ситуация. Сказал: "неявно используется Джексоном и нужен на фронте"

Answer 1

[Dmitry Roo]

Общий подход такой - не использовать доменную модель в интеграциях.
Для общения с внешним миром у вас должны быто DTO классы, которые служат только для того чтобы отдать ответ/получить запрос.
Идеальным решением будут для этого record-ы - иммутабельные и имеющие геттеры "из коробки".
В таком случае такие вопросы отпадут сами по себе, как не имеющие смысла.

Comments

[JacksonRest]

Уточняю, использую Lombok, а геттер нужен чтобы вывести новые данные, основанные на двух полях объекта или с обработкой. То есть рекорд вообще не подойдёт, потому что я геттер не смогу у него сделать

Объект и так DTO

[Dmitry Roo]

JacksonRest, в рекорды можно прекрасно добавлять методы:

public static void main(String[] args) throws Exception {
        var example = new Example(1, 2);
        System.out.println(example.getAPlusB());
    }

    record Example(Integer a, Integer b) {
        Integer getAPlusB() {
            return  a + b;
        }
    }

Непонятно только, что это у вас за умная дто-шка со своей логикой.

[JacksonRest]

Dmitry Roo, дто без умной логики, из строки отчество вырезается. Спасибо за ответ

[Dmitry Roo]

JacksonRest, я придумал пример получше:

public static void main(String[] args) throws Exception {
        var example = new NameContainer("Ivanov", "Petr");
        var objectMapper = new ObjectMapper();
        var json = objectMapper.writeValueAsString(example);
        System.out.println(json); // {"firstName":"Ivanov","lastName":"Petr","fullName":"Ivanov Petr"}
    }

    record NameContainer(String firstName, String lastName) {

        @JsonProperty("fullName")
        String getFullName() {
            return firstName + " " + lastName;
        }
    }

тут и геттер есть и понятно зачем он нужен.
Для обычного класса (не рекорда) тоже подойдет