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

[peterswan]

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

[anonymous0user]

小公司比大公司更容易 CodeReview ……

[peterswan]

@anonymous0user 为啥我经历的小公司也没有 CodeReview，感觉有的公司是因为项目赶，有的根本就没有这个意识...

[liprais]

做数据库的，不做 code review 就是找死

[peterswan]

@liprais 数据库应该做 CodeReview 是必须的了，这个弄不好就从开发到跑路了。

[NCry]

我们公司就两个开发。。。

[peterswan]

@NCry。。。那你们俩启不是公司的核心技术了

[NCry]

@peterswan 公司业务比较简单，对于技术要求不高。实际上我感觉我在慢性自杀，但是又找不到方法跳出来，目前只能呆着。

[yuanrenxue]

@NCry 一个写代码，一个补漏 哇咔咔

[zclHIT]

做啊，每个版本快结束的时候改 pmd, checkstyle, codedex, simian 改到想吐，不过代码冲刺的时候 CodeReviewer 就直接点 accept，从来不看。。。

[enenaaa]

现在的这家不但做交叉 review，入库还要审批。 重构优化代码有时候还找不到理由发申请。

[xiaket]

找人过 PR 好烦... - .-

[kaito]

在公司推行 Code Review 的话需要看看周围同事是不是有那种意愿

如果是平级推进的话，看看同事的技术和代码风格是否 ok，可以的话找他讨论一下有没有意愿，有些程序员很无所谓的，觉得写「好代码」太浪费时间了，review 就是 挑刺的过程，自尊心太强的会非常抵触。

推行这些的前提需要比较好的同事关系和工作素质，如果都不错的话可以尝试推进一下

[peterswan]

@zclHIT 我觉得不错，对于代码质量和自己习惯都有提高，还可以对于优化进行讨论，不过代码冲刺的时候我觉得可以预留这部分时间，太急的情况只能例外了。

[hasbug]

上一家我去的时候给我说会，我害怕我的面条式代码会有危险，待了一段，没有的事。。。

[peterswan]

@kaito 恩恩，我觉得可以和同事商量一下，对于一些代码风格问题可以统一定一个规范，Review 过程也是提高的过程。

[peterswan]

@xiaket 等待 pr 的心理就像等待女朋友消息的心理，哈哈。

[peterswan]

@enenaaa 我觉得领导要是有技术热情，应该会喜欢优化的，重构要看付出的代价了。

[peterswan]

@hasbug 为啥，难道审批的人不在乎都给过。。。

[crayygy]

我们 team 里面是必须过的，而且至少有两个人，一个是需要是本模块的，另一个可以是其他的，而且 arch manager 都会查最近的 commit 有没有按照规定来完成

[mikuazusa]

持续集成流程强制必须过 CR