最近在写代码的时候 遇到个奇怪的问题 使用 list.contains(obj) 方法判断,明明是两个内容一样的对象,却返回了false
这里用伪代码还原一下场景:
// 从数据库取的所有数据
List<SysMenuDTO> allList = getDataFromDB();
// 一个id等于1的菜单对象 很明确属于上面的allList
SysMenuDTO menu = getDataFromDB(id = 1);
// res为false
boolean res = allList.contains(menu) ;
contains的源码 其实就是循环调用equals方法
我们都知道 equals方法需要重写 ,否则判断就是 == 比较,在java开发中,一般我们都是会使用lombok插件 注解形式简洁代码的。
// 生成get set等方法
@Data
// 重写equals和hashCode方法
@EqualsAndHashCode(callSuper = true)
检查 SysMenuDTO 实体类 确实也有注解,callSuper = true 表示父类也需要字段相等,equals才会返回true, SysMenuDTO 父类为 Tree
可以看到父类也是重写了equals方法
那么为什么还会出现问题呢?
只能跟踪equals方法的源码了,由于lombok生成的代码 在idea中 非常不方便调试,
我们直接把class文件里面的equals方法代码复制出来
并@EqualsAndHashCode注解去掉,手动调试
我们可以看到 有一行关键代码 如果父类equals方法判断为false 直接就返回false
而我们的Tree方法 是继承于BaseVO类,
public class BaseVO implements Serializable {
public BaseVO() {
super();
}
}
BaseVO类中,为了简洁 并没有加上lomhok相关注解,这也导致BaseVO类并没有去重写equals方法,使用Object默认的 == 去比较,以至于子类Tree调用super.equals时 会返回false,至此真相大白
我们需要手动重写一下equals方法
@Override
public boolean equals(Object o) {
if (o == this) {
return true;
} else if (!(o instanceof BaseVO)) {
return false;
} else {
// baseVo类中目前没其它字段 直接返回true
return true;
}
}
对于super.equals这行代码,本身有没有问题 这里不做评价,
但对我们来说 无疑是一个坑点,毕竟给我们的直觉都是 已经重写过equals 就不会有问题