就因为加了Lombok的@Accessors(chain = true),bean拷贝工具类不干活了
前言
这次新建了一个工程,因为 Lombok
用得很习惯,但以前的话,一般只用了@Data
,@AllArgsConstructor
,@EqualsAndHashCode
等常规注解;那这个Accessors(chain = true)
注解是干嘛的呢?
用了这个注解后,生成的set方法是这样的:
#加了Accessors(chain = true)
public Devolution setCenterId(Long centerId) {
this.centerId = centerId;
return this;
}
注意,正常情况下,方法应该是下面这样的:
#没加Accessors(chain = true)
public void setCenterId(Long centerId) {
this.centerId = centerId;
}
为什么要用这个方法?主要是方便级联操作。基于这个考虑就加了。
加了后,出现了什么问题?
我们之前有个bean拷贝的工具类,用于在 po 和 vo 间拷贝属性。
import org.springframework.cglib.beans.BeanCopier;
public static void copyProperties(Object source,Object target){
BeanCopier copier = getBeanCopier(source.getClass(), target.getClass());
copier.copy(source, target, null);
}
结果,同事反映说,当target的类型,加了 Accessors(chain = true)
时, 这个工具类不能用了!
跟踪问题
我本来以为改改spring源码就可以了,结果发现org.springframework.cglib.beans.BeanCopier
源码打不开,换了个spring 4的版本,也不行。看到包里面,是待了cglib的,于是本地找了个cglib的包,发现是带source的,于是解压后导入工程,嗯,还不错,可以用!
工程代码在:
https://gitee.com/ckl111/cglib-lombok-test
我这里先说问题原因:
我找到了一个测试用例,大概如下:
public void testSimple() {
BeanCopier copier = BeanCopier.create(MA.class, MA.class, false);
MA bean1 = new MA();
bean1.setIntP(42);
MA bean2 = new MA();
copier.copy(bean1, bean2, null);
assertTrue(bean2.getIntP() == 42);
}
然后自己改造了一下,加了个类:
@Data
@Accessors(chain = true)
class MaWithLombok {
private Long id;
private String name;
private String privateName;
private int intP;
private long longP;
private boolean booleanP;
private char charP;
private byte byteP;
private short shortP;
private float floatP;
private double doubleP;
private String stringP;
public String publicField;
}
这里是测试用例:
public void testSimpleLombok() {
BeanCopier copier = BeanCopier.create(MA.class, MaWithLombok.class, false);
MA bean1 = new MA();
bean1.setIntP(42);
MaWithLombok bean2 = new MaWithLombok();
copier.copy(bean1, bean2, null);
assertTrue(bean2.getIntP() == 42);
}
接下来,就是调试了,在不打断点直接run时,会抛下面异常:
java.lang.NullPointerException
at net.sf.cglib.core.ReflectUtils.getMethodInfo(ReflectUtils.java:424)
at net.sf.cglib.beans.BeanCopier$Generator.generateClass(BeanCopier.java:133)
at net.sf.cglib.core.DefaultGeneratorStrategy.generate(DefaultGeneratorStrategy.java:25)
at net.sf.cglib.core.AbstractClassGenerator.create(AbstractClassGenerator.java:216)
at net.sf.cglib.beans.BeanCopier$Generator.create(BeanCopier.java:90)
at net.sf.cglib.beans.BeanCopier.create(BeanCopier.java:50)
at net.sf.cglib.beans.TestBeanCopier.testSimpleLombok(TestBeanCopier.java:38)
打断点时,发现:
参数member为null,ok,把堆栈退一层(鼠标点到上一层frame)
然后寻找setter的来源:
PropertyDescriptor[] setters = ReflectUtils.getBeanGetters(target);
单步调试,会找到这个地方:
这里是进到了jdk的类,这里
java.beans.Introspector#getBeanInfo()
private BeanInfo getBeanInfo() throws IntrospectionException {
// the evaluation order here is import, as we evaluate the
// event sets and locate PropertyChangeListeners before we
// look for properties.
BeanDescriptor bd = getTargetBeanDescriptor();
MethodDescriptor mds[] = getTargetMethodInfo();
EventSetDescriptor esds[] = getTargetEventInfo();
PropertyDescriptor pds[] = getTargetPropertyInfo();//在这里,获取目标类的属性描述符列表
int defaultEvent = getTargetDefaultEventIndex();
int defaultProperty = getTargetDefaultPropertyIndex();
return new GenericBeanInfo(bd, esds, defaultEvent, pds,
defaultProperty, mds, explicitBeanInfo);
}
我们进入该方法,下图就能告诉你为什么(java/beans/Introspector.java:520):
原因总结
好了,经过上面的问题,大家能发现,因为我们注解的原因,导致setXXX方法的返回值不为void,所以使用
java.beans.Introspector#getTargetPropertyInfo
来获取 PropertyDescriptor
的时候,出现了问题。
问题解决
问题发现了,要怎么解决呢,也简单,我google了一下,哈哈哈。
参考:https://github.com/cglib/cglib/issues/108
使用下面这个工具方法即可:
org.springframework.beans.BeanUtils.copyProperties(source, target);
我的测试工程在,如果大家需要调试 cglib
源码,也可以看看,里面有很多功能的test用例: