Note when going straight for success of Mr.Wang

0%

控制语句

发表于 更新于 分类于

控制语句

  1. 在一个 switch 块内，每个 case 要么通过 continue/break/return 等来终止，要么注释说明程序将继续执行到哪一个 case 为止；在一个 switch 块内，都必须包含一个 default 语句并且放在最后，即使它什么代码也没有。
    reason: 有确定性的退出程序，不然以后程序会执行到哪个地方自己都搞不清楚

  2. 当 switch 括号内的变量类型为 String 并且此变量为外部参数时，必须先进行 null 判断
    reason：根本不会进入语句体，直接NPE（NullPointerException）

  3. 在 if/else/for/while/do 语句中必须使用大括号(哪怕只有一行逻辑代码)

  4. 在高并发场景中，避免使用”等于”判断作为中断或退出的条件（使用大于或小于的区间判断条件来代替）
    reason: 如果并发控制没有处理好，容易产生等值判断被“击穿”的情况 栗子：判断剩余奖品数量等于 0 时，终止发放奖品，但因为并发处理错误导致奖品数量瞬间变成了负数，这样的话，活动无法终止。

  5. 表达异常的分支时，少用 if-else 方式
    reason: 后期代码维护困难
    正例：

    1
    2
    3
    4
    5
    if (condition) {
    	...
    	return obj;
    }
    // 接着写 else 的业务逻辑代码

  6. if else 语句不要超过 3 层（超过 3 层的 if-else 的逻辑判断代码可以使用卫语句、策略模式、状态模式等来实现）
    知识点：卫语句即代码逻辑先考虑失败、异常、中断、退出等直接返回的情况，以方法多个出口的方式，解决代码中判断分支嵌套的问题

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

  8. 不要在其它表达式（尤其是条件表达式）中插入赋值语句（赋值语句单独占一行来执行）。
    reason: 提高代码可读性
    反例：

    1
    2
    3
    4
    5
    6
    public Lock getLock(boolean fair) {
    	// 算术表达式中出现赋值操作，容易忽略 count 值已经被改变
    	threshold = (count = Integer.MAX_VALUE) - 1;
    	// 条件表达式中出现赋值操作，容易误认为是 sync==fair
    	return (sync = fair) ? new FairSync() : new NonfairSync();
    }

  9. 循环体中的语句要考量性能，以下操作尽量移至循环体外处理，如定义对象、变量、获取数据库连接，进行不必要的 try-catch 操作（这个 try-catch 是否可以移至循环体外）

  10. 避免采用取反逻辑运算符
    reason：提高代码可读性
    正例：使用 if (x < 628) 来表达 x 小于 628。
    反例：使用 if (!(x >= 628)) 来表达 x 小于 628。

  11. 接口入参保护，这种场景常见的是用作批量操作的接口
    todo：

  12. 参数校验：

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

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

-------------the end-------------