[经验] 关于 Java 中的非空判断

在写项目的时候, 遇到一个问题

假设有一个控制层接口为:

    @ResponseBody
    @RequestMapping(value = "test", method = RequestMethod.GET, produces = "application/json;charset=UTF-8")
    public Result<String> test(String user, String number, String time){
        String type="error";
        try{
            rType = SendType.create_send.toString();
            return firstSendService.createSend(type,uName, pNum, time);
        }catch (Exception ex){
            LOGGER.error(ex.getMessage(), ex);
        }
        return Result.failure(type, "fail of connect");
    }

前端在调用这个接口的时候, 并没有传递参数, 于是  user, number, time 就默认为 null.

然后在服务层, 有一个方法

    public Result<String> createSend(String type, String user, String number, String time) throws Exception {
        String token = "not fond";
        try{
            if(user.isEmpty() && StringUtils.isBlank(user)){
                return Result.failure(type, "user is not null");
            }
            StringBuffer stringBuffer = new StringBuffer();
            stringBuffer.append(user).append(time);
            String signChar = stringBuffer.toString();
            token = TokenUtil.sign(signChar);
        }catch (Exception ex){
            LOGGER.error(ex.getMessage(), ex);
        }
        return Result.success(type, token);
    }

其中, 关于参数的验证中

if(user.isEmpty() && StringUtils.isBlank(user)){
    return Result.failure(type, "user is not null");
}

这里的判断条件并未被执行, 去验证了一下

换了一种方法

然后再来看看 StringUtils.isBlank();

再把 null 变成 "" 试试,

也没什么问题, 那么再来试试看 StringUtils.isNotBlank();

如上图所示, isNotBlank() 的判断是跟 isBlank() 反过来的.

于是, 得出一个结论:

(0)  对于  isEmpty() , 当字符串为 "" 时, return true; 但是 isEmpty() 不能用于判断为 null 的字符串, 会报 空指针异常 的错误,

所以 isEmpty()  更适合写  if(str == "" && str.isEmpty()) { return }

(1) 相较于 isEmpty()  只能判断 "" 的字符串, isBlank() 的功能性要更大更完善一些, 当字符串为 null 时, isBlank()  为 true; isNotBlank() 则跟 isBlank() 相反, 为 false.

所以 isBlank()  更适合写 if( str == null && StringUtils.isBlank(str)){ return }  或者  if( str == "" && StringUtils.isBlank(str)){ return }

但是

这样是不行的, 看来 isBlank() 好像看起来要更加全能一些, 因为有时候参数有可能为 null, 也有可能为"" ,用 isBlank() 的话可以避免服务器停掉, 如何优雅的抛出错误或者异常, 且避免服务器的运行因此受到影响, 是我在努力的一个方向, 希望这些可以有些帮助

再补上一点

这就奇怪了, 话说上面的 if( a.isEmpty() || StringUtils.isBlank(a)){ } 中, || 不是如果左边不成功就使用右边的判断条件么? 为什么会直接报错呢? 那如果是 || 的判断是先判断左边, 那在 a == "" 的时候,为什么又可以成功运行? isEmpty() 这么坑的么?

再来一组对比

那还是强烈推荐使用  isBlank()  吧, "" 和 isEmpty() 是局限性有点大, 灵活性不够高, 容易出事....