转载

编码不规范，同事真的会两行泪？

案发现场

我们在Dubbo中定义一个接口，这个接口采用上方说的 欺骗性 的命名方式，这个 getFeiChaoInfo() 中并没有返回值。

好了，然后我们将这个服务暴露，然后启动。按照肥朝之前的观念，命名不规范，无非是理解起来恶心了点，但是跑还是能跑的。结果一启动

之前看过肥朝Dubbo源码解析的同学，对这个服务暴露再熟悉不过了，根据异常栈，我们很快定位到了关键位置。

就算你连Dubbo都没用过也没关系，其实你从 javassist 和 CannotCompileException 这两个关键词就能猜到异常的原因。 javassist 常用于操作字节码， CannotCompileException 根据我小学三年级的英语都知道是无法编译异常。那为什么出现没办法编译通过呢？我们把这段Dubbo拼接出来，准备要用 javassist 进行编译的代码格式化一下就一目了然了

格式化后就很明显可以看出， getFeiChaoInfo() 这个方法没有返回值是编译不过去的。那么这个时候有同学就想说了，Dubbo这段拼接代码进行编译的逻辑有bug啊。鉴于公众号目前有小部分粉丝没用过Dubbo，我们先不讨论Dubbo为什么要这么做，我们反省一下自己，你这种欺骗性的方法名本身就有问题，再者，就算Dubbo把这个代码的容错性做好了又如何，你这种不规范的编码习惯，就算成功，也是 偶然成功！ ，不信？肥朝带你再看一个案发现场。

又一起案发现场

在项目中，我们经常会遇到DTO、BO、DO等转换的问题，很多同学用的是Apache或者Spring的BeanUtils来做copy，我们来一组性能测试

编码不规范，同事真的会两行泪？

另外肥朝给大家总结了一条结论

凡是和反射相关的操作，基本都是低性能的。凡是和字节码相关的操作，基本都是高性能的。

由此可见，在各种POJO间转化，最高性能的肯定是直接操作get/set，但是这样写，肯定不够优雅。从性能报告明显看出较优方案是使用cglib的 BeanCopiers 。 BeanCopiers 怎么使用这个自己搜索一下就知道了，那么我们再来看一起案发现场。为了使用建造者模式，我们有同事这么做

好了。然后跑一个简单的demo

看到有异常一些同学就慌了，就产生了这个东西虽然性能高，但是感觉好像不稳定的样子错觉。其实并不是这东西不稳定，关键还是在于你会不会用。再说了，世界每天都在变，除了肥朝会稳定给大家输出原创之外，还有什么是稳定的呢？为此，肥朝给大家总结了以下几点使用上的常识

1.当源类和目标类的属性名称、类型都相同，拷贝结果棒棒哒。
2.当源对象和目标对象的属性名称相同、类型不同,那么名称相同而类型不同的属性不会被拷贝。另外注意，原始类型（int，short，char）和 他们的包装类型，在这里都被当成了不同类型。因此不会被拷贝。
3.源类或目标类的setter比getter少，拷贝没问题，此时setter多余，但是不会报错。
4.源类和目标类有相同的属性（两者的getter都存在），但是目标类的setter不存在，此时会抛出NullPointerException

关键是我们目标类 FeiChaoBO 的setter方法是存在的啊，那为什么还会出现这个异常？很明显，正常的set方法都是 void 的。然而这个案例中，set方法设置了返回值，存在一定的欺骗性。而且就算要用建造者模式也不是这么用的，再退一万步说，建造者模式一般也是 build 命名而不是改set方法。

你再注意观察一下这两个案发现场，有没有发现一些共同的特点？ javassist 和 cglib ，这两个框架最擅长的就是操作字节码，所以他们对 set 和 get ，都和如白纸版清纯的肥朝一样，非常的敏感！所以建议老司机也不要随便乱动。