- 优点：保证代码质量，保证工程代码风格一致。
- 缺点：过度 review 导致项目效率低下，遇到不专业的 review 只想喷人。
最近就遇到个烦人的，改来改去，就写个模块的接口，反反复复改，想到喊我改到哪。 比如一个函数：
int xxxx_init(CTX * ctx,A *a){
if (xxx){
return rc;
}
xxxx // 实际逻辑代码段
return rc;
}
一般来说，项目风格就是这样的，先检查 ctx 啥的，然后如果实际逻辑，最后返回调用状态。 这个方法能给我提 3 个 comments 。
- it is simple memcpy... do we really need all the checks below?
- i guess this function should return，remove A * a 。
- is it documented on API level? (实际逻辑代码段上一顿 bb)
看到这个我都不想回也不再改了，想问你调用 CTX 是有状态的，你调用函数前不需要 check ？其二本身方法逻辑就是大页来分配 A，名字也叫 xxx_init，我还返回个毛球啊。
搞得真想跑路。
2
Select Language