Redian新闻
>
翻车了,Lombok 这玩意儿有坑!!!

翻车了,Lombok 这玩意儿有坑!!!

公众号新闻

大家好!

这两天,群里有铁子又讨论到 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 方法

@Data
public class NMetaVerify {
    private Long id;
    private NMetaType nMetaType;
    private Date createTime;
    
    public void lombokFound(){
        NMetaVerify nMetaVerify = new NMetaVerify();
        nMetaVerify.setNMetaType(NMetaType.TWO); //注意:nMetaType的set方法为setNMetaType,第一个n字母大写了,
        nMetaVerify.getNMetaType();                                  //getxxxx方法也是大写
    }
}

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

public class NMetaVerify {
    private Long id;
    private NMetaType nMetaType;
    private Date createTime;

    public Long getId() {
        return id;
    }

    public void setId(Long id) {
        this.id = id;
    }

    public NMetaType getnMetaType() {//注意:nMetaType属性的第一个字母小写
        return nMetaType;
    }

    public void setnMetaType(NMetaType nMetaType) {//注意:nMetaType属性的第一个字母小写
        this.nMetaType = nMetaType;
    }

    public Date getCreateTime() {
        return createTime;
    }

    public void setCreateTime(Date createTime) {
        this.createTime = createTime;
    }
}

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

package org.apache.ibatis.reflection.property;

import java.util.Locale;

import org.apache.ibatis.reflection.ReflectionException;


public final class PropertyNamer {

     private PropertyNamer() {
         // Prevent Instantiation of Static Class
       }

    public static String methodToProperty(String name) {
      if (name.startsWith("is")) {//is开头的一般是bool类型,直接从第二个(索引)开始截取(简单粗暴)
          name = name.substring(2);
      } else if (name.startsWith("get") || name.startsWith("set")) {//set-get的就从第三个(索引)开始截取
          name = name.substring(3);
      } else {
          throw new ReflectionException("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是大写的,那就直接返回name
      if (name.length() == 1 || (name.length() > 1 && !Character.isUpperCase(name.charAt(1)))) {
          name = name.substring(01).toLowerCase(Locale.ENGLISH) + name.substring(1);//让属性名第一个字母小写,然后加上后面的内容
      }

      return name;
    }

    public static boolean isProperty(String name) {
       return name.startsWith("get") || name.startsWith("set") || name.startsWith("is");
    }

    public static boolean isGetter(String name) {
       return name.startsWith("get") || name.startsWith("is");
    }

    public static boolean isSetter(String name) {
       return name.startsWith("set");
    }

}

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

    @Test
    public void foundPropertyNamer() {
        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

解决方案

  1. 修改属性名字,让第二个字母小写,或者说是规定所有的属性的前两个字母必须小写
  2. 如果数据库已经设计好,并且前后端接口对接好了,不想修改,那就专门为这种特殊的属性使用idea生成get-set方法复制代码

@Accessor(chain = true)注解的问题

问题发现

在使用 easyexcel 导出的时候,发现以前的实体类导出都很正常,但是现在新加的实体类不正常了,比对了发现,新加的实体类增加了@Accessor(chain = true)注解,我们的目的主要是方便我们链式调用set方法:

new UserDto()
.setUserName("")
.setAge(10)
........
.setBirthday(new Date());

原因

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 = new IndexedPropertyDescriptor(this.beanClass, name.substring(3), nullnull, method, null);
   //下面这行判断,只获取返回值是void类型的setxxxx方法
 } else if (void.class.equals(resultType) && name.startsWith(SET_PREFIX)) {
    // Simple setter
    pd = new PropertyDescriptor(this.beanClass, name.substring(3), null, method);
    if (throwsException(method, PropertyVetoException.class)) {
       pd.setConstrained(true);
    }
}

解决方案

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

来源:blog.csdn.net/weixin_40379712

/article/details/131062125


微信扫码关注该文公众号作者

戳这里提交新闻线索和高质量文章给我们。
相关阅读
断不了舍不了离不了!烟火气十足的厨房就需要这些个小玩意儿们!吓尿!澳洲海滩惊现神秘“怪物”!外表黑黢黢,像颗肿瘤!网友警告:快跑!这玩意儿很可怕!巨婴之国意大利?42岁妈宝兄弟啃老数年,7旬老母上庭险败诉,网友怒斥:什么臭玩意儿!绝了!靠着这玩意,牛牛暑期一天轻松背2首古诗~5131 血壮山河之武汉会战 信罗战役 4又一个梦上厕所被偷拍?“凶手”竟然是他?澳专家警告:家里这玩意儿正监视你!大家小心!五个月,写了个这玩意美滋滋!自从有了这些不出门就能玩嗨的玩意儿,遛娃终于省心了!华男运这玩意栽了!后备箱放$49万现金,被警察拦下了…深度使用学而思学习机1个暑假,听到大宝说,“这玩意不好糊弄啊”,我彻底放心了服了!这玩意儿,土到中国人翻白眼,却被老外搬去爆红……最后清仓!45%惊天折扣!手慢无!这玩意让澳洲华人爽上天!一下班就想冲回家!现在都抢疯了!大为震惊!都快2024了,澳洲居然还有这种玩意儿!三万平米元宇宙博览会,新玩意儿全在这里了Linus 怒批 AMD fTPM:愚蠢、破玩意儿、建议禁用!几百块的玩意儿搞定娃0岁到高中的中英文学习,怕是来砸场子吧?《花信风之立秋》研究称:这玩意一天一包这流行食物患癌风险大增谁说这玩意非要去北京排队才能买?动动手趁着双十一比平常划算太多!我花500请的家政阿姨,用的也是这几十块的玩意儿!香港华人留学生吐槽:澳洲物价比香港贵多了!就连麦当劳这破玩意儿都要3.8刀!吃不起了!天气冷了,这玩意真暖啊!只要百元,给足爸妈安全感!Legault硬撑也没用! 蒙特利尔市服软: 这玩意我们不打算装回去了…吃一把这个玩意儿,能把舌头吞进肚子里,根本停不下来~突发! 超8万人确诊! 中国海关紧急宣布:这些人必须申报! 悉尼2周暴涨4000例! 新冠新毒王放倒全球!还有华人染这玩意死亡!新款 Fedora Slimbook 14 加入 Fedora Slimbook 阵营 | Linux 中国昙花一现的刘队长蓝环章鱼重出江湖,这玩意有多可怕?天气冷了,用这玩意!只要百元,给足爸妈安全感!准备好屏幕 | 现场:BimBamBoom - Shinzo BakuBaku Ochokochoi @ 下北沢SHELTER《封神》翻车了?吹牛得大奖被威尼斯官方打假了澳洲华人点了下这玩意,$1900万没了!太可怕了!200刀Logitech Combo Touch Keyboard Case with Trackpad for iPad Pr可怕!悉尼华人区公寓突发火灾!现场全是黑烟,爆炸声不断!罪魁祸首竟然又是这玩意儿!老海归给首长会诊
logo
联系我们隐私协议©2024 redian.news
Redian新闻
Redian.news刊载任何文章,不代表同意其说法或描述,仅为提供更多信息,也不构成任何建议。文章信息的合法性及真实性由其作者负责,与Redian.news及其运营公司无关。欢迎投稿,如发现稿件侵权,或作者不愿在本网发表文章,请版权拥有者通知本网处理。