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

[peterswan]

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

[hoichallenger]

我们公司有 Code Review, 不过正打算用 Pair Programming + Mob Programming 取代 Code Review

[monkeylyf]

@RorschachZZZ 讨论的。但都是基于表面或者 bug 本身的讨论，深层次原因应该大家都知道但是不会拿出来说罢了。

[xufeng]

@NCry 简单的事情可以创新做，《并非喊口号》

[peterswan]

@hoichallenger 刚刚了解了 Pair Programming 和 Mob Programming，真是个不错的想法，公司的技术管理也很不错啊。

[peterswan]

@loveCoding 准备搭建一发推荐给团队用。。。

[agagega]

你们在 review 时有些特别特别小的改动也要新增提交吗？有时候想 commit amend 然后 force push，但是又觉得不太好。看来还是 git 太自由了，用 hg 就没有这种强迫症。

[fanqianger]

@coolhubery 你们什么公司。

[peterswan]

@agagega 应该是按照功能细分粒度的，我觉得尽可能小的好。还有远程提交信息不是不应该修改嘛，除非只有自己提交分支。而且我觉得应该过了 Review 再入库。

[nanyang24]

很幸运的是，所在的公司技术 leader 强制了这项规定，合并代码前必须经过任意两个同事的 CodeReview。
我觉得好处是项目代码健壮性得到了一定的保证，对个人的话是学习认识不同风格和 level 的代码，还能熟悉别人做的需求，对整体项目理解有帮助。

[qiaobeier]

@coolhubery sap 核心部门的啊，运气真不错，我前同事分去做业务据说非常废人。。。

[agagega]

@peterswan 嗯，我说的就是自己的 PR 里面

[coolhubery]

@fanqianger SAP...

[coolhubery]

@qiaobeier 不了解产品，SAP 的数据库平台技术还是相当不错的。

[ymj123]

@zclHIT findbugs 不跑？

[fanqianger]

@coolhubery 这厉害的啊，hana 挺牛逼的。你是在德国吗

[liyuhang]

我觉得应该先 Review 一下你的标题

[jiangjz]

不做，所以有的代码维护起来让人想离职😂😂

[jaaazzz]

中等规模公司，做个屁

[micean]

作为管理人员想知道你们有多少时间花在 code review 上，我每天都忙得要死
现在只能了解下面人的代码水平，交流一些好的习惯，从编码规范上要求他们自觉

[dosmlp]

没。。。。。。时。。。。。。间。。。。。。