80 | 开源实战二（下）：从Unix开源开发学习应对大型复杂项目开发
王争
该思维导图由 AI 生成，仅供参考
上两节课，我们分别从代码编写、研发管理的角度，学习了如何应对大型复杂软件开发。在研发管理这一部分，我们又讲到比较重要的几点，它们分别是编码规范、单元测试、持续重构和 Code Review。其中，前三点在专栏的理论部分都有比较详细的讲解，而唯独 Code Review 我们还没有讲过，所以，今天我就借机会和你补充一下这一部分的内容。
很多年前，我跟一个有十几年研发经验的某一线大厂的技术专家聊天，聊天中我提起了 Code Review，他便对 Code Review 一顿否定。他说，Code Review 比较浪费时间，往往会虎头蛇尾，不可能在企业中很好地落地执行。当我又提起，Code Review 在 Google 执行得很好，并且是已经习以为常的开发流程的时候，他竟然说这绝对不可能。
一个技术不错，可以玩转各种架构、框架、中间件的资深 IT 从业者，居然对 Code Review 有如此的偏见，这到底是哪里出了问题呢？我觉得问题主要还是出自认知上。
所以，今天，我并不打算讲关于如何做 Code Review 的方法论，我更希望充当一个 Code Review 布道师的角色，讲一讲为什么要进行 Code Review，Code Review 的价值在哪里，让你重视、认可 Code Review。因为我觉得，只要从认知上接受了 Code Review，对于高智商的 IT 人群来说，搞清楚如何做 Code Review 并不是件难事。而且，Google 也开源了它自己的 Code Review 最佳实践，网上很容易搜到，你完全可以对照着来做。
公开
同步至部落
取消
完成
0/2000
荧光笔
直线
曲线笔记
复制
AI
- 深入了解
- 翻译
- 解释
- 总结
在团队中执行Code Review是一项至关重要的技术实践。本文强调了Code Review的多重作用，包括提高代码质量、摒弃个人英雄主义、提高代码可读性、技术传帮带、保证代码多人熟悉、打造良好的技术氛围、作为技术沟通方式以及提高团队的自律性。作者还就在团队中落地执行Code Review过程中可能遇到的问题提出了解决方案。文章强调了Code Review对于团队合作和技术交流的促进作用，以及在大型复杂项目开发中的重要性。通过丰富的实践经验和观察，作者鼓励读者重视和认可Code Review的作用，以期提高团队的自驱力和凝聚力。
仅可试看部分内容，如需阅读全部内容，请付费购买文章所属专栏
《设计模式之美》，新⼈⾸单¥98
《设计模式之美》，新⼈⾸单¥98
立即购买
© 版权归极客邦科技所有，未经许可不得传播售卖。 页面已增加防盗追踪，如有侵权极客邦将依法追究其法律责任。
登录 后留言
全部留言(42)
- 最新
- 精选
- 小黑能分享下review的checklist么
作者回复: 在前面的34节有个粗略的，详细的我也没有
2020-05-0629 
xindoohttps://github.com/xindoo/eng-practices-cn 我们翻译的谷歌工程实践中文版，老师说的谷歌开源的code review就在这，欢迎查阅。2020-05-061162
马球先生所在公司非常重视code review 需要至少得到两个人的approved 分别是 业务组一个和架构组一个 刚来公司时的第一次代码提交 有20多个comments 现在也能给别人review了2020-05-06542
Richie如何看待： 一、Jeff Dean提交代码前都会编译以及运行他的代码，目的仅仅是为了检验编译器以及链接器是否有问题。 二、编译器从来不会给Jeff Dean警告，相反，Jeff Dean会给编译器警告。2020-05-1120
天华如果老师能把过去cr经验，需要注意的关键点整理出来就更好了2020-05-06112
yu先code review，还是先联调提测呢。 如果先code review，项目时间表是没安排这个时间的，review+改应该需要个1~2天。 如果先联调提测，review出问题后也没法改了。 之前我们code review，是后端写完代码，边跟前端联调边review，因为是一个新的业务项目，review完之后要改的比较多，前端一直反映联调不流畅。2020-05-14110
HeavenCode Review是个好东西,懂得人不少,可是能够执行起来的难上加难,大厂何尝不是,好处能列一大堆,可是咱不是领导,没有带头的能力,自己一个人搞Code Review起不了什么作用,只能各扫门前雪,自己对自己的些重构罢了2020-05-0616
花花大脸猫深有感触，自己再怎么强调code review的重要性，再怎么推动，上层不在意，马虎应对了事，每次一堆checklist需要修改，结果就是走走过场象征性的改几个，软件开发规范形同虚设，代码走查这个事跟公司文化有很大关系，至少说cto或者技术老大对这块有强烈认知才行，不然就是一个虎头蛇尾，吃力不讨好的工作！2020-07-095
小喵喵如果老师能讲解一些实战就更好了。多举例说明这段代码审查出什么问题以及如何修改。2020-05-0635
Jackey团队现在的code review基本流于形式了，想要改善感觉也没什么太好的办法，太多人自由惯了…只能做好自己了2020-05-065
收起评论
