| ... | ... | @@ -3,16 +3,18 @@ |
|
|
|
2. 代码的会味道
|
|
|
|
3. 规范性
|
|
|
|
|
|
|
|
### 实现方案的正确性
|
|
|
|
* 被评审人需先整体描述需要解决的问题、解决流程
|
|
|
|
|
|
|
|
|
|
|
|
### 流程
|
|
|
|
1. 被评审人需先整体描述需要解决的问题、解决流程 （被评审人讲解过程中，评审人可以记录问题，不要打断被评审者的思路）
|
|
|
|
2. 被评审人讲完，评审人和与会人员可以提问题
|
|
|
|
3. 评审人进行评审 （被评审者或者与会人员记录评审待改进的内容，有时并不是只针对被评审者，而是所有编码者）
|
|
|
|
4. 评审完成之后，落实待修改项，主要是缺陷和规范性
|
|
|
|
|
|
|
|
### 代码的会味道
|
|
|
|
1. 无效代码
|
|
|
|
2. 重复代码
|
|
|
|
3. 异常处理
|
|
|
|
4. 必要注释
|
|
|
|
5. 。。。
|
|
|
|
|
|
|
|
|
|
|
|
### 规范性
|
| ... | ... | @@ -22,7 +24,10 @@ |
|
|
|
---
|
|
|
|
|
|
|
|
### 改进落实
|
|
|
|
缺陷：
|
|
|
|
|
|
|
|
规范性：
|
|
|
|
|
|
|
|
时间：
|
|
|
|
负责人：
|
|
|
|
|
|
|
|
必须纠正： |
|
|
\ No newline at end of file |
|
|
|
负责人： |
|
|
\ No newline at end of file |
