获得徽章 0
![[泣不成声]](http://lf-web-assets.juejin.cn/obj/juejin-web/xitu_juejin_web/img/jj_emoji_40.348108b.png)
赞了这篇沸点
因为要对上线保持敬畏，所以要有Code Review，在严谨的氛围和规则内，团队内每个人都可以做到服从每个上线环节的要求，形式是可以保证的，但如何保证效果？
就算一个人有着好几年的工作经验，也在团队内待了足够长的时间，这也并不意味着他对所有的经手项目都一清二楚，特别是对于那些有着深厚历史包袱的项目，存在着各种所谓的设计模式和业务上的妥协，不深入了解或者是在上面栽个跟头其实是很难搞明白那些逻辑到底代表着什么
那么在 Code Review 的时候我作为一个 reviewer 就算是想好好给代码把一下关也是有心无力，最多看一下局部的代码逻辑写得对不对，写得好不好，而实际上大部分的 bug 都是业务bug，光看局部代码是无法发现的，局部的代码可能都是对的，但组合在一起形成了整体逻辑可能就是错的
作为众多reviewer中的一员，我可能存在侥幸心理，想着其他人可能会发现的吧？而事实是，可能所有的reviewer都是这么想的，大家都对本次 review负有责任，但大家都不是主要负责的人，旁观者效应导致责任分散，所以就没有真的负责的人了
那么就需要指定一个主要负责的人，肯定不能随便指定，让一个并不熟悉这个项目的人来当负责人，只会平白给这个人造成压力而对真正想达到的结果毫无帮助
所以，谁最熟悉这个项目那么谁就是这个项目的负责人，他是这个项目所有code review的主要负责人
但个人精力是有限的，所以不可能让一个人当多个项目的负责人，那么就需要进行分工，而为了帮助其他人都有各自最熟悉的项目，那么就需要在平时做需求的时候明确分工，在人手平衡的情况下，尽量将所有的A项目分给小张，将所有的B项目分给小王，培养具体个人对具体项目的 Owner 意识
就算一个人有着好几年的工作经验，也在团队内待了足够长的时间，这也并不意味着他对所有的经手项目都一清二楚，特别是对于那些有着深厚历史包袱的项目，存在着各种所谓的设计模式和业务上的妥协，不深入了解或者是在上面栽个跟头其实是很难搞明白那些逻辑到底代表着什么
那么在 Code Review 的时候我作为一个 reviewer 就算是想好好给代码把一下关也是有心无力，最多看一下局部的代码逻辑写得对不对，写得好不好，而实际上大部分的 bug 都是业务bug，光看局部代码是无法发现的，局部的代码可能都是对的，但组合在一起形成了整体逻辑可能就是错的
作为众多reviewer中的一员，我可能存在侥幸心理，想着其他人可能会发现的吧？而事实是，可能所有的reviewer都是这么想的，大家都对本次 review负有责任，但大家都不是主要负责的人，旁观者效应导致责任分散，所以就没有真的负责的人了
那么就需要指定一个主要负责的人，肯定不能随便指定，让一个并不熟悉这个项目的人来当负责人，只会平白给这个人造成压力而对真正想达到的结果毫无帮助
所以，谁最熟悉这个项目那么谁就是这个项目的负责人，他是这个项目所有code review的主要负责人
但个人精力是有限的，所以不可能让一个人当多个项目的负责人，那么就需要进行分工，而为了帮助其他人都有各自最熟悉的项目，那么就需要在平时做需求的时候明确分工，在人手平衡的情况下，尽量将所有的A项目分给小张，将所有的B项目分给小王，培养具体个人对具体项目的 Owner 意识
展开
3
3
赞了这篇文章
赞了这篇文章
赞了这篇文章