ModelMapper: выберите отображение на основе дочернего класса

9

TL; DR

Я хочу использовать modelMapper таким образом, чтобы я отображал от AbstractParent до AbstractParentDTO, а затем в ModelMapper-Config вызывал конкретные mappers для каждого подкласса, а затем пропускал остальные (abstrac-class) сопоставления.

Как это возможно? Правильно ли это? Есть ли недостаток в дизайне?


Что я имею:

Материнская организация:

@Inheritance(strategy = InheritanceType.JOINED)
@DiscriminatorColumn(name = "type")
public abstract class Parent {
//some more fields
}

Один дочерний объект:

//Basic Lombok Annotations
@DiscriminatorValue("child_a")
public class ChildA extends Parent {
//some more fields
}

Другой дочерний объект:

@DiscriminatorValue("child_b")
public class ChildB extends Parent {
//some more fields   
}

Тогда у меня есть родительский класс DTO:

@JsonTypeInfo(use = JsonTypeInfo.Id.NAME)
@JsonSubTypes({
@JsonSubTypes.Type(value = ChildA.class, name = "child_a"),
@JsonSubTypes.Type(value = ChildB.class, name = "child_b"),
public abstract class ParentDTO {
//some more fields
}

Один ребенок DTO:

public class ClassADTO extends ParentDTO {
//some more fields
}

и другой DTO:

public class ClassBDTO extends ParentDTO {
//some more fields
}

В моем случае я получаю DTO от контроллера и сопоставляю их с Entities при предоставлении их службе. Я должен буду сделать то же самое в 5-6 конечных точках.

Конечные точки выглядят примерно так:

@PreAuthorize(CAN_WRITE)
@PutMapping("/{id}")
public ResponseEntity<ParentDTO> update(
        @PathVariable("id") UUID id,
        @RequestBody @Valid ParentDTO parentDTO) {

    Parent parent = parentService.update(id, parentDTO);

    if (parentDTO instanceof ChildADTO) {
        return ResponseEntity.ok(modelMapper.map(parent, ChildADTO.class));
    } else if (parentDTO instanceof ChildBDTO) {
        return ResponseEntity.ok(modelMapper.map(parent, ChildBDTO.class));
    }
    throw new BadRequestException("The Parent is not Valid");
}

Только у меня есть еще несколько детей, которые делают вещи еще более громоздкими.


Что я хочу:

Вместо того, чтобы проверять кучу раз, в каком экземпляре DTO (или Entity), я просто хочу написать, например:

modelmapper.map(parent, ParentDTO.class)

и выполните "экземпляр...", отметьте ОДИН в моей конфигурации ModelMapper.


Что я пробовал:

У меня уже есть разные преобразователи для каждого возможного направления и отображения, определенные в моей конфигурации ModelMapper (так как они требуют более сложного отображения в любом случае).

Я попытался решить мою проблему, написав еще один конвертер для родительских классов и установив его как PreMonverter ModelMapper:

    //from Entity to DTO
    Converter<Parent, ParentDTO> parentParentDTOConverter = mappingContext -> {
        Parent source = mappingContext.getSource();
        ParentDTO dest = mappingContext.getDestination();

        if (source instanceof CHildA) {
            return modelMapper.map(dest, ChildADTO.class);
        } else if (source instanceof ChildB) {
            return modelMapper.map(dest, ChildBDTO.class);
        } 
        return null;
    };

а также:

modelMapper.createTypeMap(Parent.class, ParentDTO.class)
                .setPreConverter(parentParentDTOConverter);

Но я всегда получаю тот же MappingError:

1) Не удалось создать экземпляр адресата com.myexample.data.dto.ParentDTO. Убедитесь, что com.myexample.data.dto.ParentDTOO имеет конструктор no-private без аргументов.

который я получаю (я думаю), я не могу построить объект абстрактного класса. Но это не то, что я пытаюсь, не так ли? Я предполагаю, что modelMapper все еще делает остальную часть Mapping после окончания моего PreConverter. Я также пытался установить его с.setConverter, но всегда с тем же результатом.


  • Кто-нибудь знает, как отключить пользовательские сопоставления? Я действительно не хочу писать "псевдомапперы", которые действуют как картографы и просто вызывают специфические картографы для каждого сценария.

  • Мой дизайн плохой? Как бы вы его улучшили?

  • Это еще не реализовано в ModelMapper?

Любая помощь и намек оцениваются.

Теги:
spring-boot
spring
modelmapper

3 ответа

4
Лучший ответ

Я бы использовал ObjectMapper вместо ModelMapper.

В родительском классе добавьте возможность получить значение дискриминатора.

//..
public class Parent {

    @Column(name = "type", insertable = false, updatable = false)
    private String type;
    //getters and setters
}

Ваш ParentDTO должен быть сопоставлен с Child (*) DTO

@JsonTypeInfo(
        use = JsonTypeInfo.Id.NAME,
        include = JsonTypeInfo.As.PROPERTY,
        property = "type")
@JsonSubTypes({
        @JsonSubTypes.Type(value = ChildADTO.class, name = "child_a"),
        @JsonSubTypes.Type(value = ChildBDTO.class, name = "child_b")
})
 public abstract class ParentDTO {
   // ..
 }

в сервисе/методе преобразования добавьте объект mapper с игнорированием неизвестного (чтобы игнорировать то, что вы не объявили в своем классе DTO)

    ObjectMapper objectMapper = new ObjectMapper();
    objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);

просто позвоните:

Parent parent = // get from repository
ParentDTO parentDTO = objectMapper.readValue(objectMapper.writeValueAsBytes(parent), ParentDTO.class);

Таким образом, ваш ParentDTO всегда создается с правильным типом.

0

Как насчет

    TypeMap<Parent.class, ParentDTO.class> typeMap = modelMapper.createTypeMap(Parent.class, ParentDTO.class);

    typeMap
     .include(ChildA .class, ClassADTO .class)
    .include(ChildB.class, ClassbDTO.class);

ссылка: http://modelmapper.org/user-manual/type-map-inheritance

  • 1
    Я попробовал это, и, к сожалению, это не делает то, что хочет оригинальный вопрос. Он включает информацию об отображении Parent-> ParentDTO при сопоставлении ChildA-> ClassADTO, но пытается отобразить parent (который является экземпляром некоторого дочернего элемента) в ParentDTO.class не будет пытаться сопоставить его с ClassADTO, а с ParentDTO. Ответ от Акселя Р работает, хотя (за исключением некоторых угловых случаев).
0

Ну, решение, которое я нашел, использует преобразователи. В этом случае modelMapper не пытается создать новый экземпляр абстрактного класса, но напрямую использует конвертер.

Вы можете поместить все конвертеры в одном месте

modelMapper.createTypeMap(ChildA.class, ParentDTO.class)
            .setConverter(mappingContext -> modelMapper.map(mappingContext.getSource(), ClassADTO.class));

modelMapper.createTypeMap(ChildB.class, ParentDTO.class)
            .setConverter(mappingContext -> modelMapper.map(mappingContext.getSource(), ClassBDTO.class));
....

Ещё вопросы

Сообщество Overcoder
Наверх
Меню