1. 首页
  2. IT资讯

几个 BeanUtils 中的坑,千万别踩

“u003Cimg src=”http:u002Fu002Fp1.pstatp.comu002Flargeu002Fpgc-imageu002FRf8n3jdBqecfy8″ img_width=”1080″ img_height=”345″ alt=”几个 BeanUtils 中的坑,千万别踩” inline=”0″u003Eu003Cpu003Eu003Cstrong toutiao-origin=”strong” class=”highlight-text”u003E背景u003Cu002Fstrongu003Eu003Cu002Fpu003Eu003Cpu003E最近项目中在和第三方进行联调一个接口,我们这边发送u003Ci class=”chrome-extension-mutihighlight chrome-extension-mutihighlight-style-2″u003Ehttpu003Cu002Fiu003E请求给对方,然后接收对方的回应,代码都是老代码。u003Cu002Fpu003Eu003Cpu003E根据注释,对方的SDK中写好的Request类有一个无法序列化的bug,所以这边重新写了一个Request类,基本属性都是相同的,但是重点是有一个属性是u003Cstrongu003E静态内部类u003Cu002Fstrongu003E,还有两个是u003Cstrongu003Elist属性。u003Cu002Fstrongu003Eu003Cu002Fpu003Eu003Cpu003E类似于下面这样:u003Cu002Fpu003Eu003Cpreu003Eu003Cdivu003Eu003Cpreu003Eu003Ccodeu003Eprivate List orders;u003Cu002Fcodeu003Eu003Ccodeu003Eprivate AddRequest.Ticket ticket;u003Cu002Fcodeu003Eu003Ccodeu003Eprivate List payments;u003Cu002Fcodeu003Eu003Cu002Fpreu003Eu003Cu002Fdivu003Eu003Cu002Fpreu003Eu003Cpu003EAddRequest就是我们自己重写的请求类,他们SDK中的请求类是MixAddRequest,我们组装好请求参数后利用Spring的BeanUtils的copyProperties方法将AddRequest中的属性拷贝到MixAddRequest,然后发送请求。u003Cu002Fpu003Eu003Cpu003E到此为止,照理说一切完美!u003Cu002Fpu003Eu003Cpu003E结果请求失败,纳尼?对方说缺少一个必要的字段,参数校验不通过!u003Cu002Fpu003Eu003Cpu003E一查字段名称,是Ticket这个类里面的某个字段,赶紧看代码,心里充满对老代码的自信,想着一定是哪里搞错了,或者是他们那边偷偷动了代码,把字段从可选改为了必选,嘿嘿。u003Cu002Fpu003Eu003Cpu003E果然在代码里找到了设置的地方,这下应该是他们的问题确信无疑了,再开一把调试,准备宣判他们的死刑,结果发现发给他们的请求就是没有这个字段。。。u003Cu002Fpu003Eu003Cpu003E中间只有一个Spring的copy属性的方法,当时觉得很诡异,由于中间只有这么一行代码,玄机肯定在这里面,初步怀疑是两个静态内部类不同导致,所以自己写Demo,准备搞一把这个BeanUtils的copyProperties方法u003Cu002Fpu003Eu003Cpu003E写了两个类和一个Main,@Data和@ToString是lombok插件的注解,这里用来自动生成getter和setter方法以及toString方法。u003Cu002Fpu003Eu003Cpreu003Eu003Cdivu003Eu003Cpreu003Eu003Ccodeu003E@ToStringu003Cu002Fcodeu003Eu003Ccodeu003E@Datau003Cu002Fcodeu003Eu003Ccodeu003Epublic class CopyTest1 {u003Cu002Fcodeu003Eu003Ccodeu003E public String outerName;u003Cu002Fcodeu003Eu003Ccodeu003E public CopyTest1.InnerClass innerClass;u003Cu002Fcodeu003Eu003Ccodeu003E public List clazz;u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E @ToStringu003Cu002Fcodeu003Eu003Ccodeu003E @Datau003Cu002Fcodeu003Eu003Ccodeu003E public static class InnerClass {u003Cu002Fcodeu003Eu003Ccodeu003E public String InnerName;u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Cu002Fpreu003Eu003Cu002Fdivu003Eu003Cu002Fpreu003Eu003Cpreu003Eu003Ccodeu003E@ToStringu003Cu002Fcodeu003Eu003Ccodeu003E@Datau003Cu002Fcodeu003Eu003Ccodeu003Epublic class CopyTest2 {u003Cu002Fcodeu003Eu003Ccodeu003E public String outerName;u003Cu002Fcodeu003Eu003Ccodeu003E public CopyTest2.InnerClass innerClass;u003Cu002Fcodeu003Eu003Ccodeu003E public List clazz;u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E @ToStringu003Cu002Fcodeu003Eu003Ccodeu003E @Datau003Cu002Fcodeu003Eu003Ccodeu003E public static class InnerClass {u003Cu002Fcodeu003Eu003Ccodeu003E public String InnerName;u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Ccodeu003E}u003Cu002Fcodeu003Eu003Cu002Fpreu003Eu003Cpreu003Eu003Ccodeu003ECopyTest1 test1 = new CopyTest1;u003Cu002Fcodeu003Eu003Ccodeu003Etest1.outerName = “hahaha”;u003Cu002Fcodeu003Eu003Ccodeu003ECopyTest1.InnerClass innerClass = new CopyTest1.InnerClass;u003Cu002Fcodeu003Eu003Ccodeu003EinnerClass.InnerName = “hohoho”;u003Cu002Fcodeu003Eu003Ccodeu003Etest1.innerClass = innerClass;u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003ESystem.out.println(test1.toString);u003Cu002Fcodeu003Eu003Ccodeu003ECopyTest2 test2 = new CopyTest2;u003Cu002Fcodeu003Eu003Ccodeu003EBeanUtils.copyProperties(test1, test2);u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003ESystem.out.println(test2.toString);u003Cu002Fcodeu003Eu003Cu002Fpreu003Eu003Cpu003E这里遇到了第一个坑,一开始图省事,属性写为public,想着省掉了getter和setter方法,没加@Data注解,结果运行完test2所有属性都为,一个都没copy过去。u003Ci class=”chrome-extension-mutihighlight chrome-extension-mutihighlight-style-1″u003E加上u003Cu002Fiu003E@Data继续跑,果然,基本属性(String)复制过去了,但是内部类在test2中还是。u003Cu002Fpu003Eu003Cpu003E那就验证了真的是内部类的问题,有点不敢相信自己的眼睛,毕竟线上跑了这么久的代码。。。u003Cu002Fpu003Eu003Cpu003E知道了问题,总要想着怎么解决吧,所以需要单独设置一下内部类,单独copy。u003Cu002Fpu003Eu003Cpu003E如果内部类的bean属性较多或者递归的bean属性很多,那可以自己封装一个方法,用于递归拷贝,我这里只有一层,所以直接额外copy一次。u003Cu002Fpu003Eu003Cpreu003Eu003Ccodeu003ECopyTest1 test1 = new CopyTest1;u003Cu002Fcodeu003Eu003Ccodeu003Etest1.outerName = “hahaha”;u003Cu002Fcodeu003Eu003Ccodeu003ECopyTest1.InnerClass innerClass = new CopyTest1.InnerClass;u003Cu002Fcodeu003Eu003Ccodeu003EinnerClass.InnerName = “hohoho”;u003Cu002Fcodeu003Eu003Ccodeu003Etest1.innerClass = innerClass;u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003ESystem.out.println(test1.toString);u003Cu002Fcodeu003Eu003Ccodeu003ECopyTest2 test2 = new CopyTest2;u003Cu002Fcodeu003Eu003Ccodeu003Etest2.innerClass = new CopyTest2.InnerClass;u003Cu002Fcodeu003Eu003Ccodeu003EBeanUtils.copyProperties(test1, test2);u003Cu002Fcodeu003Eu003Ccodeu003EBeanUtils.copyProperties(test1.innerClass, test2.innerClass);u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003ESystem.out.println(test2.toString);u003Cu002Fcodeu003Eu003Cu002Fpreu003Eu003Cpu003E记得内部类的属性也是要有setter方法的,不然也会导致copy失败,大家还记得我开头说到还有两个List属性的吧,为什么要提到这个呢?你猜u003Cu002Fpu003Eu003Cpu003E其实list里面的两个类也都是重写的内部类,他们也是不同的,当时他们却顺利copy过去了u003Cu002Fpu003Eu003Cpu003E为什么呢?因为java的泛型只在编译期起作用,在运行期,list属性就是一个存放Object的集合u003Cu002Fpu003Eu003Cpu003E在copy后,MixAddRequest的orders属性其实是一个Order类的集合,但却不是自己内部类的集合,是AddRequest的内部类Order的集合,但因为对方是解析json的,所以没有发生错误。。。u003Cu002Fpu003Eu003Cpu003Eu003Cstrong toutiao-origin=”strong” class=”highlight-text”u003E总结u003Cu002Fstrongu003Eu003Cu002Fpu003Eu003Colu003Eu003Cliu003Eu003Cpu003ESpring的BeanUtils的CopyProperties方法需要对应的属性有getter和setter方法;u003Cu002Fpu003Eu003Cu002Fliu003Eu003Cliu003Eu003Cpu003E如果存在属性完全相同的内部类,但是不是同一个内部类,即分别属于各自的内部类,则spring会认为属性不同,不会copy;u003Cu002Fpu003Eu003Cu002Fliu003Eu003Cliu003Eu003Cpu003E泛型只在编译期起作用,不能依靠泛型来做运行期的限制;u003Cu002Fpu003Eu003Cu002Fliu003Eu003Cliu003Eu003Cpu003E最后,spring和apache的copy属性的方法源和目的参数的位置正好相反,所以导包和调用的时候都要注意一下。u003Cu002Fpu003Eu003Cu002Fliu003Eu003Cu002Folu003Eu003Cpu003Eu003Cstrong toutiao-origin=”strong” class=”highlight-text”u003E最后的最后u003Cu002Fstrongu003Eu003Cu002Fpu003Eu003Cpu003E附上spring的源码,getWriteMethod是jdk的方法,会去取set开头的方法,所以没有setter方法是不行滴。u003Cu002Fpu003Eu003Cpreu003Eu003Ccodeu003Eprivate static void copyProperties(Object source, Object target, @able Class<?> editable, @able String… ignoreProperties) throws BeansException {u003Cu002Fcodeu003Eu003Ccodeu003E Assert.not(source, “Source must not be “);u003Cu002Fcodeu003Eu003Ccodeu003E Assert.not(target, “Target must not be “);u003Cu002Fcodeu003Eu003Ccodeu003E Class<?> actualEditable = target.getClass;u003Cu002Fcodeu003Eu003Ccodeu003E if (editable != ) {u003Cu002Fcodeu003Eu003Ccodeu003E if (!editable.isInstance(target)) {u003Cu002Fcodeu003Eu003Ccodeu003E throw new IllegalArgumentException(“Target class [” + target.getClass().getName() + “] not assignable to Editable class [” + editable.getName() + “]”);u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E actualEditable = editable;u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E PropertyDescriptor targetPds = getPropertyDescriptors(actualEditable);u003Cu002Fcodeu003Eu003Ccodeu003E List<String> ignoreList = ignoreProperties != ? Arrays.asList(ignoreProperties) : ;u003Cu002Fcodeu003Eu003Ccodeu003E PropertyDescriptor var7 = targetPds;u003Cu002Fcodeu003Eu003Ccodeu003E int var8 = targetPds.length;u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E for(int var9 = 0; var9 < var8; ++var9) {u003Cu002Fcodeu003Eu003Ccodeu003E PropertyDescriptor targetPd = var7[var9];u003Cu002Fcodeu003Eu003Ccodeu003E Method writeMethod = targetPd.getWriteMethod;u003Cu002Fcodeu003Eu003Ccodeu003E if (writeMethod != && (ignoreList == || !ignoreList.contains(targetPd.getName))) {u003Cu002Fcodeu003Eu003Ccodeu003E PropertyDescriptor sourcePd = getPropertyDescriptor(source.getClass, targetPd.getName);u003Cu002Fcodeu003Eu003Ccodeu003E if (sourcePd != ) {u003Cu002Fcodeu003Eu003Ccodeu003E Method readMethod = sourcePd.getReadMethod;u003Cu002Fcodeu003Eu003Ccodeu003E if (readMethod != && ClassUtils.isAssignable(writeMethod.getParameterTypes[0], readMethod.getReturnType)) {u003Cu002Fcodeu003Eu003Ccodeu003E try {u003Cu002Fcodeu003Eu003Ccodeu003E if (!Modifier.isPublic(readMethod.getDeclaringClass.getModifiers)) {u003Cu002Fcodeu003Eu003Ccodeu003E readMethod.setAccessible(true);u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E Object value = readMethod.invoke(source);u003Cu002Fcodeu003Eu003Ccodeu003E if (!Modifier.isPublic(writeMethod.getDeclaringClass.getModifiers)) {u003Cu002Fcodeu003Eu003Ccodeu003E writeMethod.setAccessible(true);u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E writeMethod.invoke(target, value);u003Cu002Fcodeu003Eu003Ccodeu003E } catch (Throwable var15) {u003Cu002Fcodeu003Eu003Ccodeu003E throw new FatalBeanException(“Could not copy property ‘” + targetPd.getName + “‘ from source to target”, var15);u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Cbru003Eu003Ccodeu003E }u003Cu002Fcodeu003Eu003Cu002Fpreu003Eu003Cp class=”pgc-end-source”u003E作者:绝色天龙u003Cu002Fpu003Eu003Cp class=”pgc-end-source”u003E来源:u003Ci class=”chrome-extension-mutihighlight chrome-extension-mutihighlight-style-3″u003Ewwwu003Cu002Fiu003E.jianshuu003Ci class=”chrome-extension-mutihighlight chrome-extension-mutihighlight-style-3″u003E.comu003Cu002Fiu003Eu002Fpu002F357b55852efcu003Cu002Fpu003E”

原文始发于:几个 BeanUtils 中的坑,千万别踩

主题测试文章,只做测试使用。发布者:℅傍ㄖ免沦陷dε鬼,转转请注明出处:http://www.cxybcw.com/17906.html

联系我们

13687733322

在线咨询:点击这里给我发消息

邮件:1877088071@qq.com

工作时间:周一至周五,9:30-18:30,节假日休息

QR code