13518219792

建站动态

根据您的个性需求进行定制 先人一步 抢占小程序红利时代

一不小心就踩了Lombok的坑?

 [[413622]]

引言

今天中午正在带着耳机遨游在代码的世界里,被运营在群里@了,气冲冲的反问我最近有删生产的用户数据的吗?我肯定客气的回答道没有呀?生产的数据我怎么能随随便便可以删除,这可是公司的红线,再说了我也没有数据库的删除权限啊,不过查询权限还是有的。赶紧登上堡垒机,然后去生产数据库查一下数据,查了一下数据是还在的,吓死了,数据还在问题就不大了,无非就是应用程序出问题了,赶紧打开代码查看下,为什么会少了一条用户数据,看了下代码貌似没啥问题就是比较简单的一个逻辑,直接从DB通过分页查询数据给到前端,然后前端负责展示,没有啥复杂的逻辑。心想肯定是前端的问题,肯定是他少展示了数据,立马把问题也甩给了他,让他帮忙配合一起看看是否是前端的问题,然后自己也仔细看看代码,不到一分钟前端说他展示的数据没有问题,都是后端给到的,没有漏掉展示的。那就是后端的bug了罗。肉眼望去觉得可能出问题的就是分页导致的数据丢了。不过这个分页插件是全公司都在用,应该不至于出问题吧,找不到问题只能让测试帮忙在测试环境试试,看看是否可以复现。

测试环境复现

仔细看了一眼,居然有个去重的方法,去重逻辑也比较简单就是把list通过转为set去下重,看下来应该就是这个去重方法有问题了 大致写了单元测试模仿了下生产的数据,大致逻辑如下:

 
 
 
 
  1. public static void main(String[] args) {
  2.         Set userSet = new HashSet<>();
  3.         UserDTO userDTO = new UserDTO();
  4.         userDTO.setId(1);
  5.         userDTO.setUserName("java金融");
  6.         UserDTO userDTO1 = new UserDTO();
  7.         userDTO1.setId(2);
  8.         userDTO1.setUserName("java金融");
  9.         userSet.add(userDTO);
  10.         userSet.add(userDTO1);
  11.         System.out.println(userSet.size());
  12.         System.out.println(userDTO1.equals(userDTO));
  13.     }
  14.     @Data
  15.     static class UserDTO extends BaseDTO {
  16.         private String userName;
  17.     }
  18.     @Data
  19.     static class BaseDTO {
  20.         private Integer id;
  21.     }

我们可以输出结果set集合的长度是1,user1和user2 是相等的,明明两个user的ID是不一样的,为何会相等,我们知道set可以去重 是因为Set的操作,都是通过操作map来实现的,set的add其实就是调用map的put方法,map的put方法我相信大家应该都去看过其源码,这里就不详细再说了,大概流程就是通过key通过hash算法定位到数组的下标,先判断key的hash是否相等,如果相等再去判断key的value相等,如果都相等就会覆盖原来的值。我们上面这个例子就是对象的hash和value都相等导致,但是我们的两个对象user1和user2 应该是不等的,因为ID不等,那为什么会相等列?我们仔细看下上面的代码,我们使用了lombok里面@Data注解,我们可以看看这个注解帮我们生成了哪些方法

通过上面的对比我们可以看出@Data注解帮我们生成了 注在类上,提供类的get、set、equals、hashCode、canEqual、toString方法,这个注解确实比较方便。上面那个bug 就是因为它生成的equals方法有问题,我们可以把上述代码编译下,然后把class 里面生成的equals方法拷贝出来看看

通过上述生成的代码我们可以看出equals方法只比较了userName这个字段,也就是当前类的字段,并没有去比较父类的字段,这就是导致两个对象相等的原因,我们既然找到问题了,那解决问题就比较简单。

解决问题

图片差异点还是很明显的,加入了@EqualsAndHashCode(callSuper = true) 会去调用父类的equals方法比较,所以这个注解也能够解决这个问题。

总结

本文转载自微信公众号「 java金融」,可以通过以下二维码关注。转载本文请联系 java金融公众号。


新闻名称:一不小心就踩了Lombok的坑?
文章URL:http://cdbrznjsb.com/article/djeopso.html

其他资讯

让你的专属顾问为你服务