一不小心年终奖差点没了！

引言

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

测试环境复现

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

public static void main(String[] args) {
        Set<UserDTO> userSet = new HashSet<>();
        UserDTO userDTO = new UserDTO();
        userDTO.setId(1);
        userDTO.setUserName("java金融");

        UserDTO userDTO1 = new UserDTO();
        userDTO1.setId(2);
        userDTO1.setUserName("java金融");
        userSet.add(userDTO);
        userSet.add(userDTO1);
        System.out.println(userSet.size());
        System.out.println(userDTO1.equals(userDTO));
    }

    @Data
    static class UserDTO extends BaseDTO {
        private String userName;
    }
    @Data
    static class BaseDTO {
        private Integer id;
    }

我们可以输出结果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这个字段，也就是当前类的字段，并没有去比较父类的字段，这就是导致两个对象相等的原因，我们既然找到问题了，那解决问题就比较简单。

一不小心年终奖差点没了！

引言

测试环境复现

解决问题

总结

结束

全站热榜