Java 包装类型 == 操作引发的 Bug

背景

一个简单的列表检索功能,列表元素有一个 Long 类型的属性,遍历过程中犯了一个低级错误,导致功能流程始终错误,本文将分享两个低级错误引发的 bug。

两个 Long 类型的 ==

查找某个元素 A 在列表 B 中对应的对象的时候,根据元素主键查询,主键类型为包装类型 Long ,遍历流程如下:

for(MyData temp:b){
     if (temp.getId() == a.getId() { //  MyData 的 id 属性为 Long 类型
          return temp;
     }
}

这么一段简单的查找代码,结果怎么都找不到目标对象,断点调试发现问题出在 == 操作上,改成 equals 就可以了。

关于 Java Long 的包装类型和元素类型的判断相等的操作回顾:

Long a = 81487354807713792L;
Long b = 81487354807713792L;
System.out.println(a==b); // false
System.out.println(a.equals(b)); // true

long c = 81487354807713792L;
long d = 81487354807713792L;
System.out.println(c==d); // true

对 Collections.EMPTY_SET 进行 add 引发的异常

另一个低级错误是对 Collections.EMPTY_SET 进行 add 引发的,需要合并两个集合,第一个集合 A 可能是 Collections.EMPTY_SET ,最终将另一个集合 B 合并到 A 得到一个大集合。

Set<MyData> a = getDatas();// 如果为空,返回了 Collections.EMPTY_SET
Set<MyData> b = getDatas1();
a.addAll(b);

当集合 a 为集合的空对象时,操作异常:

Exception in thread "main" java.lang.UnsupportedOperationException
	at java.util.AbstractCollection.add(AbstractCollection.java:262)

修正方式 :如果需要直接对一个集合进行 add ,就不能用 Collections.EMPTY_SET

Collections 的空集合使用注意事项

Collections.EMPTY_SET 为例,跟源码它的定义主要是下面三行代码:

public static final Set EMPTY_SET = new EmptySet<>();

private static class EmptySet<E> extends AbstractSet<E>

AbstractSet<E> extends AbstractCollection

核心在于 AbstractCollection 类的 add,默认直接抛出了异常,限制了空集合不允许添加:

public boolean add(E e) {
     throw new UnsupportedOperationException();
 }

结论java.util.Collections 类中所有的 EMPTY_XXX 对象都不能进行 add 操作。

启示录

定位到这两个低级错误后,想起那句调侃:代码编写分分钟,bug 查找两小时。这两个问题恰好是一个比较复杂的流程的一部分,构建环境进行测试,测一次差不多十几分钟,加上机器怠工,跟这俩小问题,耗了两个小时。

以此文为记,这么深刻的过程,估计以后也不会再犯了!

相关推荐
小扳1 分钟前
微服务篇-深入了解使用 RestTemplate 远程调用、Nacos 注册中心基本原理与使用、OpenFeign 的基本使用
java·运维·分布式·后端·spring·微服务·架构
阿乾之铭2 分钟前
Spring Framework的常用工具类
java
军训猫猫头5 分钟前
44.扫雷第二部分、放置随机的雷,扫雷,炸死或成功 C语言
c语言·开发语言
北巷!!6 分钟前
宇信科技JAVA笔试(2024-11-26日 全部AK)
java·开发语言·科技
ᝰꫝꪉꪯꫀ3617 分钟前
JavaWeb——SpringBoot原理
java·开发语言·后端·springboot
Octopus207710 分钟前
【C++】读取数量不定的输入数据
开发语言·c++·笔记·学习
忘梓.10 分钟前
C嘎嘎探索篇:栈与队列的交响:C++中的结构艺术
c语言·开发语言·c++·
HaiFan.12 分钟前
Spring日志
java·spring boot
爬山算法13 分钟前
Tomcat(36)Tomcat的静态资源缓存
java·缓存·tomcat
TPBoreas15 分钟前
手搓一个不用中间件的分表策略
java