背景
某项目某个功能点是接受前端传参,将其存入MongoDB。这个传参的核心数据是一个二维数组List<List<Object>>,可以放字符串、整型,也可以放null。
在测试时发现,前端明明传的是整数,查出来却变成了字符串,比如1234变成了"1234"。经过排查发现,问题出在公司内部使用的一个Bean复制工具类,这个工具类简单封装了DozerMapper,主要功能是将一个Bean复制成一个新的Bean,并且允许这两个Bean的Class不同,从而完成各种类型转换,如:VO <-> Model、Model <-> DO、DO <-> DTO等。
为了快速修复问题从而不影响项目进度,我手写了前端传参和MongoDB的Entity类的转换逻辑,规避了这个问题。这个工具类在公司内部的代码中大量使用,问题的根因是什么?为了搞明白,我写了一个简单的demo,通过debug这部分代码来一探究竟。
关于DozerMapper
DozerMapper有一些高级用法和对应的传参,但是日常中仅仅用到DozerBeanMapperBuilder.buildDefault()来处理。
DozerMapper的官方github,在mvnrepository上可以看到它的最新版本是7.0.0。
公司的工具类用的是6.5.2,也就是6.x的最后一个版本。经验证:
- 7.0.0和6.5.2都有这个bug
- 6.5.2可以运行在JDK8,7.0.0必须运行在JDK11及以上
本文基于JDK8+DozerMapper6.5.2分析。
问题简化和复现
将实际的传参简化如下。该类必须有无参数构造器,否则DozerMapper创建Bean时会报错。
public class ListObjectWrapper { private List<Object> list; public ListObjectWrapper() { } public ListObjectWrapper(List<Object> list) { this.list = list; } public List<Object> getList() { return list; } public void setList(List<Object> list) { this.list = list; } }
对应的测试代码:
public class Test { public static void main(String[] args) { Mapper mapper = DozerBeanMapperBuilder.buildDefault(); List<Object> list = new ArrayList<Object>(); list.add("123"); list.add(456); list.add(null); list.add(new Date()); ListObjectWrapper wrapper1 = new ListObjectWrapper(list); ListObjectWrapper wrapper2 = mapper.map(wrapper1, ListObjectWrapper.class); for (Object value : wrapper2.getList()) { if(value == null) { System.out.println(value); continue; } System.out.println("type:" + value.getClass() + ", value=" + value); } } }
可见,wrapper2的list里的元素全部变成了String:

问题定位
进行debug时,发现在对456调用primitiveConverter.convert()时,此时是知道该元素类型是Integer,调用的返回值却成了字符串:

深入一层,可以看到convert()做了两件事:先确认使用哪个Converter,然后由这个Converter进行实际的转换。这里暗藏了问题:取Converter时,没有用原始数据的实际类型信息,而是取的是Object(这里为什么是Object,接下来会继续深入探讨):

Object类型取不到对应的Converter,就由以下的分支判断,最后还是取不到,就是使用了StringConstructorConverter:

StringConstructorConverter的内部调用了StringConverter,实际上做的只不过是调用了toString(),因此456变成了"456"。
寻根究底
Ojbect类型是从哪里取的?
取Converter时,destFieldType=java.lang.Object,是怎么来的?直觉上,我认为是从List
深入进去,可以看到在这个场景下取的是目标对象的Hint,而非原始值的类型:

目标对象的Hint是怎么生成的?
再次重新debug,回到相对上层的位置,可以看到这里设置的destHintContainer,genericType.getName()就是java.lang.Object:

跟着getGenericType()及后续的propertyDescriptor.genericType()再深入两层就可以看到,是从目标对象的写方法的入参上取到泛型的实际类型也就是java.lang.Ojbect的:

至此,原因已完整呈现。
引申1———给List褪去Bean的外衣
根据上面的分析,List
public class Test1 { public static void main(String[] args) { Mapper mapper = DozerBeanMapperBuilder.buildDefault(); List<Object> list = new ArrayList<Object>(); list.add("123"); list.add(456); list.add(null); list.add(new Date()); List<Object> list2 = mapper.map(list, List.class); for (Object value : list2) { if(value == null) { System.out.println(value); continue; } System.out.println("type:" + value.getClass() + ", value=" + value); } } }

公司的工具类单独写了一个List的复制方法mapList(),对List里的元素逐项调用DozerMapper。不过这个工具类里的方法仍然无法正确复制List
那么一开始为什么不直接用List
引申2——类的继承如何处理?
既然Object是Java里一切类的基类,一个存放了基类对象和继承类对象的容器是否能正确处理呢?根据直觉,应该是不能,实际情况也和直觉一样。读者可以用下面的代码自行验证:
public class Test3 { public static void main(String[] args) { Mapper mapper = DozerBeanMapperBuilder.buildDefault(); List<Parent> list1 = new ArrayList<>(); list1.add(new Parent("张三")); list1.add(new Child("张四", "张三")); List<Parent> list2 = mapper.map(list1, List.class); for (Parent p : list2) { System.out.println("type:" + p.getClass() + ", name=" + p.getName()); } } public static class Parent { private String name; public Parent(String name) { this.name = name; } public Parent() { } public String getName() { return name; } public void setName(String name) { this.name = name; } } public static class Child extends Parent { private String parentName; public Child(String name, String parentName) { super(name); this.parentName = parentName; } public Child() { } public String getParentName() { return parentName; } public void setParentName(String parentName) { this.parentName = parentName; } } }
执行结果提示,list2的两个对象都是Parent类型。
替代方案调研1——BeanUtils
我的前司有同事是用BeanUtils做对象复制的,同名工具类很多,这里的完整类名是org.apache.commons.beanutils.BeanUtils。
按照之前的讨论,继续做测试。BeanUtils有个麻烦的地方在于,你要手动编写它的异常处理代码:
点击查看代码
ListObjectWrapper wrapper3 = new ListObjectWrapper(); try { BeanUtils.copyProperties(wrapper3, wrapper1); } catch (Exception e) { e.printStackTrace(); } // 正确复制 System.out.println(wrapper3); List<Object> list4 = new ArrayList<>(); try { BeanUtils.copyProperties(list4, list); } catch (Exception e) { e.printStackTrace(); } // 复制完list4是空的 System.out.println(list4);
结果是,List
替代方案调研2——MapStruct
在研究DozerMapper的问题和解决方案时,我看到有的文章提到MapStruct是DozerMapper的替代方案,并且速度也更快一些,因此做了一个简单的调研。
依赖处理
MapStruct官网上有一个简单的Demo,直接照搬是运行不起来的,要处理一些依赖。以maven为例,pom.xml要添加以下内容:
- MapStruct依赖
- 注解处理配置中添加mapstruct-processor
- 如果项目使用了Lombok,还需要在注解处理配置中增加lombok的配置,否则可能Build失败
综合后如下:
pom.xml片段
<dependencies> <dependency> <groupId>org.mapstruct</groupId> <artifactId>mapstruct</artifactId> <version>1.5.5.Final</version> </dependency> </dependencies> <build> <plugins> <plugin> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-compiler-plugin</artifactId> <version>3.13.0</version> <configuration> <source>17</source> <target>17</target> <annotationProcessorPaths> <!-- 如果使用 Lombok --> <path> <groupId>org.projectlombok</groupId> <artifactId>lombok</artifactId> <version>1.18.32</version> </path> <!-- MapStruct 处理器 --> <path> <groupId>org.mapstruct</groupId> <artifactId>mapstruct-processor</artifactId> <version>1.5.5.Final</version> </path> </annotationProcessorPaths> </configuration> </plugin> </plugins> </build>
依赖是否配置正确,可以通过以下两步验证:
- 编译是否通过
- 编译完成后,target/gnerated-souces/annotations下是否有编写的接口对应的实现
Mapper和测试代码
由于被测的类比较简单,不需要做转换前后的字段映射,因此对应Mapper也很简单:
点击查看ListObjectWrapperMapper
@Mapper public interface ListObjectWrapperMapper { ListObjectWrapperMapper INSTANCE = Mappers.getMapper(ListObjectWrapperMapper.class); ListObjectWrapper map(ListObjectWrapper wrapper); }
对应地,测试代码如下:
点击查看代码
public class Test2 { public static void main(String[] args) { List<Object> list = new ArrayList<>(); list.add("123"); list.add(456); list.add(null); list.add(new Date()); ListObjectWrapper wrapper1 = new ListObjectWrapper(list); ListObjectWrapper wrapper2 = ListObjectWrapperMapper.INSTANCE.map(wrapper1); for (Object value : wrapper2.getList()) { if(value == null) { System.out.println(value); continue; } System.out.println("type:" + value.getClass() + ", value=" + value); } } }
运行时可见,List

如果转换前后的类,字段不同名,可以用@Mapping来指定。MapStruct的编程接口是比较丰富且强大的,读者可以自行研究。
那么,参考“引申2————类的继承如何处理?”这一节,使用MapStruct是否能正确映射呢?答案是肯定的,新的List里两个元素类型分别是Parent和Child:

小结
- 当目标容器的泛型类型参数是Object类型时,或者当容器中存放了泛型类型参数的子类对象时,DozerMapper的默认用法无法按照元素的实际类型正确地处理
- BeanUtils可以正确复制成员变量包括List
的对象,但是不能直接复制List 本身;此外还要做异常处理 - MapStruct作为DozerMapper的替换时,可以正确处理第一种情况的转换,不过用法显然不如DozerMapper简单:必须编写转换接口、明确入参和返回值的转换方法。但优点是有丰富而强大的相关注解,可以通过注解指定不同名和类型的字段映射。