0


翻车了,lombok这玩意真坑

目录

背景

青柠最近在写自己的项目,刚开始就写不下去了,心态崩了,这啥玩意啊,就是找不到问题在哪?

早前,在项目当中引入了Lombok插件,着实解放了双手,代替了一些重复的简单工作(Getter,Setter,toString等方法的编写)。

但是,在使用的过程当中,也发现了一些问题,开始的时候并没有察觉到是 Lombok 的问题,后来跟踪了对应的其他组件的源码,才发现是Lombok的问题!
在这里插入图片描述

排查

Setter-Getter方法的坑

问题发现
我们在项目当中主要使用Lombok的Setter-Getter方法的注解,也就是组合注解@Data,但是在一次使用Mybatis插入数据的过程当中,出现了一个问题,问题描述如下:

我们有个实体类:

@Data
public class NMetaVerify{
    private NMetaType nMetaType;
    private Long id;
    ....其他属性
}

当我们使用Mybatis插入数据的时候,发现,其他属性都能正常的插入,但是就是nMetaType属性在数据库一直是null。

解决

当我debug项目代码到调用Mybatis的插入SQL对应的方法的时候,我看到NMetaVerify对象的nMetaType属性还是有数据的,但是执行插入之后,数据库的nMetaType字段就是一直是null,原先我以为是我的枚举类型写法不正确,看了下别的同样具有枚举类型的字段,也是正常能插入到数据库当中的,这更让我感觉到疑惑了。

于是,我就跟踪Mybatis的源码,发现Mybatis在获取这个nMetaType属性的时候使用了反射,使用的是getxxxx方法来获取的,但是我发现nMetaType的get方法好像有点和Mybatis需要的getxxxx方法长的好像不一样,问题找到了!

原因

Lombok对于第一个字母小写,第二个字母大写的属性生成的get-set方法和Mybatis以及idea或者说是Java官方认可的get-set方法生成的不一样:

Lombok生成的Get-Set方法

@DatapublicclassNMetaVerify{privateLong id;privateNMetaType nMetaType;privateDate createTime;publicvoidlombokFound(){NMetaVerify nMetaVerify =newNMetaVerify();
        nMetaVerify.setNMetaType(NMetaType.TWO);//注意:nMetaType的set方法为setNMetaType,第一个n字母大写了,
        nMetaVerify.getNMetaType();//getxxxx方法也是大写}}

idea,Mybatis,Java官方默认的行为为:

publicclassNMetaVerify{privateLong id;privateNMetaType nMetaType;privateDate createTime;publicLonggetId(){return id;}publicvoidsetId(Long id){this.id = id;}publicNMetaTypegetnMetaType(){//注意:nMetaType属性的第一个字母小写return nMetaType;}publicvoidsetnMetaType(NMetaType nMetaType){//注意:nMetaType属性的第一个字母小写this.nMetaType = nMetaType;}publicDategetCreateTime(){return createTime;}publicvoidsetCreateTime(Date createTime){this.createTime = createTime;}}

Mybatis(3.4.6版本)解析get-set方法获取属性名字的源码:

packageorg.apache.ibatis.reflection.property;importjava.util.Locale;importorg.apache.ibatis.reflection.ReflectionException;publicfinalclassPropertyNamer{privatePropertyNamer(){// Prevent Instantiation of Static Class}publicstaticStringmethodToProperty(String name){if(name.startsWith("is")){//is开头的一般是bool类型,直接从第二个(索引)开始截取(简单粗暴)
          name = name.substring(2);}elseif(name.startsWith("get")|| name.startsWith("set")){//set-get的就从第三个(索引)开始截取
          name = name.substring(3);}else{thrownewReflectionException("Error parsing property name '"+ name +"'.  Didn't start with 'is', 'get' or 'set'.");}//下面这个判断很重要,可以分成两句话开始解释,解释如下//第一句话:name.length()==1//       对于属性只有一个字母的,例如private int x;//          对应的get-set方法是getX();setX(int x);//第二句话:name.length() > 1 && !Character.isUpperCase(name.charAt(1)))//      属性名字长度大于1,并且第二个(代码中的charAt(1),这个1是数组下标)字母是小写的//      如果第二个char是大写的,那就直接返回nameif(name.length()==1||(name.length()>1&&!Character.isUpperCase(name.charAt(1)))){
          name = name.substring(0,1).toLowerCase(Locale.ENGLISH)+ name.substring(1);//让属性名第一个字母小写,然后加上后面的内容}return name;}publicstaticbooleanisProperty(String name){return name.startsWith("get")|| name.startsWith("set")|| name.startsWith("is");}publicstaticbooleanisGetter(String name){return name.startsWith("get")|| name.startsWith("is");}publicstaticbooleanisSetter(String name){return name.startsWith("set");}}

Mybatis解析get-set方法为属性名字测试

@TestpublicvoidfoundPropertyNamer(){String isName ="isName";String getName ="getName";String getnMetaType ="getnMetaType";String getNMetaType ="getNMetaType";Stream.of(isName,getName,getnMetaType,getNMetaType).forEach(methodName->System.out.println("方法名字是:"+methodName+" 属性名字:"+PropertyNamer.methodToProperty(methodName)));}

输出结果如下:

方法名字是:isName 属性名字:name 
方法名字是:getName 属性名字:name 
方法名字是:getnMetaType 属性名字:nMetaType //这个以及下面的属性第二个字母都是                大写,所以直接返回name
方法名字是:getNMetaType 属性名字:NMetaType

解决方案

修改属性名字,让第二个字母小写,或者说是规定所有的属性的前两个字母必须小写。如果数据库已经设计好,并且前后端接口对接好了,不想修改,那就专门为这种特殊的属性使用idea生成get-set方法复制代码
@Accessor(chain = true)注解的问题
问题发现
在使用 easyexcel 导出的时候,发现以前的实体类导出都很正常,但是现在新加的实体类不正常了,比对了发现,新加的实体类增加了@Accessor(chain = true)注解,我们的目的主要是方便我们链式调用set方法:

newUserDto().setUserName("").setAge(10).........setBirthday(newDate());

原因

easyexcel底层使用的是cglib来做反射工具包的:

com.alibaba.excel.read.listener.ModelBuildEventListener 类的第130行
BeanMap.create(resultModel).putAll(map);

最底层的是cglib的BeanMap的这个方法调用

abstract public Object put(Object bean, Object key, Object value);
但是cglib使用的是Java的rt.jar里面的一个Introspector这个类的方法:

Introspector.java 第520行

if(int.class.equals(argTypes[0])&& name.startsWith(GET_PREFIX)){
   pd =newIndexedPropertyDescriptor(this.beanClass, name.substring(3),null,null, method,null);//下面这行判断,只获取返回值是void类型的setxxxx方法}elseif(void.class.equals(resultType)&& name.startsWith(SET_PREFIX)){// Simple setter
    pd =newPropertyDescriptor(this.beanClass, name.substring(3),null, method);if(throwsException(method,PropertyVetoException.class)){
       pd.setConstrained(true);}}

解决方案

去掉Accessor注解。

要么就等待easyexcel的作者替换掉底层的cglib或者是其他,反正是支持获取返回值不是void的setxxx方法就行复制代码。

总结

踩坑是避免不了的,一时踩坑一时爽,一直踩坑就完犊子了,遇到问题不要慌,记录一下,慢慢找解决方法。 如果对你有帮助,留下三联


本文转载自: https://blog.csdn.net/weixin_40379712/article/details/131062125
版权归原作者 青柠编程 所有, 如有侵权,请联系我们删除。

“翻车了,lombok这玩意真坑”的评论:

还没有评论