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

xiaoronglv · March 12, 2015 · Last by xiaoronglv replied at March 15, 2015 · 2451 hits

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

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

1. 表

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

2. 数据库索引

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

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

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

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

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

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

当局者迷,旁观者清

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

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

You need to Sign in before reply, if you don't have an account, please Sign up first.