新手问题 [已解决] review 代码的要点是什么?

xiaoronglv · 2015年03月12日 · 最后由 xiaoronglv 回复于 2015年03月15日 · 2453 次阅读

你们一般是如何 review 代码?求经验之谈。

我在 review 代码时,就检查这么几点

1. 表

* 表结构对吗? * 字段名是否符合习惯?比如时间用 *_at, 日期用 *_on

2. 数据库索引

* 唯一值有没有加 唯一索引 做约束? * 联合索引的顺序是否正确?

3. 乱七八糟的逻辑有没有封装到 serve 层或 concern 层?

4. 单元测试都覆盖了吗?

业务实现是否有偏差 代码是否会影响到已有实现

#2 楼 @ruby_sky 你说的第二个,应该通过写测试来保证。

#3 楼 @xiaoronglv 关键的关键实际情况,很多时候,测试是顾不上来写的。

当局者迷,旁观者清

#1 楼 @huacnlee 多久一次 review 呢?

#6 楼 @mogodb 每次 pull request 都要 review

需要 登录 后方可回复, 如果你还没有账号请 注册新账号