CoDE REVIEW怎么做？[隐]

1、 什么是 Code Review？

Review 代码评审是指在软件开发过程中，通过对源代码进行系统性检查的过程。通常的目的是查找各种 缺陷 ，包括代码缺陷、功能实现问题、编码合理性、性能优化等；保证软件总体质量，提供开发者自身水平。

Code Review 是轻量级代码评审，相对于正式代码评审，轻量级代码评审所需要的各种成本要明显低得多，如果流程正确，它可以起到更加积极的效果。因此，轻量级代码评审经常性地被引入到软件开发过程中。 [ 摘 自网络 ]

 

2、 为什么 Code Review？

提 高代码质量；

及 早发现潜在缺陷，降低修改缺陷的成本；

促 进团队内部知识共享，提供团队整体水平；

评 审过程中对于评审人员 来说，也是一种思路重构的过程；

是一个传递知识的手段，可以让其他并不熟悉代码的人知道作者的意图和想法，从而可以在以后轻松维护代码；

鼓 励程序员们相互学习对方的长处和优点；

可 以被用来确认自己的设计和实现是一个清楚和简单的

 

3、 如何做 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 语句 等