99%的Java程序员会踩的6个坑

2022-07-18 14:59:38

前言

作为Java程序员的你，不知道有没有踩过一些基础知识的坑。

有时候，某个bug，你查了半天，后发现竟然是一个非常低级的错误。

有时候，某些代码，这一批数据功能正常，但换了一批数据就出现异常了。

有时候，你可能会看着某行代码目瞪口呆，心里想：这行代码为什么会出错？

今天跟大家一起聊聊99%的Java程序员踩过，或者即将踩的6个坑。

1. 用==号比较的坑

不知道你在项目中有没有见过，有些同事对Integer类型的两个参数使用==号比较是否相等？

反正我见过的，那么这种用法对吗？

我的回答是看具体场景，不能说一定对，或不对。

有些状态字段，比如：orderStatus有：-1(未下单)，0（已下单），1（已支付），2（已完成），3（取消），5种状态。

这时如果用==判断是否相等：

Integer orderStatus1 = new Integer(1);
Integer orderStatus2 = new Integer(1);
System.out.println(orderStatus1 == orderStatus2);

返回结果会是true吗？

答案：是false。

有些同学可能会反驳，Integer中不是有范围是：-128-127的缓存吗？

为什么是false？

先看看Integer的构造方法：

它其实并没有用到缓存。

那么缓存是在哪里用的？

答案在valueOf方法中：

如果上面的判断改成这样：

String orderStatus1 = new String("1");
String orderStatus2 = new String("1");
System.out.println(Integer.valueOf(orderStatus1) == Integer.valueOf(orderStatus2));

返回结果会是true吗？

答案：还真是true。

我们要养成良好编码习惯，尽量少用==判断两个Integer类型数据是否相等，只有在上述非常特殊的场景下才相等。

而应该改成使用equals方法判断：

Integer orderStatus1 = new Integer(1);
Integer orderStatus2 = new Integer(1);
System.out.println(orderStatus1.equals(orderStatus2));

运行结果为true。

2. Objects.equals的坑

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

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

UserInfo userInfo = CurrentUser.getUserInfo();

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

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

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

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

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

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

后来，发现UserInfo类是这样定义的：

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

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

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

什么问题呢？

下面我们重点看看它的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方法进一步判断值是否相等。

这就要从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。

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

常见的坑有：

Long类型和Integer类型比较，比如：用户id的场景。
Byte类型和Integer类型比较，比如：状态判断的场景。
Double类型和Integer类型比较，比如：金额为0的判断场景。

如果你想进一步了解Objects.equals方法的问题，可以看看我的另一篇文章《Objects.equals有坑》。

3. BigDecimal的坑

通常我们会把一些小数类型的字段（比如：金额），定义成BigDecimal，而不是Double，避免丢失精度问题。

使用Double时可能会有这种场景：

double amount1 = 0.02;
double amount2 = 0.03;
System.out.println(amount2 - amount1);

正常情况下预计amount2 - amount1应该等于0.01

但是执行结果，却为：

.009999999999999998

实际结果小于预计结果。

Double类型的两个参数相减会转换成二进制，因为Double有效位数为16位这就会出现存储小数位数不够的情况，这种情况下就会出现误差。

常识告诉我们使用BigDecimal能避免丢失精度。

但是使用BigDecimal能避免丢失精度吗？

答案是否定的。

为什么？

BigDecimal amount1 = new BigDecimal(0.02);
BigDecimal amount2 = new BigDecimal(0.03);
System.out.println(amount2.subtract(amount1));

这个例子中定义了两个BigDecimal类型参数，使用构造函数初始化数据，然后打印两个参数相减后的值。

结果：

.0099999999999999984734433411404097569175064563751220703125

不科学呀，为啥还是丢失精度了？

Jdk中BigDecimal的构造方法上有这样一段描述：

大致的意思是此构造函数的结果可能不可预测，可能会出现创建时为0.1，但实际是0.1000000000000000055511151231257827021181583404541015625的情况。

由此可见，使用BigDecimal构造函数初始化对象，也会丢失精度。

那么，如何才能不丢失精度呢？

BigDecimal amount1 = new BigDecimal(Double.toString(0.02));
BigDecimal amount2 = new BigDecimal(Double.toString(0.03));
System.out.println(amount2.subtract(amount1));

我们可以使用Double.toString方法，对double类型的小数进行转换，这样能保证精度不丢失。

其实，还有更好的办法：

BigDecimal amount1 = BigDecimal.valueOf(0.02);
BigDecimal amount2 = BigDecimal.valueOf(0.03);
System.out.println(amount2.subtract(amount1));

使用BigDecimal.valueOf方法初始化BigDecimal类型参数，也能保证精度不丢失。在新版的阿里巴巴开发手册中，也推荐使用这种方式创建BigDecimal参数。

4. Java8 filter的坑

对于Java8中的Stream用法，大家肯定再熟悉不过了。

我们通过对集合的Stream操作，可以实现：遍历集合、过滤数据、排序、判断、转换集合等等，N多功能。

这里重点说说数据的过滤。

在没有Java8之前，我们过滤数据一般是这样做的：

public List<User> filterUser(List<User> userList) {
    if(CollectionUtils.isEmpty(userList)) {
        return Collections.emptyList();
    }
    
    List<User> resultList = Lists.newArrayList();
    for(User user: userList) {
        if(user.getId() > 1000 && user.getAge() > 18)   {
           resultList.add(user);
        }
    }
    return resultList;
}

通常需要另一个集合辅助完成这个功能。

但如果使用Java8的filter功能，代码会变得简洁很多，例如：

public List<User> filterUser(List<User> userList) {
    if(CollectionUtils.isEmpty(userList)) {
        return Collections.emptyList();
    }
    
    return userList.stream()
    .filter(user -> user.getId() > 1000 && user.getAge() > 18)
    .collect(Collectors.toList());
}

代码简化了很多，完美。

但如果你对过滤后的数据，做修改了：

List<User> userList = queryUser();
List<User> filterList = filterUser(userList);
for(User user: filterList) {
   user.setName(user.getName() + "测试");
}

for(User user: userList) {
   System.out.println(user.getName());
}

你当时可能只是想修改过滤后的数据，但实际上，你会把元素数据一同修改了。

意不意外，惊不惊喜？

其根本原因是：过滤后的集合中，保存的是对象的引用，该引用只有一份数据。

也就是说，只要有一个地方，把该引用对象的成员变量的值，做修改了，其他地方也会同步修改。

如下图所示：

5. 自动拆箱的坑

Java5之后，提供了自动装箱和自动拆箱的功能。

自动装箱是指：JDK会把基本类型，自动变成包装类型。

比如：

Integer integer = 1;

等价于：

Integer integer = new Integer(1);

而自动拆箱是指：JDK会把包装类型，自动转换成基本类型。

例如：

Integer integer = new Integer(2);
int sum = integer + 5;

等价于：

Integer integer = new Integer(2);
int sum = integer.intValue() + 5;

但实际工作中，我们在使用自动拆箱时，往往忘记了判空，导致出现NullPointerException异常。

5.1 运算

很多时候，我们需要对传入的数据进行计算，例如：

public class Test2 {
    public static void main(String[] args) {
        System.out.println(add(new Integer(1), new Integer(2)));
    }

    private static Integer add(Integer a, Integer b) {
        return a + b;
    }
}

如果传入了null值：

System.out.println(add(null, new Integer(2)));

则会直接报错。

5.2 传参

有时候，我们定义的某个方法是基本类型，但实际上传入了包装类，比如：

public static void main(String[] args) {
    Integer a = new Integer(1);
    Integer b = null;
    System.out.println(add(a, b));
}

private static Integer add(int a, int b) {
    return a + b;
}