判断List集合为空还是null的正确打开方式

2019-10-21 10:58:20 浏览数 (1)

事故场景还原

最近在写一个项目的时候遇到一个这样一个问题,我简单的还原一下场景,这是模拟一个简单的管理系统

① 一张简单的客户表

代码语言:javascript复制
CREATE TABLE customer(
    id INT(11) NOT NULL AUTO_INCREMENT UNIQUE,
    NAME VARCHAR(255) NOT NULL,
    gender VARCHAR(255) NOT NULL,
    phonenumber VARCHAR(255) NOT NULL,
    balance DECIMAL(10,1) UNSIGNED,
    PRIMARY KEY (id)
);

② CustomerService 层中的 updateBalance 方法

代码语言:javascript复制
/**
* 通过id以及name字段查询用户信息
* @param id
* @param name
* @return
*/
public List<Custormer> queryByIdAndName(String id,String name) {
    try {
        return new CustormerDaoImpl().queryByIdAndName(id, name);
    } catch (SQLException e) {
        e.printStackTrace();
        return null;
    }
}

③ 调用

代码语言:javascript复制
public static void test() {
    CustormerService cs = new CustormerService();
    //用户输入的卡号和姓名,String类型,list在成员位置被定义
    list = cs.queryByIdAndName(input_id,input_name());

    if ( list != null) {
        //获取充值或消费的金额数,这里简单用一个字符串模拟
        String str = input_money;
        BigDecimal bd = new BigDecimal(str);   
        updateBalance(input_id, bd, false);
    }else {
        OptionPane.showMessageDialog(null, "没有此用户信息");
    }
}

public static void updateBalance(String id, BigDecimal consumptionAmount, Boolean type) {
    CustormerService cs = new CustormerService();
    int back = cs.updateBalance(id, consumptionAmount, type);
    if (back == 0) {
        //我们用输出语句,模拟具体的弹窗或者显示的提示语
        System.out.println("消费成功");
    } else {
        System.out.println("消费失败,请检查余额是否不足");
    }
}

注意:省略了Dao层以及实体层等具体代码,同时对代码做了一定修改以及简化,旨在还原问题,如有纰漏请见谅

我们可以看到真正的消费扣款或者存款 是在这一句中被执行的 updateBalance(input_id, bd, false);,但是在执行对用余额的修改操作之前,我们需要对用户的身份进行核实,即通过 list = cs.queryByIdAndName(input_id,input_name()); 这一句进行校验,只有满足id和name在sql中查询,可以得到返回值则校验通过,进行下一步消费操作,所以我在test()方法中增添一个if判断 判断 list 的值是否为空

但是!每一次执行,无论卡号以及姓名输入是否正确,对金额的修改操作总会执行,这让我百思不得其解

问题所在

我们Service中的代码是这样的

代码语言:javascript复制
try {
    return new CustormerDaoImpl().queryByIdAndName(id, name);
} catch (SQLException e) {
    e.printStackTrace();
    return null;
}

查得到,返回一个用户信息集合,查不到则返回null

所以在list的判断中,我理所当然的使用了 if (list != null) 这样一个判断条件,而问题恰恰出现在这里

集合为空还是null?

集合为空:集合内没有元素,即为空

  • isEmpty:boolean isEmpty() : 如果此列表不包含元素,则返回 true

null:没有对 List 集合分配空间,即未实例化

你可以这样理解,逢时过节,锅里在煮饺子,你只有你的碗,在等饺子熟,所以此时你的碗中没有任何饺*,也就是集合为空的概念,但是如果你连碗都没拿,这个时候你就必须拿了碗,(分配了空间),饺子熟了你才有地方可以装饺子,但是你如果没有碗,就直接给你倒饺子,这个时候,就出现了空指针异常

总结:

  1. isEmpty() : 用于判断List中元素是否为空,必须在已经分配内存空间的前提下,否则报出异常
  2. == null : 用于判断 List 集合是否已经被分配内存空间
  3. list.size() == 0 : 与 isEmpty() 方法效果一致,但更推荐使用 isEmpty()

简单代码证明

  • 未实例化的情况
代码语言:javascript复制
ArrayList<Student> list = null;
System.out.println(null == list);//返回 true
System.out.println(list.isEmpty());// 空指针异常
  • 实例化后的情况
代码语言:javascript复制
ArrayList<Student> list = new ArrayList<Student>();
System.out.println(list.isEmpty());//返回 true
System.out.println(list == null);//返回 false

问题解决 以及搭配方式

由于我们将具体的 List实例化代码写在了DaoImpl层中的具体实现中 List<Custormer> custormer = new ArrayList<>();,所以执行校验方法后,实际上list已经被实例化了,所以如果使用 list != null来进行判断,这就会导致一直返回 true,所以我们将判断语句改为 if ( !list.isEmpty())就可以了

那么在我们实际开发中可以这样搭配,先判断是否已经分配空间,防止isEmpty判空时空指针异常,同时满足集合中没有元素

代码语言:javascript复制
if(list!= null && !list.isEmpty()){
   //不为空执行的代码
}else{
   //为空执行的代码
}

还有一种做法就是在Service层中不返回null,而是返回一个空的集合 (已经实例化)

仁者见仁,智者见智,你觉得呢?

虽然这个问题很基础,但是确实让我头疼了好一阵,在开发的过程中,你遇到的一些莫名其妙的坑,往往来于自身对于基础知识的掌握没有很彻底,基础无论到什么时候,都是很重要啊!

如果文章中有什么不足,或者错误的地方,欢迎大家留言分享想法,感谢朋友们的支持!

0 人点赞