(七) 控制语句

1.【强制】在一个switch块内,每个case要么通过break/return等来终止,要么注释说明程序将继续执行到哪一个case为止;在一个switch块内,都必须包含一个default语句并且放在最后,即使它什么代码也没有。

2.【强制】在if/else/for/while/do语句中必须使用大括号,即使只有一行代码,避免使用下面的形式:if (condition) statements;

3.【推荐】推荐尽量少用else, if-else的方式可以改写成:

  1. if(condition){
  2. ...
  3. return obj;
  4. }
  5. //接着写else的业务逻辑代码;
  6. 说明:如果非得使用if()...else if()...else...方式表达逻辑,【强制】请勿超过 3层,
  7. 超过请使用状态设计模式。
  8. 正例:逻辑上超过 3层的if-else代码可以使用卫语句,或者状态模式来实现。

4.【推荐】除常用方法(如 getXxx/isXxx)等外,不要在条件判断中执行其它复杂的语句,将复杂逻辑判断的结果赋值给一个有意义的布尔变量名,以提高可读性。

说明:很多 if语句内的逻辑相当复杂,阅读者需要分析条件表达式的最终结果,才能明确什么样的条件执行什么样的语句,那么,如果阅读者分析逻辑表达式错误呢?

  1. 正例:
  2. //伪代码如下
  3. boolean existed = (file.open(fileName, "w") != null) && (...) || (...);
  4. if (existed) {
  5. ...
  6. }
  7. 反例:
  8. if ((file.open(fileName, "w") != null) && (...) || (...)) {
  9. ...
  10. }

5.【推荐】循环体中的语句要考量性能,以下操作尽量移至循环体外处理,如定义对象、变量、获取数据库连接,进行不必要的try-catch操作(这个try-catch是否可以移至循环体外)。

6.【推荐】接口入参保护,这种场景常见的是用于做批量操作的接口。

7.【参考】方法中需要进行参数校验的场景:

  • 调用频次低的方法。
  • 行时间开销很大的方法,参数校验时间几乎可以忽略不计,但如果因为参数错误导致中间执行回退,或者错误,那得不偿失。
  • 需要极高稳定性和可用性的方法。
  • 对外提供的开放接口,不管是RPC/API/HTTP接口。
  • 敏感权限入口。

8.【参考】方法中不需要参数校验的场景:

  • 极有可能被循环调用的方法,不建议对参数进行校验。但在方法说明里必须注明外部参数检查。
  • 底层的方法调用频度都比较高,一般不校验。毕竟是像纯净水过滤的最后一道,参数错误不太可能到底层才会暴露问题。一般DAO层与Service层都在同一个应用中,部署在同一台服务器中,所以DAO的参数校验,可以省略。
  • 被声明成private只会被自己代码所调用的方法,如果能够确定调用方法的代码传入参数已经做过检查或者肯定不会有问题,此时可以不校验参数。