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

[peterswan]

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

[winterfell30]

提代码的时候必须要有该模块的负责人打分，但是感觉大家也不怎么看随手就给过了

[zclHIT]

@peterswan 每次都能把一个月的需求压缩到一周然后搞冲刺，扯皮 2 周，冲刺 1 周，测 1 周

[peterswan]

@crayygy 你们这个流程可以的，避免了哪些乱七八糟的提交和代码。

[poorcai]

我们部门每周一次

[klren0312]

讲真的，做外包本来就很敢，恨不得早点写完，没人愿意看代码

[d18]

大公司也有代码质量很差的，只要最后能跑起来就行，比如鹅厂。

[wobushizhangsan]

今年做了个项目别说 review，连测试都没了。开发，上线，生产问题，补丁。

[wuzhizhan]

今天需求，明天上线

[dangluren]

哇，和楼主很像，目前的公司也不做，之前经历过的都做，虽然有时候有的自尊会受到打击，但感觉是非常有效的方式，由于不做，目前的项目是天天出问题。向领导反馈过，领导貌似自己都不想做

[peterswan]

@dangluren 恩恩 这个就像 12 楼所说的，要大部分人都有这个意识，尤其是技术领导或者团队领导，否则会趋于形式化起不到作用，现在我们也没有这个意识，大多数人更在乎速度，对于质量都想着以后优化，但是以后永远在以后。

[peterswan]

@d18。。。没去过大公司还，不过我觉得大公司推行这个应该更容易吧，资源充足，技术大牛也多，不过这个还是要看领导了

[monkeylyf]

长远来看不做 code review 就是自杀行为。
code review

[monkeylyf]

从中长远来没有 code review 看就是自杀行为。
了解同事的技术水平，了解同事在做什么，肉眼抓 bug，等等等。 因为要急着上线不审代码的先不说技术水平如何， 工程水平肯定有提高空间。

[HuHui]

回武汉后就没做过了。

[hiluluke]

先发 issue,然后技术讨论，然后再发 pr。pr 再 review 一遍。PR 过大，需要拆分。。。

[imdupeng]

没有 codereview，不过老大总喜欢改别人的代码，每次他的意见都很中肯，但是你改了要调试好啊，每次改了出问题，还得我去调试好。
一方面催进度，一方面跑来改我代码找麻烦。。能不能等逻辑跑完了再来优化呀？搞得我反复在原来的代码上做工。

[mhtt]

其实对没有足够的 code review 经验，而去做 code review 是件挺难受的事情

[coolhubery]

我部门是做数据库执行引擎的，C++ 14。CodeReview 到了“变态”的地步。。。
所有 change 都必须尽量的小，必须写 Unit Test，代码覆盖率 100%。一个 change 从开始 push 到最终 merge 最少怎么着也得 5 到 6 轮，当然国外的大牛同事基本上 2/3 个 patch 就 merge 了。
从错别字，注释的准确性，英语表达是否合适，代码可重用，设计角度是否合适，性能是否有影响，Unit Test 是否真得测到了该有的功能，是否应用了 C++ 14 的标准等诸多方面进行考量。。。
这样做的好处是作为非常核心的一个组件，模块非常稳定，新增代码量很大，bug 却非常少。
虽然作为个人来讲，初期自己的修改进展很慢，要被 challenge 很多次，但是一旦熟悉了也就很快了。
不管是那个公司，核心的组件必然要经过极其严格的 Review，否则后期的成本会非常高。

[nicevar]

配置 SonarQube，减少 code review 工作量，同是提升代码质量，还能让人的强迫症变得更严重

[Just1n]

我们每个 checkin 都必须有 code review。
而且，如果修改了不属于自己组的代码，会自动触发一个流程让其他组的人介入进行 review。