高效代码审查：来自前质疑者的9个建议
转自：http://www.iteye.com/news/30235
理论我知道。代码审查（Code Review）有助于：
- 抓bug
- 保证代码的可读性，可维护性
- 在团队中散播代码的知识
- 让新人适应团队的工作方式
- 让大家接触不同的思路
或者按另一种看法，代码审查就是极大的浪费时间。至少我对代码审查的最初感受就是这样。
当时我是新人，刚毕业，在伦敦一家软件公司开发插件。
随着时间推移，我得提交大量样子都差不多或干脆一样的代码。另一个可怜的家伙（“他是最好的。”我的经理告诉我。好在哪儿啊……）会来Review我的代码。而每次审查之后都会返回不一样的结果。看起来都是全无必要的挑剔又主观的结果。
更糟的是，审查过程的时间，哪怕没有几周，也得有好几天。等代码返回给我的时候，我几乎都记不得那是我写的。这不是那个家伙的错。他肯定也想要个更有经验的开发者，不过他分到了我。他厌倦了处理每个没经验的开发者搞出的那些低级问题，代码审查过程变成了他祛除沮丧情绪的方式。
再加上这些浪费的时间：不同分支的代码同步，上下文切换……我不是代码审查的粉儿，团队其他人也一样。
几年之后，我发现我很同意Jeff Atwood所发表的推特：
经过这几年的时间我开始赞同代码审查，是因为我发现了代码审查并不是坏事，代码审查执行不好才是坏事。伙计，我们就执行的很不好。
我付出了惨痛代价才意识到这一点。当然不是一夜之间的转变。反思之后，代码审查把我从尴尬的，足以破坏构建的代码修改中拯救出来不少回。而自从我在其他地方工作后，我积累了一些和以前不同的、更好的工作方式的经验。这给了我机会，让我能切身体会到我以前曾错过的代码审查的好处。所以现在我认为自己是一个皈依了的质疑者。
为了你能避免这些痛苦，看看我们的视频，读完这些建议，这样就能带给你更有效的代码审查过程。
一、写给所有人的：
1.只审查正确的东西，让工具干别的
你不需要在代码风格和格式问题上与人争论。有大量的工具可以持续地强调这些内容。确保代码正确、易于理解和可维护性强才是重要的。当然了，编码风格和格式也是这些的一部分，只是你更应该让工具去检查。
2.所有人都该做代码审查
一些人比另一些人更擅长审查。更有经验的人可以发现更多的bug，这很重要。不过更重要的是在总体上维持一个针对代码审查的积极态度，也就是说要避免任何“我们和他们”的对抗态度，或者是让代码审查成为某个人的负担。
3.审查所有的代码
没有代码是因为太短或者太简单而不值得审查。当你审查一切，就没有什么会漏掉。另外这样做会成为流程的一部分，成为一种习惯，而不总是事后诸葛。
4.态度积极
这一点对审查者和提交者都很重要。代码审查不是你要拿全A，发动代码神技的时候，也不是你需要摆出防御姿态的时候。带着对建设性批评的积极态度投入进去，你就可以在此过程中收获信任。
二、写给对审查者的：
5.代码审查应该短期高频
你的审查效率在一个小时后开始下降。所以推迟审查共走，然后在一个极限的周期内赶完对谁也没用。在你的一天中留出时间来定期处理，别打乱自己的工作节奏并养成习惯。你的同事会为此而感谢你。等待会让人沮丧。而且当代码还新鲜的保存在他们脑海里时，他们解决问题也会快一些。
6.It’s OK to say “It’s all good” |“都挺好”挺好
别太挑剔了，你不是一定要每次审查都发现问题。
7.使用一个清单
代码审查清单 确保一致性——每个人都了解哪些是重要的，哪些是常见错误。
三、写给代码提交者的
8.保证代码简短
超过200行，审查效率就会显著下降。一旦你超过400行，那基本就没什么意义了。
9.提供上下文
要提供相关的单子，或者规格说明的链接。像Kiln这样的代码审查工具可以提供帮助。应提供简短而有用的提交信息，在代码中留下大量注释。这会帮助审查者，你得到的问题反馈也会少一些。
高效代码审查：来自前质疑者的9个建议相关推荐
- 坚果云开发团队分享高效代码审查经验
代码审查是软件开发中常用的手段,坚果云开发团队最近在"月光博客"上撰文分享了高效代码审查的十个经验.和QA测试相比,代码审查更容易发现和架构以及时序相关等较难发现的问题,还可以帮助 ...
- 中国三大港进入全球最高效港口排名前十，新兴市场物流竞争力中国第一，印度和南亚或继续充当西方买家采购替代市场 | 美通社头条...
美通社消息:全球供应链仍在努力应对2021年的不均衡复苏和物流混乱局面,与以往任何时候相比,物流与供应链的效率.多元化和敏捷性都更加重要.世界银行和标普全球市场财智.亚致力.启迈发布榜单报告,分析全球 ...
- 转：高效代码审查的八条准则和十个经验
代码审查(Code Review)是软件开发中常用的手段,和QA测试相比,它更容易发现和架构以及时序相关等较难发现的问题,还可以帮助团队成员提高编程技能,统一编程风格等. 1. 代码审查要求团队有良好 ...
- 来自前苹果高管Heidi Roizen的经验之谈
摘要:Heidi Roizen曾是T/Maker的联合创始人兼CEO:此后Heidi Roizen加入苹果公司担任开发者关系的高级副总裁.本文她分享了8条箴言,希望能为各位开发者带来启迪,体会生活与工 ...
- 高效代码审查的十个经验
代码审查 摘要:我们在实践中发现,随着开发平台和开发语言的不同,最优的代码审查量有所不同.但是限制每次审查的数量确实非常必要,因为这个过程是高强度的脑力密集型活动.时间一长,代码在审查者眼里只是字母, ...
- 高效代码审查的八条准则和十个经验
代码审查(Code Review)是软件开发中常用的手段,和QA测试相比,它更容易发现和架构以及时序相关等较难发现的问题,还可以帮助团队成员提高编程技能,统一编程风格等. 1. 代码审查要求团队有良好 ...
- 一封来自前ThoughtWorks总监咨询师、致国内程序员的一封信
先打个招呼,这期推文我要介绍一个老朋友--熊节. 推荐他的重要原因是,从"大学肄业"到ThoughtWorks总监咨询师,他倡导的敏捷开发影响了包括我在内的一代程序员. 顺便说一句 ...
- 打脸质疑者！Mobileye市值冲高，公司CEO说出心声
三个多月前, Mobileye以21美元发行价第二次在美股IPO上市.截止上周五,该公司股价升至32.98美元,市值达到264.47亿美元.相比于美股其他自动驾驶公司股价跌跌不休,显然,资本市场更看重 ...
- 图神经网络发Nature子刊！却被爆比普通算法慢104倍，质疑者：灌水新高度？
点击下方卡片,关注"CVer"公众号 AI/CV重磅干货,第一时间送达 点击进入-> CV 微信技术交流群 转载自:机器之心 GNN 是近年来非常火的一个领域.最近,一篇 N ...
最新文章
- python dataframe取某行某列_pandas dataframe.apply() 实现对某一行/列进行处理获得一个新行/新列...
- C语言可以不用，但是不能不会！
- jittor和pytorch生成网络对比之relativistic_gan
- c 语言学习：基本操作，数组，字符串，指针
- Linux使用ffmpeg合并视频
- html表单赋值提交,jQuery自动给表单赋值
- mysql js 命令行登录_mysqlsh 命令行模式与密码保存-爱可生
- Rancher如何按计算资源调度
- 战舰少女服务器不显示,战舰少女进不去 闪退及连接不上解决方法
- 美团团购订单系统优化记
- 新萝卜家园 Ghost XP SP3 电脑城装机专用版 10.5
- python实现查询qq是否在线
- 我是吴恩达：人在美国，刚上知乎，先答个「如何系统学习机器学习」
- SUMPRODUCT函数详解
- 服务器苹果系统安装教程,苹果Mac系统安装Win10正式版两种方法图文教程
- 《玩转Spring全家桶》第70-77节
- mysql5.5图标_MySql5.5安装详细说明
- 2016第19本：至关重要的关系
- 一步一步教你用Python爬虫框架Scrapy下载Pdf文件
- JS中通过指定大小来压缩图片