polishing
This commit is contained in:
parent
45a0ae3fb9
commit
71e0a506b9
|
|
@ -49,7 +49,8 @@ final class CollectionToCollectionConverter implements ConditionalGenericConvert
|
|||
}
|
||||
|
||||
public boolean matches(TypeDescriptor sourceType, TypeDescriptor targetType) {
|
||||
return ConversionUtils.canConvertElements(sourceType.getElementTypeDescriptor(), targetType.getElementTypeDescriptor(), conversionService);
|
||||
return ConversionUtils.canConvertElements(
|
||||
sourceType.getElementTypeDescriptor(), targetType.getElementTypeDescriptor(), this.conversionService);
|
||||
}
|
||||
|
||||
@SuppressWarnings("unchecked")
|
||||
|
|
@ -67,14 +68,15 @@ final class CollectionToCollectionConverter implements ConditionalGenericConvert
|
|||
}
|
||||
else {
|
||||
for (Object sourceElement : sourceCollection) {
|
||||
Object targetElement = this.conversionService.convert(sourceElement, sourceType.elementTypeDescriptor(sourceElement), targetType.getElementTypeDescriptor());
|
||||
Object targetElement = this.conversionService.convert(sourceElement,
|
||||
sourceType.elementTypeDescriptor(sourceElement), targetType.getElementTypeDescriptor());
|
||||
target.add(targetElement);
|
||||
if (sourceElement != targetElement) {
|
||||
copyRequired = true;
|
||||
}
|
||||
}
|
||||
}
|
||||
return copyRequired ? target : source;
|
||||
return (copyRequired ? target : source);
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
|||
|
|
@ -70,17 +70,19 @@ final class MapToMapConverter implements ConditionalGenericConverter {
|
|||
copyRequired = true;
|
||||
}
|
||||
}
|
||||
return copyRequired ? targetMap : sourceMap;
|
||||
return (copyRequired ? targetMap : sourceMap);
|
||||
}
|
||||
|
||||
// internal helpers
|
||||
|
||||
private boolean canConvertKey(TypeDescriptor sourceType, TypeDescriptor targetType) {
|
||||
return ConversionUtils.canConvertElements(sourceType.getMapKeyTypeDescriptor(), targetType.getMapKeyTypeDescriptor(), this.conversionService);
|
||||
return ConversionUtils.canConvertElements(sourceType.getMapKeyTypeDescriptor(),
|
||||
targetType.getMapKeyTypeDescriptor(), this.conversionService);
|
||||
}
|
||||
|
||||
private boolean canConvertValue(TypeDescriptor sourceType, TypeDescriptor targetType) {
|
||||
return ConversionUtils.canConvertElements(sourceType.getMapValueTypeDescriptor(), targetType.getMapValueTypeDescriptor(), this.conversionService);
|
||||
return ConversionUtils.canConvertElements(sourceType.getMapValueTypeDescriptor(),
|
||||
targetType.getMapValueTypeDescriptor(), this.conversionService);
|
||||
}
|
||||
|
||||
private Object convertKey(Object sourceKey, TypeDescriptor sourceType, TypeDescriptor targetType) {
|
||||
|
|
|
|||
Loading…
Reference in New Issue