Cglib中BeanCopier#Generator.generateClass报空指针问题

素颜马尾好姑娘i 2021-06-10 20:40 578阅读 0赞

#### 问题描述： ####

###### 错误日志 ######

java.lang.NullPointerException: null
    	at net.sf.cglib.core.ReflectUtils.getMethodInfo(ReflectUtils.java:421)
    	at cn.com.servyou.framework.beans.TypeAwareBeanCopier$TypeAwareGenerator.generateClass(TypeAwareBeanCopier.java:219)
    	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 cn.com.aaa.framework.beans.TypeAwareBeanCopier.instantiate(TypeAwareBeanCopier.java:77)
    	at cn.com.aaa.framework.beans.ConverterUtil.getBeanCopierInstance(ConverterUtil.java:223)
    	at cn.com.aaa.framework.beans.ConverterUtil.copy(ConverterUtil.java:189)
    	at cn.com.aaa.framework.beans.ConverterUtil.convert(ConverterUtil.java:165)
    	at cn.com.aaa.framework.beans.ConverterUtil.convertList(ConverterUtil.java:114)
    	at cn.com.aaa.framework.beans.ConverterUtil.convertList(ConverterUtil.java:84)

这是一个对象类型属性复制的方法。类似于Spring中的BeanUtil.copyProperties（）方法，它是基于Cglib中BeanCopier类实现的。这是类的位置：  
![在这里插入图片描述][2020110620351145.png_pic_center]

#### 问题原因 ####

通过测试得知，因为我在要复制的A类里面增加了get\*（以get打头的方法）的方法,但没有对应的set方法。另外这个场景也比较特殊，要复制的target类B刚好继承了A类。类似BeanUtil.copyProperties（A，B）;

#### 源码 ####

简略版的代码  
详细的请看：[BeanCopier][]

public void generateClass(ClassVisitor v) { 
    			// 获取source和target的getter,setter方法
                PropertyDescriptor[] getters = ReflectUtils.getBeanGetters(source);
                // 问题就出在这！！！！
                PropertyDescriptor[] setters = ReflectUtils.getBeanGetters(target);
    
                Map names = new HashMap();
                for (int i = 0; i < getters.length; i++) { 
                    names.put(getters[i].getName(), getters[i]);
                }
                // 遍历target的setter方法，如果source中有同名的get方法，就获取值，复制target
                for (int i = 0; i < setters.length; i++) { 
                    PropertyDescriptor setter = setters[i];
                    PropertyDescriptor getter = (PropertyDescriptor)names.get(setter.getName());
                    if (getter != null) { 
                        MethodInfo read = ReflectUtils.getMethodInfo(getter.getReadMethod());
                        MethodInfo write = ReflectUtils.getMethodInfo(setter.getWriteMethod());
    					...省略
                    }
                }
            }

上面代码的思路很清晰，获取target的setter方法，根据setter方法去source中找相应的getter方法，如果有，则取值并复制给target。  
但问题就出在PropertyDescriptor\[\] setters = ReflectUtils.getBeanGetters(target);一句。我的原意其实是获取target的所有setter，再从source里面找同名的get方法。  
可是他为什么取得是getter方法？？？？？？  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L3dlaXhpbl8zNzk2ODYxMw_size_16_color_FFFFFF_t_70_pic_center]

#### 去github上看看 ####

![Github上源码][Github]  
在最新的代码处原来是改过来了。这的确是个bug。修改记录在2015年。  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L3dlaXhpbl8zNzk2ODYxMw_size_16_color_FFFFFF_t_70_pic_center 1]

#### 小结： ####

1、这是一个jar包版本太老了，导致的问题，最新的代码已经修复。

[2020110620351145.png_pic_center]: /images/20210518/ecff9b43ff8c4ee6a7e40b82a5dbbf86.png
[BeanCopier]: https://github.com/cglib/cglib/blob/master/cglib/src/main/java/net/sf/cglib/beans/BeanCopier.java
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L3dlaXhpbl8zNzk2ODYxMw_size_16_color_FFFFFF_t_70_pic_center]: /images/20210518/24f3df5685094ff79e3af3cbec1b09b3.png
[Github]: /images/20210518/5645b707cbb647f4a6dfeaee5e15fcb7.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L3dlaXhpbl8zNzk2ODYxMw_size_16_color_FFFFFF_t_70_pic_center 1]: /images/20210518/b3f59f6aea944b91928cc92069258ffb.png