大家所在的公司都做 CodeReview 么？

[peterswan]

我现在所经历的公司，还没有做 CodeReview 的，也许公司太小的原因，但是我觉得 CodeReview 是非常有效的方式，能提高代码可控性和质量，还能促进技术交流。不知道你们经历的公司有 CodeReview 么，有的话可以分享一下感受，和如何进行 CodeReview 比较好。

[peterswan]

@monkeylyf 同意啊，见了一些项目因为技术债实在开发不下去的项目。。。

[czk1997]

能跑就行，出 Bug 再说……测试全靠脸（能跑就行），服务器部署全随缘..

[peterswan]

@hiluluke 这个工作流程有点像 github，你们用的啥平台，github ？？ 我也想用这个流程试试

[peterswan]

@imdupeng 下次不要让他直接该代码，让他提 pr 或者开个分支，做好了之后你来 Review，有问题让他改好了在合并，不过这好像是你的老大。。。

[peterswan]

@coolhubery 这个流程有很多值得学习的地方啊，能提供这样严格的 Review 肯定公司也特别棒了，谢谢分享

[lsyAndroid]

没有，全靠自觉，本身就是赶项目的，没有时间搞

[peterswan]

@nicevar 用自动化工具是会加快这个过程，不过我感觉每次 pr 应该粒度尽可能的小，Code Review 压力就不会那么大了，而且可控性更高，如果太大的 pr，直接让他拆分后再 Review。

[peterswan]

@Just1n 恩恩 强制的 Review 才能保证代码可控，不过我觉得为什么要让其他组的人来 review 呢，毕竟其他组的肯定不熟悉业务，找同组的人会不会更节省时间。

[66beta]

本组的人，没人负责一个模块，也只能 review 语言本身了，无法察觉业务漏洞

[peterswan]

@czk1997
@IsyAndroid
我觉得这样做会加大技术债，现在赶出来的时间以后会加倍付出对于项目来说。当然要看老大有没有这个意识。。。

[A555]

形式大于内容

[peterswan]

@mhtt 刚开始要是没有经验，可以大家一起开个会学习一下 Review 嘛，以后应用到项目中就会越来越熟练了。

[peterswan]

@66beta 对于语言和代码优化相关的 Review 也会提升代码质量，更关键能在 Review 过程中提升自己，既可以挑别人代码的不足也可以学习别人代码的优点。

[hiluluke]

@peterswan 就是用的 github 企业版。

[natscat]

做啊 代码功能 实现方式 都做

[monkeylyf]

@peterswan 我亲身经历过没有代码审核，快糙猛急着上线，等代码库到一定体量一个没更新一个版本都是提心吊胆，开发人员身心俱疲，最后项目烂了，公司黄了。
话说回来，我现在所待的地方，是有代码审核的，同样老板喜欢催上线，大佬们都相互给个 LGTM 都不仔细看。现在代码体量已经很大了，基本两天一个中性 bug 爆炸，一周一个大 bug 核爆。
流程是用来约束自觉性的，但是完全没有自觉性，有再多流程也没用。

[loveCoding]

组内在使用 gerrit 做强制 code review 还算比较好用

[peterswan]

@monkeylyf 恩恩，这个要看有没有一个厉害的技术管理去领导做这个 Review 事情，对 Review 质量进行把关，如果只是应付形式简直就是浪费时间了，还有就是对于 Code Review 可以有一定的奖励制度调用积极性。

[RorschachZZZ]

@monkeylyf 中等或者严重的 bug，你们那不复盘下、讨论下吗

[zhaoxinz]

在就职过的公司中，只有印象笔记有融入在日常工作中的标准流程 Codereview，在 commit 之后，通过命令行指定一个人为你 Codereview