事故场景还原
最近在写一个项目的时候遇到一个这样一个问题,我简单的还原一下场景,这是模拟一个简单的管理系统
① 一张简单的客户表
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 方法
/*** 通过id以及name字段查询用户信息* @param id* @param name* @return*/public List queryByIdAndName(String id,String name) {try {return new CustormerDaoImpl().queryByIdAndName(id, name);} catch (SQLException e) {e.printStackTrace();return null;}}
③ 调用
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中的代码是这样的
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 集合分配空间,即未实例化
你可以这样理解,逢时过节,锅里在煮饺子,你只有你的碗,在等饺子熟,所以此时你的碗中没有任何饺*,也就是集合为空的概念,但是如果你连碗都没拿,这个时候你就必须拿了碗,(分配了空间),饺子熟了你才有地方可以装饺子,但是你如果没有碗,就直接给你倒饺子,这个时候,就出现了空指针异常
总结:
- isEmpty() : 用于判断List中元素是否为空,必须在已经分配内存空间的前提下,否则报出异常
- == null : 用于判断 List 集合是否已经被分配内存空间
- list.size() == 0 : 与 isEmpty() 方法效果一致,但更推荐使用 isEmpty()
简单代码证明:
- 未实例化的情况
ArrayList list = null;System.out.println(null == list);//返回 trueSystem.out.println(list.isEmpty());// 空指针异常
- 实例化后的情况
ArrayList list = new ArrayList();System.out.println(list.isEmpty());//返回 trueSystem.out.println(list == null);//返回 false
问题解决 以及搭配方式
由于我们将具体的 List实例化代码写在了DaoImpl层中的具体实现中 List custormer = new ArrayList<>();,所以执行校验方法后,实际上list已经被实例化了,所以如果使用 list != null来进行判断,这就会导致一直返回 true,所以我们将判断语句改为 if ( !list.isEmpty())就可以了
那么在我们实际开发中可以这样搭配,先判断是否已经分配空间,防止isEmpty判空时空指针异常,同时满足集合中没有元素
if(list!= null && !list.isEmpty()){ //不为空执行的代码}else{ //为空执行的代码}
还有一种做法就是在Service层中不返回null,而是返回一个空的集合 (已经实例化)
仁者见仁,智者见智,你觉得呢?
虽然这个问题很基础,但是确实让我头疼了好一阵,在开发的过程中,你遇到的一些莫名其妙的坑,往往来于自身对于基础知识的掌握没有很彻底,基础无论到什么时候,都是很重要啊!
结尾:
如果文章中有什么不足,或者错误的地方,欢迎大家留言分享想法,感谢朋友们的支持!
如果能帮到你的话,那就来关注我吧
最后,我自己是一名从事了多年开发的JAVA老程序员,今年年初我花了一个月整理了一份最适合2019年学习的java学习干货,可以送给每一位喜欢java的小伙伴,想要获取的可以关注我的头条号并在后台私信我:java,即可免费获取。
作者:BWH_Steven