CoDE REVIEW怎么做？[隐]

Code Review 检查什么

结构问题 重复拷贝代码（不封装函数、不用泛型等） 函 数过长（有人说超过一屏幕就叫过长，见仁见智） 错 误封装（不恰当的 public 、不用 interface 、不内聚、强耦合、在类中封装了无关的方法等） 内 容错误（多个无关类置于一个文件、不恰当的命名等） 慎用全局变量，尽量不用。 业 务逻辑问题 就是审查业务逻辑是否理解一致，必要的时候需要引入产品方确认。 编 程素养问 题 很多问题属于“这样也行、那样也行”的状态，比如命名、初始值、缩进、断行等 举例： boolean result=true; 检查软件升级，添加或修改的代码，是否会影响其他部分

做Code Review，应该“少吃多餐”

每次5分钟、10分钟 每天2~3次甚至更多

一些建议

编程开始前，对程序员做编程规范的培训 结对编程 引入工具（FindBugs、CheckStyle、PMD） 4、工具介绍  

做Code Review的一些工具

  检查bytecode中的bug patterns，允许自定义bug patterns

CheckStyle

     检查java源文件是否与代码规范相符。

PMD

  检查System.out.print、printStackTrace，没有打印语句

  空 try…catch…finally 、 if/while 等 未使用的局部变量、参数和 private 方法等 复杂的表达式和类，不必要的 if 语句 等