【博文推荐】如何提高团队代码质量——代码审查的实践

为什么需要代码审查

创新互联建站是由多位在大型网络公司、广告设计公司的优秀设计人员和策划人员组成的一个具有丰富经验的团队，其中包括网站策划、网页美工、网站程序员、网页设计师、平面广告设计师、网络营销人员及形象策划。承接：成都网站建设、成都做网站、网站改版、网页设计制作、网站建设与维护、网络推广、数据库开发,以高性价比制作企业网站、行业门户平台等全方位的服务。

最近看了一些文章，发现敏捷开发的一些理念越来越多的团队在实践，也觉得敏捷不再像最早提出的时候那么虚，有很多体现这个理念的工具涌现。其中，“如何提高代码质量”的讨论一直很多，敏捷开发中也有好多种提案，最广为人知、但也最不靠谱的应该就是结对编程了，只要没被敏捷洗脑的人都清楚知道这个基本没有实际可操作性，然而这个做法体现的观点是多个人互相监督可以把事情做的更好，这反而是完全没有问题的。所以还有一种方式就是代码审查了，把两人同时写代码改成在不同的时间上一个人写、另外一个人看。这个实际开发中是完全可以做到的，只是要留有审查的时间即可。

复查团队成员的代码自己一直也是无意识的在做，经常去看git log，但是这个方式效率真的很低，也没有严格的规定，所以做的也比较随意。恰巧最近看到一个论调，“越牛逼的团队，对于代码审查的态度越严谨”，顿时引发共鸣，长久以来在心里一直有一种这样去检查代码的方式是不行的感觉，但是一直也没找到合适的方式，而这次再联想到代码审查感觉如获至宝，怎么一直都把它给忘了呢？！

如果你对代码审查能带来什么好处有疑问的话，请仔细阅读Phabricator官网的这篇文章。

代码审查的方式

代码审查主要有两种方式：

1. pre-push：在提交合并代码之前，先进行审查，通过和才能合并。这是一种非常严格的审查方式，可以确保每个发布的代码都是已经被审查过的。这种放到在github上维护的开源项目极其合适，代码的所有者可以确保代码是在自己的控制范围。

2. post-push：代码提交后，再审查之前的代码。这是非常宽松的审查方式，审查的效果肯定是打折扣的，但是好处是可以忽略一些不必要的审查以节约时间。其实在国内这种没有太多工程师文化的地方，这种方式是比较好在早期推行的。

代码审查的工具

这个事情在团队中实行的话，是一定需要有个工具的，相关的工具有很多，审查方式也各有偏重。这里工具主要是解决了这几个问题：

1. 有一个更为直观的界面查看diff。

2. 可以基于工具进行简单的标记和通知，直接把标记写在代码里更利于沟通。

3. 可以知道哪些提交时已经被谁审查过了，方便审查的协作。

之前在sf写过一篇问答可以参考。这里再例举一些，供参考选择。

1. Gerrit：google的产品，名气很大，但是这个东西设计理念比较陈旧，据说也没有什么维护了，不推荐。

2. github pull request：这个当然很好，典型的pre-push方式，但是个人用也没太多协同的事情，团队用又觉得贵。其实感觉用bitbucket会经济实用些。

3. phabricator：facebook内部使用并开源出来的工具，功能超级强大，但相对的就是非常复杂，界面设计非常欧美的风格，运行速度也有点慢。东西还是很牛逼的，看你是不是喜欢了。

4. gitlab：如果是自己搭建的git server，这个是不错的选择，相当于自己弄了个github，就是配置环境会比较多工作量。

5. upsource：JetBrains的产品，只有post-push的方式，但是从安装、界面、到使用都是挺不错的，唯一问题就是10个人以上要收费，而且还很贵。

我们从中的获益

选用哪种方式，我觉得因团队文化、项目背景、效果预期而定,我们最后暂时选用的是upsource，目标是先在团队中把代码审查施行起来。

目前来看效果还可以。有了工具之后大家互相做代码审查也方便很多，心理抗拒性也没那么强。其实只要进度不那么催，研发人员还是比较愿意去做这种事情的。审查过程中目前发现了一些代码中的问题，但是现在还不多，我想只要能在出现bug之前能解决掉一些问题，就已经有很大价值了。

网页标题：【博文推荐】如何提高团队代码质量——代码审查的实践
标题URL：http://www.shufengxianlan.com/qtweb/news20/290270.html

网站建设、网络推广公司-创新互联，是专注品牌与效果的网站制作，网络营销seo公司；服务项目有等

广告

声明：本网站发布的内容（图片、视频和文字）以用户投稿、用户转载内容为主，如果涉及侵权请尽快告知，我们将会在第一时间删除。文章观点不代表本网站立场，如需处理请联系客服。电话：028-86922220；邮箱：631063699@qq.com。内容未经允许不得转载，或转载时需注明来源： 创新互联