前段时间在阅读别人所写的代码的时候 , 发现其中一些业务相关的方法体内 , 出现了比较多的if-else
语句多层嵌套的状况 . 首先我我的不是不提倡写if-else
语句 , 不得不说 , 不少时候 , 在写某些逻辑 使用if-else
去作判断 , 代码看起来仍是十分直观的 , 可是若是滥用if-else
, 造成多层嵌套或者造成, 其中每一个case 还包含了大量的逻辑 , 此时从可读性来讲 , 使用if-else
就有点得不偿失了 . 并且某些时候 , 可能并不需这么多的if-else
, 或者是可使用其余编码方式从而达到减小的if-else
的效果 .java
减小if-else
的使用的方式有不少 , 例如设计模式层面的策略模式或者是责任链模式 . 而这里跟你们分享一下一些我的在平常编码过程当中常常用到的 , 比较简单的 、从编码习惯层面上的方式 , 去一些减小没必要要if-else
使用 . 因为本人只是一个小菜鸟 , 若是有写得不对的地方 , 恳请批评指正 .数据库
首先展现一段代码示例 :express
if (condition1) { if (condition2) { return getSomething(); } else { return 0; } } else { return 0; } 复制代码
修改后的代码以下 :设计模式
//这里最好对这个flag所判断的逻辑补充注释进行描述 boolean flag = !condition1 || (condition1 && !condition2) if(flag) { return 0; } if (condition1 && condition2) { return getSomething(); } 复制代码
若是存在已知在某些条件下 , 须要返回固定值的逻辑 , 能够将这部分逻辑抽取为一个独立的 if-else block
, 并置于其余if-else block
的前面 , 当符合该特定条件时 , 直接提早 return
固定值 . 这种方式最直接的效果就是下降if-else
的嵌套数量 .数组
先上例子 , 这里以一个业务场景为例 :markdown
查询某条评论的图片URL列表 (若是有 , 评论的图片url列表以JSON数组字符串格式保存在评论表中)mybatis
修改前的代码以下 :框架
Comment comment = getById(commentId); if (Objects.isNull(comment)) { throw new RuntimeException("评论不存在或已被删除"); } String imgListStr = comment.getImgList(); if(StringUtils.isEmpty(imgListStr)) { return null; } return JSON.parseArray(imgListStr, String.class); 复制代码
修改后 :工具
Comment comment = getById(commentId); if (Objects.isNull(comment)) { throw new RuntimeException("评论不存在或已被删除"); } String imgListStr = comment.getImgList(); return StringUtils.isEmpty(imgListStr)) ? null : JSON.parseArray(imgListStr, String.class); 复制代码
在编写业务代码过程当中 , 若是须要对某些特定的条件进行判断 , 且当条件不知足时须要抛出异常 . 对于这种场景 , 除了使用上面三目运算符的示例当中的if
方式 , 还能够经过使用Spring Framework 给咱们提供的 Assert
工具类进行 .编码
其中经常使用的API 有 :
expressio == false
时 , 会抛出异常 , 异常的message
则为第二个入参 ;object == null
时 , 会抛出异常;null
或者集合元素为空时 , 会抛出异常 .还有其余较多方法 , 能够直接看源码的解析 , 固然实际上isTrue()
已经够用了 , 若是须要更加的语义化 , 可使用对应的API .
修改前代码 :
if (Objects.isNull(comment)) { throw new RuntimeException("评论不存在或已被删除"); } 复制代码
修改后代码 :
Assert.isTrue(Objects.nonNull(comment),"评论不存在或已被删除"); Assert.notNull(comment,"评论不存在或已被删除"); 复制代码
目前Assert工具方法只能抛出单一一种异常 java.lang.IllegalArgumentException
, 若是须要自定义所抛出的异常 , 则该方式不适用 .
Optional是 java8 的新特性 , 至关于一个对象的容器 , 主要用于对象的null
值校验 , 以及在进行校验后可链式地进行后续操做 , 如 : 抛出异常、null
替换 等 .
其中我我的比较经常使用的几个方法为 :
null
时 ,返回入参的对象 .null
时 , 返回Supplier
提供的值 ;null
时 , 抛出supplier
提供的自定义异常代码示例 :
Message message1 = Optional.ofNullable(getById(messageId)) .orElseThrow(() -> new RuntimeException("消息不存在!")); Message message2 = Optional.ofNullable(getById(messageId)) .orElse(new Message()); Message message3 = Optional.ofNullable(getById(messageId)) .orElseGet(Message::new); 复制代码
因为我平常须要的进行空值判断的比较多的场景是从数据库查询数据完毕时 , 须要查询结果进行空值判断 . 因为我所在的公司使用的持久层框架是mybatis , 不像Spirng Boot 2.x 默认版本的JPA 那样DAO层方法支持返回值为Optional , 因此这里若是须要使用Optional , 只能手动去使用上面列举的第一个方法对查询结果进行包装 .
固然 , IDEA其实已经给咱们提供了该包装方式的热键了 , 以下图所示 :
我的的一些减小if-else
编码习惯分享就这里了 , 这几种方式里面 , 我我的以为效果最明显的仍是第一种 提早return , 不少时候 , 提早return 也能够很好下降一段代码的复杂度 .
固然若是必需要使用大量的if-else
去控制逻辑时 , 在每一个condition 标明一下注释仍是一个挺不错的习惯 .