复盘code review
近两周都code review,CR中挣扎,主要还是所有的工作都被CR严重拖累,被问了太多为什么,而不是有价值的comment,每次review都会经历这样的事情,而且是非收敛式的。心里真想骂娘。
reviewer的逻辑
- 始终带着一个问题“这个feature如果我来开发,我会怎么写”
- 依赖于代码命名来理解代码
- 自己没法理解或者看不懂的就是代码设计不好
- “听风就是雨”——会带入自己遇到的问题,希望commiter的代码可以解决自己的问题
committer的逻辑
- 功能逻辑是否满足要求
- 是否有未考虑的异常场景
- 代码细节是否有问题(
code style rules
) - 是否有测试覆盖
思考总结
千万不要以committer的角度来review代码,reviewer还是需要本着尊重作者的角度来查漏补缺(逻辑,代码规范,测试,重大的设计问题)。
特别说明:这里针对的是团队内部的交付场景
#反思总结