Code review

Code review中常注意的点有:

1. if (k = 1)

2. 数组越界,

3. malloc/free 不匹配等

4. switch语句须有 default分支

5. 工具类必须有 private 构造器

6. 空指针风险提示

7. 逻辑复杂度控制

8. 遍历的方式, 如果用lamba就都用lamba, 不要一边lamda,一边又for()

9. for循环中 break, continue的方式. 比如 通常的循环都可以先写 break, 再写continue

for (CommonAd commonAd : candidateApps) {
  	if(xxx){
      break;
  	}
  	if(xxx){
      continue;
  	}
    xxx;
}

10. 禁止出现大段相同代码

11. 不要提前 new object

for(){
  App app = new App();//这里就可能被continue掉。
  if(xxx){
    continue;
  }
}

12. 无用的变量

13. 静态变量的声明

14. 变量的命名要一目了然. 不要出现Index, i, j, k等变量, 所有的变量命名都要有 意义

15. 注释的相关性和清晰度

16. 保持类的单一职责原则, 避免过度耦合 不可维护

17. 方法的单一职责, 要么拆方法, 要么换名字