小心!Objects.equals有坑

开发 前端
我们在使用Objects.equals方法,判断两个值是否相等时,一定要保证两个入参的类型要一致。否则即使两个值相同,但其结果仍然会返回false,这是一个大坑。

前言

最近review别人代码的时候,发现有个同事,在某个业务场景下,使用Objects.equals方法判断两个值相等时,返回了跟预期不一致的结果,引起了我的兴趣。

原本以为判断结果会返回true的,但实际上返回了false。

记得很早之前,我使用Objects.equals方法也踩过类似的坑,所以有必要把这个问题记录下来,分享给大家。

到底怎么回事呢?

1. 案发现场

假设现在有这样一个需求:判断当前登录的用户,如果是我们指定的系统管理员,则发送一封邮件。系统管理员没有特殊的字段标识,他的用户id=888,在开发、测试、生产环境中该值都是一样的。

这个需求真的太容易实现了:

UserInfo userInfo = CurrentUser.getUserInfo();

if(Objects.isNull(userInfo)) {
log.info("请先登录");
return;
}

if(Objects.equals(userInfo.getId(),888)) {
sendEmail(userInfo):
}

从当前登录用户的上下文中获取用户信息,判断一下,如果用户信息为空,则直接返回。

如果获取到的用户信息不为空,接下来判断用户id是否等于888。

  • 如果等于888,则发送邮件。
  • 如果不等于888,则啥事也不干。

当我们用id=888的系统管理员账号登录之后,做了相关操作,满怀期待的准备收邮件的时候,却发现收了个寂寞。

后来,发现UserInfo类是这样定义的:

@Data
public class UserInfo {
private Long id;
private String name;
private Integer age;
private String address;
}

此时,有些小伙伴可能会说:没看出什么问题呀。

但我要说的是这个代码确实有问题。

什么问题呢?

答:UserInfo类的成员变量id=888是Long类型的,而Objects.equals方法右边的888是int类型的,两者不一致,导致返回的结果是false。

这算哪门子原因?

答:各位看官,别急,后面会细讲的。

2. 判断相等的方法

让我们一起回顾一下,以前判断两个值是否相等的方法有哪些。

2.1 使用==号

之前判断两个值是否相等,最快的方法是使用==号。

int a = 1;
int b = 1;
byte c = 1;
Integer d1 = new Integer(1);
Integer d2 = new Integer(1);
System.out.println(a == b);
//结果:true
System.out.println(a == c);
//结果:true
System.out.println(a == d1);
//结果:true
System.out.println(d2 == a);
//结果:true
System.out.println(d1 == d2);
//结果:false

不知道大家有没有发现,java中的基本类型,包含:int、long、short、byte、char、boolean、float、double这8种,可以使用==号判断值是否相等。如果出现了基本类型的包装类,比如:Integer,用一个基本类型和一个包装类,使用==号也能正确判断,返回true。

Integer和int比较时,会自动拆箱,这是比较值是否相等。

但如果有两个包装类,比如:d1和d2,使用==号判断的结果可能是false。

两个Integer比较时,比较的是它们指向的引用(即内存地址)是否相等。

还有一个有意思的现象:

Integer d3 = 1;
Integer d4 = 1;
Integer d5 = 128;
Integer d6 = 128;
System.out.println(d3 == d4);
//结果:true
System.out.println(d5 == d6);
//结果:false

都是给Integer类型的参数,直接赋值后进行比较。d3和d4判断的结果相等,但d5和d6判断的结果却不相等。

小伙伴们,下巴惊掉了没?

答:因为Integer有一个常量池,-128~127直接的Integer数据直接缓存进入常量池。所以1在常量池,而128不在。

然而,new的Integer对象不适用常量池。从之前d1和d2例子的比较结果,就能看出这一点。

接下来,看看字符串的判断:

String e = "abc";
String f = "abc";
String g = new String("abc");
String h = new String("abc");
System.out.println(e == f);
//结果:true
System.out.println(e == g);
//结果:false
System.out.println(g == h);
//结果:false

普通的字符串变量,使用==号判断,也能返回正确的结果。

但如果一个普通的字符串变量,跟new出来的字符串对象使用==号判断时,返回false。这一点,跟之前说过的用一个基本类型和一个包装类,使用==号判断的结果有区别,字符串没有自动拆箱的功能,这一点需要特别注意。

此外,两个new出来的字符串对象使用==号判断时,也返回false。

2.2 使用equals方法

使用上面的==号,可以非常快速判断8种基本数据类型是否相等,除此之外,还能判断两个对象的引用是否相等。

但现在有个问题,它无法判断两个对象在内存中具体的数据值是否相等,比如:

String g = new String("abc");
String h = new String("abc");
System.out.println(g == h);
//结果:false

字符串对象g和h是两个不同的对象,它们使用==号判断引用是否相等时,返回的是false。

那么,这种对象不同,但数据值相同的情况,我们该如何判断相等呢?

答:使用equals方法。

equals方法其实是Object类中的方法:

public boolean equals(Object obj) {
return (this == obj);
}

该方法非常简单,只判断两个对象的引用是否相等。

很显然,如果字符串类型直接使用父类(即Object类)的equals方法,去判断对象不同,但值相同的情况,是有问题的。

所以,字符串(即String类)会重新的equals方法:

public boolean equals(Object anObject) {
if (this == anObject) {
return true;
}
if (anObject instanceof String) {
String anotherString = (String)anObject;
int n = value.length;
if (n == anotherString.value.length) {
char v1[] = value;
char v2[] = anotherString.value;
int i = 0;
while (n-- != 0) {
if (v1[i] != v2[i])
return false;
i++;
}
return true;
}
}
return false;
}

它依然会先判断两个对象引用是否相等,如果相等返回true。接下来,会把两个字符串的挨个字符进行比较,只有所有字符都相等才返回true。

nice,这样就能解决g和h判断的问题:

String e = "abc";
String f = "abc";
System.out.println(e.equals(f));
//结果:true

由此可见,我们使用String类重写后的equals方法,判断两个字符串对象不同,但值相同时,会返回true。

3. 空指针异常

从前面我们已经知道,判断两个对象是否相等,可以使用==号,或者equals方法。

但如果你更深入的使用它们,会发现一个问题,即:这两种方式判断值相等,都可能会报空指针异常。

先看看==号判断的情况:

int a = 1;
Integer b = new Integer(1);
Integer c = null;
System.out.println(a == b);
//结果:true
System.out.println(a == c);
//结果:NullPointerException

int和Integer使用==号判断是否相等时,Integer会自动拆箱成int。

但由于c在自动拆箱的过程中,需要给它赋值int的默认值0。而给空对象,赋值0,必然会报空指针异常。

接下来,看看equals方法:

String e = null;
String f = "abc";
System.out.println(e.equals(f));
//结果:NullPointerException

由于字符串对象e是空对象,直接调用它的equals方法时,就会报空指针异常。

那么,如何解决空指针问题呢?

答:在代码中判空。

String e = null;
String f = "abc";
System.out.println(equals(e, f));

我们抽取了一个新的equals方法:

private static boolean equals(String e, String f) {
if (e == null) {
return f == null;
}
return e.equals(f);
}

该方法可以解决空指针问题,但有没有办法封装一下,变得更通用一下,也适用于Integer或者其他类型的对象比较呢?

答:有办法,继续往下看。

4. Objects.equals的作用

Objects类位于java.util包下,它是里面提供了很多对象操作的辅助方法。

下面我们重点看看它的equals方法:

public static boolean equals(Object a, Object b) {
return (a == b) || (a != null && a.equals(b));
}

equals方法的判断逻辑如下:

  • 该方法先判断对象a和b的引用是否相等,如果相等则直接返回true。
  • 如果引用不相等,则判断a是否为空,如果a为空则返回false。
  • 如果a不为空,调用对象的equals方法进一步判断值是否相等。

该方法是如何使用的?

int a = 1;
int b = 1;
Integer c = null;

System.out.println(Objects.equals(a, c));
//结果:false
System.out.println(Objects.equals(c, a));
//结果:false
System.out.println(Objects.equals(a, b));
//结果:true

从上面的列子看出,使用Objects.equals方法比较两个对象是否相等,确实可以避免空指针问题。

但这个有个疑问:前面使用a==b这种方式比较引用是否相等,当时b为空时,程序直接抛了空指针异常。

而Objects.equals方法内部也使用了a==b比较引用是否相等,为啥它没有抛异常?

答:因为而Objects类的equals方法,使用了Object类型接收参数,它的默认值是null,不用进行类型转换,也不用像int类型对象赋值默认值0。

从上面的理论可以看出,如果我们把代码改成这样,也不会抛异常:

int a = 1;
Integer c = null;
System.out.println(equals(a, c));
//结果:false

新定义了一个方法:

private static boolean equals(Object a, Object b) {
return a == b;
}

执行之后发现,确实没有抛空指针了。

所以Objects.equals方法再比较两个对象是否相等时,确实是一个不错的方法。

但它有坑,不信继续往下看。

5. Objects.equals的坑

各位小伙们看到这里,可能有点心急了,到底是什么坑?

废话不多说,直接上例子:

Integer a = 1;
long b = 1L;
System.out.println(Objects.equals(a, b));
//结果:false

什么?返回结果是false?

而如果你直接用==号判断:

Integer a = 1;
long b = 1L;
System.out.println(a == b);
//结果:true

返回又是true。

a和b明明都是1,为什么使用Objects.equals方法判断不相等呢?

这就要从Integer的equals方法说起来了。

它的equals方法具体代码如下:

public boolean equals(Object obj) {
if (obj instanceof Integer) {
return value == ((Integer)obj).intValue();
}
return false;
}

先判断参数obj是否是Integer类型,如果不是,则直接返回false。如果是Integer类型,再进一步判断int值是否相等。

而上面这个例子中b是long类型,所以Integer的equals方法直接返回了false。

也就是说,如果调用了Integer的equals方法,必须要求入参也是Integer类型,否则该方法会直接返回false。

原来坑在这里!!!

其实,如果把代码改成这样:

Integer a = 1;
long b = 1L;
System.out.println(Objects.equals(b, a));
//结果:false

执行结果也是false。

因为Long的equals方法代码,跟之前Integer的类似:

public boolean equals(Object obj) {
if (obj instanceof Long) {
return value == ((Long)obj).longValue();
}
return false;
}

也是判断入参,如果不是Long类型,则该方法直接返回false。

除此之外,还有Byte、Short、Double、Float、Boolean和Character也有类似的equals方法判断逻辑。

由此可见,我们在使用Objects.equals方法,判断两个值是否相等时,一定要保证两个入参的类型要一致。否则即使两个值相同,但其结果仍然会返回false,这是一个大坑。

那么,如何解决上面的问题呢?

可以将参数b的类型强制转换成int。

Integer a = 1;
long b = 1L;
System.out.println(Objects.equals(a, (int)b));
//结果:true

或者将参数a的类型强制转换成long。

Integer a = 1;
long b = 1L;
System.out.println(Objects.equals(b, (long)a));
//结果:true

有些情况也可以直接用==号判断:

Integer a = 1;
long b = 1L;
System.out.println(a==b);
//结果:true

除了Objects.equals方法在两个入参类型不同,而会直接返回false之外,java的8种基本类型包装类的equals也会有相同的问题,需要小伙们特别注意。

之前,如果直接使用java基本类型包装类的equals方法判断相等。

Integer a = new Integer(1);
long b = 1L;
System.out.println(a.equals(b));

在idea中,如果你将鼠标放在equals方法上,会出现下面的提示:

这时你就知道方法用错了,赶紧修正。但如果直接用包装类的equals方法,有个问题就是可能存在报空指针异常的风险。

如果你使用Objects.equals方法判断相等,在idea中就并没有错误提示。

除此之外,我还测试了findBug、sonar等工具,Objects.equals方法两个参数类型不一致的问题,也没有标识出来。

小伙们赶紧看看你们的代码,踩坑了没?

常见的坑有:

  • Long类型和Integer类型比较,比如:用户id的场景。
  • Byte类型和Integer类型比较,比如:状态判断的场景。
  • Double类型和Integer类型比较,比如:金额为0的判断场景。
责任编辑:武晓燕 来源: 苏三说技术
相关推荐

2020-02-17 10:34:04

箭头函数开发 JavaScript

2018-03-16 17:25:22

存储

2021-02-24 09:43:36

MySQL数据库双引号

2023-01-05 07:55:59

Zookeeper服务注册

2021-07-28 05:01:29

Lombok前端测试

2023-06-20 08:25:53

NESTED源码mybatis

2023-12-15 09:21:17

ObjectJavaString

2020-03-17 21:53:09

路由器WiFi信号

2021-03-30 15:10:50

Java序列化

2022-11-14 07:08:23

Python函数参数

2020-03-13 08:41:47

无线路由器路由器无线技术

2018-04-16 10:22:08

超融合基础设施

2024-05-29 08:49:45

2024-06-14 10:26:30

2021-01-08 10:20:16

大数据网络数据

2020-03-12 13:58:19

MySQL分页数据库

2023-10-08 07:13:19

equalshashCode哈希表

2018-03-15 10:56:42

无线路由网口

2018-02-09 05:08:26

2020-10-28 15:07:01

Arthas
点赞
收藏

51CTO技术栈公众号