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?
Любая помощь и намек оцениваются.
Я бы использовал 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 всегда создается с правильным типом.
Как насчет
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
Ну, решение, которое я нашел, использует преобразователи. В этом случае 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));
....