大家的工作中会有 Code Review 吗？

[iceAD]

这段时间看了王争的《设计模式之美》的课程。 个人觉得里面提到的谷歌 CodeReview 很有帮助。

• 确实有的时候需要另一双眼睛看自己的代码。

正文中这样总结：

让国内大部分 IT 从业人士认识到 Code Review 的重要性，形成 Code Review 的技术文化，可能还需要一个漫长的时间。不过，我特别希望，你在学完专栏之后，能够意识到 Code Review 的重要性。有朝一日，当你成了领导，有了话语权、影响力之后，能够推动在团队、公司内进行 Code Review ，甚至为 Code Review 在整个国内技术圈中发扬光大贡献一份力量。

大家会有 Code Review 吗？ 程度会到多少呢？范围会针对那些呢？

[BUHeF254Lpd1MH06]

有 假的 CR

[SeaTac]

@Morii
一般每一个 cr comment 都是需要 resolve 的（给出解释或者改掉）
nit comment 就是可改可不改

[777777]

所以别用国产，都是垃圾，没人 cr 的

[Rache1]

我们有，但是，我们的主要目的是减少低级错误，不怎么关注代码质量 🙇

[mantouboji]

Code Review 的质量度量：单位时间内听到的脏话次数。

[paopjian]

一个前端服务五个后端,改啥代码都是直接梭,爽翻

[hongweijie8]

小公司基本没有，review 最起码要懂业务，有的公司一个项目就一个前端一个后端，怎么 review

[Quarter]

现在项目太屎人手不够，一般人都是外包，写的很随便，没时间搞 code review ，只有 shit create

[CaptainD]

小公司，完全没有

[liprais]

水平都差不多的话能 review 出来个啥,就挑点回字有几个写法这种问题

[zxf4399]

有的

[astray1988]

来给个 DP ， 某电商跟云的北美大公司，无论是做 design 还是 code ， 都有相应的 review 机制。。如果是根据业务需求来添加小的 feature/功能，需要做 design ，然后组内跟相关组都有相应的 design review ，需要组里比较 senior engineer 来 sign-off ，如果是大的业务改动（例如开一个新的 business ，对应的 org 内部有 principle engineer 的 samurai design reviews 。。所谓的 code review ，每个组内部都有相应的 code review 机制，同时 org 内部标准也会对 code review 做一定的强制要求，例如提交的 CR/PR 的 Unit Test 必须 90%以上，还要通过各种所谓的 code quality test cases...具体到各小组/或者具体的 service ，code quality 也不一样，比较 legacy 的 service 会整体对 code review 松一些， 比较新的 service 会相对高些，例如每次做个新的 feature 还要写 integration tests 来保证 full CD

[wojiaoxiaojuan]

待过两家外企，都有 code review 流程。国内的小公司里，看同事 merge 就跟玩儿似的

[crazyTanuki]

没有 cr ，因为没有统一培训，代码风格各异，能跑就行

[OneMan]

code review 对当好螺丝钉有用，类似茴香豆有几种写法的检查。
对提高你的工资、职级，并没有鸟用。

[burymme11]

没时间 cr ，都是写业务，再大的公司也是 CRUD 占大头吧，中小公司就更是了，那怕每次都 cr ，时间久了，项目一样成屎山，因为需求一直迭代一直变的，业务越来越绕越繁琐，cr 的成本指数升高，而为了能过 cr ，开发写代码的成本也越来越高，死循环了。所以我司后来全面拥抱单元测试。
我觉的核心问题要解决垃圾的产品设计和业务架构，只要这两个合理，cr 真的就啥可过的。标题中正文部分，我个人觉的不太符合大清国情。

[ren2881971]

没卵用， 公司几十款产品，评审的人能清楚所有产品的业务逻辑么。。。 如果不清楚那 review 个毛线，只能为了提高存在感挑毛病。

[dsggnbsp]

我们是草台班子

[urlpha]

油管有个很有名的视频可以搜来看看。
“你看了，但是你对提交的具体内容并不关心。”

[flyv2x]

国内小公司基本不重视技术文档和 Code Review 这些基本开发流程