我在 Spring 的 BeanUtils 踩到的那些坑，千万不要犯！

缺乏、安全感 2022-10-18 12:43 153阅读 0赞

## 背景 ##

最近项目中在和第三方进行联调一个接口，我们这边发送 http 请求给对方，然后接收对方的回应，代码都是老代码。根据注释，对方的 SDK 中写好的 Request 类有一个无法序列化的 bug，所以这边重新写了一个 Request 类，基本属性都是相同的，但是重点是有一个属性是静态内部类，还有两个是 list 属性，类似于下面这样：

private List<Order> orders;  
    private AddRequest.Ticket ticket;  
    private List<Payment> payments;

AddRequest 就是我们自己重写的请求类，他们 SDK 中的请求类是MixAddRequest，我们组装好请求参数后利用 Spring 的 BeanUtils 的 copyProperties 方法将 AddRequest 中的属性拷贝到 MixAddRequest，然后发送请求。到此为止，照理说一切完美。

![ec74892df5343a8e4253e95ec6e729f6.png][] 我就是这么自信

结果请求失败，纳尼？对方说缺少一个必要的字段，参数校验不通过，一查字段名称，是 Ticket 这个类里面的某个字段，赶紧看代码，心里充满对老代码的自信，想着一定是哪里搞错了，或者是他们那边偷偷动了代码，把字段从可选改为了必选，嘿嘿。

![c6fcf058cfe505e5f41f2962dc692bcf.png][] 厉害厉害

果然在代码里找到了设置的地方，这下应该是他们的问题确信无疑了，再开一把调试，准备宣判他们的死刑。结果发现发给他们的请求就是没有这个字段。。。中间只有一个 Spring 的 copy 属性的方法，当时觉得很诡异。

![31ac643d1c0f5e5334946267dd0be842.png][] 死不瞑目

由于中间只有这么一行代码，玄机肯定在这里面，初步怀疑是两个静态内部类不同导致，所以自己写 Demo，准备搞一把这个 BeanUtils 的 copyProperties 方法，写了两个类和一个 Main，@Data和 @ToString 是 lombok 插件的注解，这里用来自动生成 getter 和 setter 方法以及 toString 方法。

@ToString  
    @Data  
    public class CopyTest1 {  
        public String outerName;  
        public CopyTest1.InnerClass innerClass;  
        public List<CopyTest1.InnerClass> clazz;
    
        @ToString  
        @Data  
        public static class InnerClass {  
            public String InnerName;  
        }  
    }  
    @ToString  
    @Data  
    public class CopyTest2 {  
        public String outerName;  
        public CopyTest2.InnerClass innerClass;  
        public List<CopyTest2.InnerClass> clazz;
    
        @ToString  
        @Data  
        public static class InnerClass {  
            public String InnerName;  
        }  
    }  
    CopyTest1 test1 = new CopyTest1();  
    test1.outerName = "hahaha";  
    CopyTest1.InnerClass innerClass = new CopyTest1.InnerClass();  
    innerClass.InnerName = "hohoho";  
    test1.innerClass = innerClass;
    
    System.out.println(test1.toString());  
    CopyTest2 test2 = new CopyTest2();  
    BeanUtils.copyProperties(test1, test2);
    
    System.out.println(test2.toString());

这里遇到了第一个坑，一开始图省事，属性写为 public，想着省掉了 getter 和 setter 方法，没加 @Data 注解，结果运行完 test2 所有属性都为 null，一个都没 copy 过去，加上 @Data 继续跑，果然，基本属性（String）复制过去了，但是内部类在 test2 中还是 null。那就验证了真的是内部类的问题，有点不敢相信自己的眼睛，毕竟线上跑了这么久的代码。。。

![0f869df3af599a8945011f108bae7c92.png][] 内心奔溃

知道了问题，总要想着怎么解决吧，所以需要单独设置一下内部类，单独 copy，如果内部类的 bean 属性较多或者递归的 bean 属性很多，那可以自己封装一个方法，用于递归拷贝，我这里只有一层，所以直接额外 copy 一次。

CopyTest1 test1 = new CopyTest1();  
    test1.outerName = "hahaha";  
    CopyTest1.InnerClass innerClass = new CopyTest1.InnerClass();  
    innerClass.InnerName = "hohoho";  
    test1.innerClass = innerClass;
    
    System.out.println(test1.toString());  
    CopyTest2 test2 = new CopyTest2();  
    test2.innerClass = new CopyTest2.InnerClass();  
    BeanUtils.copyProperties(test1, test2);  
    BeanUtils.copyProperties(test1.innerClass, test2.innerClass);
    
    System.out.println(test2.toString());

记得内部类的属性也是要有 setter 方法的，不然也会导致 copy 失败，大家还记得我开头说到还有两个 List 属性的吧，为什么要提到这个呢？你猜。

![20a502587e8d458047fccd04d11bb87f.png][] 死的，你打我啊

其实 list 里面的两个类也都是重写的内部类，他们也是不同的，当时他们却顺利 copy 过去了，为什么呢？

因为 java 的泛型只在编译期起作用，在运行期，list属性就是一个存放 Object 的集合，在 copy 后，MixAddRequest 的 orders 属性其实是一个 Order 类的集合，但却不是自己内部类的集合，是 AddRequest 的内部类 Order 的集合，但因为对方是解析 json 的，所以没有发生错误。。。

![1dc2363fe92269798716fc537f119bee.png][] 总能遇到坑队友

## 总结 ##

1.  Spring 的 BeanUtils 的 copyProperties 方法需要对应的属性有 getter 和 setter 方法；
2.  如果存在属性完全相同的内部类，但是不是同一个内部类，即分别属于各自的内部类，则 spring 会认为属性不同，不会 copy；
3.  泛型只在编译期起作用，不能依靠泛型来做运行期的限制；
4.  最后，spring 和 apache 的 copy 属性的方法源和目的参数的位置正好相反，所以导包和调用的时候都要注意一下。

## 最后的最后 ##

附上 spring 的源码，getWriteMethod 是 jdk 的方法，会去取 set 开头的方法，所以没有 setter 方法是不行滴。

private static void copyProperties(Object source, Object target, @Nullable Class<?> editable, @Nullable String... ignoreProperties) throws BeansException {
        Assert.notNull(source, "Source must not be null");
        Assert.notNull(target, "Target must not be null");
        Class<?> actualEditable = target.getClass();
        if (editable != null) {
            if (!editable.isInstance(target)) {
                throw new IllegalArgumentException("Target class [" + target.getClass().getName() + "] not assignable to Editable class [" + editable.getName() + "]");
            }
    
            actualEditable = editable;
        }
    
        PropertyDescriptor[] targetPds = getPropertyDescriptors(actualEditable);
        List<String> ignoreList = ignoreProperties != null ? Arrays.asList(ignoreProperties) : null;
        PropertyDescriptor[] var7 = targetPds;
        int var8 = targetPds.length;
    
        for(int var9 = 0; var9 < var8; ++var9) {
            PropertyDescriptor targetPd = var7[var9];
            Method writeMethod = targetPd.getWriteMethod();
            if (writeMethod != null && (ignoreList == null || !ignoreList.contains(targetPd.getName()))) {
                PropertyDescriptor sourcePd = getPropertyDescriptor(source.getClass(), targetPd.getName());
                if (sourcePd != null) {
                    Method readMethod = sourcePd.getReadMethod();
                    if (readMethod != null && ClassUtils.isAssignable(writeMethod.getParameterTypes()[0], readMethod.getReturnType())) {
                        try {
                            if (!Modifier.isPublic(readMethod.getDeclaringClass().getModifiers())) {
                                readMethod.setAccessible(true);
                            }
    
                            Object value = readMethod.invoke(source);
                            if (!Modifier.isPublic(writeMethod.getDeclaringClass().getModifiers())) {
                                writeMethod.setAccessible(true);
                            }
    
                            writeMethod.invoke(target, value);
                        } catch (Throwable var15) {
                            throw new FatalBeanException("Could not copy property '" + targetPd.getName() + "' from source to target", var15);
                        }
                    }
                }
            }
        }
    
    }

![4af4a85d990731f6e52bb56f4db6542e.png][]

[ec74892df5343a8e4253e95ec6e729f6.png]: /images/20221005/579f4daf42154ee7a0df8d8ccd7351bf.png
[c6fcf058cfe505e5f41f2962dc692bcf.png]: /images/20221005/034f7ed2bdfc4f3eaeb5713f4729e130.png
[31ac643d1c0f5e5334946267dd0be842.png]: /images/20221005/41fb224d93df49b999b73567f247b9a0.png
[0f869df3af599a8945011f108bae7c92.png]: /images/20221005/3283c8aea14a43d2bcc40a2e250413d6.png
[20a502587e8d458047fccd04d11bb87f.png]: /images/20221005/52f00dba61454ae3b5d902daeb6c5605.png
[1dc2363fe92269798716fc537f119bee.png]: /images/20221005/26a7099a6537403c9f32b8821186874f.png
[4af4a85d990731f6e52bb56f4db6542e.png]: /images/20221005/0f29799f7bd3418990cd5c0d68007def.png