复盘code review


复盘code review


近两周都code review,CR中挣扎,主要还是所有的工作都被CR严重拖累,被问了太多为什么,而不是有价值的comment,每次review都会经历这样的事情,而且是非收敛式的。心里真想骂娘。

reviewer的逻辑

  • 始终带着一个问题“这个feature如果我来开发,我会怎么写”
  • 依赖于代码命名来理解代码
  • 自己没法理解或者看不懂的就是代码设计不好
  • “听风就是雨”——会带入自己遇到的问题,希望commiter的代码可以解决自己的问题

committer的逻辑

  • 功能逻辑是否满足要求
  • 是否有未考虑的异常场景
  • 代码细节是否有问题(code style rules
  • 是否有测试覆盖

思考总结

千万不要以committer的角度来review代码,reviewer还是需要本着尊重作者的角度来查漏补缺(逻辑,代码规范,测试,重大的设计问题)。

特别说明:这里针对的是团队内部的交付场景


#反思总结

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out /  Change )

Google photo

You are commenting using your Google account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )

Connecting to %s