发起个讨论，你们公司有 code review 吗？

[zzzzzy]

code review 好处很多，可以规范代码、传递知识和保证代码质量等，但是因为项目进度和其他因素，不一定保证 review 被有效执行。请教各路大神都是怎么做的？有没有啥套路、工具、方法之类的。

[kkzxak47]

我们组加上负责人才 3 个人，名存实亡

[sc3263]

之前的项目组有。
svn 上新建 work 目录，上传原始代码，上传正式代码
->更新 redmine 上对应的案件，通知相关人员 review
->review 代码，有啥问题提一下，有啥 bug 修复一下，都在 redmine 上更新
->结束了上传到 trunk 目录，结束 redmine 上的案件
代码质量确实高，不管是代码风格、注释，还是业务逻辑。写的人可能只考虑当前模块当前需求， review 的人会考虑到系统整体。毕竟 reviewer 对整个系统的了解，比我们不知道高到哪里去了。至于耗时什么的，我是觉得，整套流程跑起来之后，其实也不会多花很多时间，毕竟这边在 review 的时候，那边可以继续做别的需求的开发。

[qqzj]

有，即使我改一行代码都有两个人来 review ，你得解释清楚为什么这么改，他们看得非常认真。有时还不止看改动，还把前面的代码也看下。顺便说下，他们都是工作 2 ， 30 年的人，有时我都不好意思。

[czheo]

有，两人 review 之后才能 merge 到 master branch

[yangxiongwei]

用 phabricator ，强制 review

[easing]

必须要有啊， code review 对质量的保证还是很有效果的

[SourceMan]

看造化吧，新功能都不够时间做

[shenxian]

木有

[twoyuan]

@zzzzzy 小鲜肉表示刚开始老员工 review 比较好，熟悉后谁有时间谁 review 吧

[kimmykuang]

gitlab 会发 commit 邮件，然后我偶尔会 review 下我们组的提交；每个项目版本提测前会组内 review

[broadliyn]

每次看到这类问题，我就想到那张图。
要 copy － paste ， good design 又不会给你加工资。
人生没有时间给你重构。

review 有个蛋用，功能都赶不及。

逃

[young]

都是形式

review 个蛋蛋

[zzzvvvxxxd]

要的，你可以前期自己 review
后期让实现功能的小鲜肉串讲一下，介绍一下代码的组织和设计的思路，你听一听就好了。

[Ixizi]

每次 commit 必须 review 。

所有的队友必须一起 review 。

[Tonni]

我们公司用的 GitHub Enterprise ，没个代码改动都要发送一个 Pull Request ，没个 Pull Request 必须要至少两个人 review 并且给予 👍 后才能 merge 到 master 里面去。

[daochouwangu]

用 facebook 的 phabricator 来做 code review

[messense]

Bitbucket PR + 自行开发的 CI 自动化测试、 Code Lint 自动化代码风格检查

[lovedebug]

每次提交修改前都要 review 吧~~
公司用的 perforce, 和 Idea 以及 JIRA 结合 review code 还不错

[proudzhu]

@messense 道理我都懂，流程这么正规，为啥文件名这么随便。。。

[flydogs]

有 review 率，有指摘率