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

[iceAD]

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

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

正文中这样总结：

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

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

[jedihy]

什么？难道还有公司不做 code review 的？我见过的所有公司都要求至少一个 Approval 才能 merge PR 。

[Maboroshii]

不做，没空。要抓紧时间让代码跑起来

[emSaVya]

我都把 code review 放在 kpi 里作为工作的一部分了。怎么可能没有。

[tramm]

@jedihy 直接批准

[zylll520]

必须有，在团队中必须推行 code review ！

[yujianwjj]

要真正 code review 的话，至少懂别人的代码的业务。国内很多公司都是一个人干两个人的活，根本没这个精力管别人。

[leehomyhh]

以前有的，我领导会 review 我的代码，现在自己负责开发了，基本就是提交了扫一眼就过了

[wlfeng]

据我所知，小公司基本是没有的

[zackzergzeng]

code review 确实很有帮助，但是就是没时间，各种地方各种人都会催你尽快交付代码，能给你 review 代码的人也在忙着开发他的代码……

[darkengine]

IT 从业人士认识到 Code Review 的重要性有用吗，话事人认识到才有意义

[yigecaiji]

刚毕业没事会做，现在领导说我也不干，谁爱干谁干

[coderzhangsan]

中小型公司基本没有，测试没问题就直接上线了，code review 只是技术角度看方案设计，有一定的积极作用，由于国内中小型公司多数业务驱动，迭代频繁的话，早期的技术设计很可能不太适应，所以国内公司产品把关的重点仍然是测试环节，只要做好测试用例，可以规避 90%以上的较大的生产事故。

[iceAD]

@coderzhangsan 这么看来，确实只适合大公司。

[billzhuang]

我觉得 code review 比 unit test 更重要，

如果非要二选一的话。

[SeaTac]

cr review 很看人
我遇到过极其 picky 一堆说是 nit comment 但是就是不给 approve 的
也遇到看都不看就 approve 结果上了 pipeline 才发现问题一堆
真正好的 cr reviewer 很少见

[kneep]

规则都很完善且复杂，但闭着眼睛打勾的人占多数

[wu67]

中小型没有.
反正我呆过的团队, 前端仔从一组一人两人到一组 8 人, 只有一家公司有, 而且是简单走查, 没有详细 review 到很具体的逻辑, 只是大致看看没有些智障写法就给过了.
还有一些是直接给我把 eslint 关掉的...从 origin pull 下来, 好家伙, 我本地 eslint 一堆红红黄黄的...

[28Sv0ngQfIE7Yloe]

快速迭代的业务的迭代方向往往是核心逻辑的重写，这种业务往往不会有 PR/Merge 的流程

[codingmiao]

从刚进公司的转正报告开始，每次有和大领导汇报的机会我都要插一两页 PPT 说 CR 的事，方案、对比、收益都说过，每次领导都记下了下面就没了，哎。。之前我还在组内交叉 CR ，现在 C 个锤子，等出事了再说。

[28Sv0ngQfIE7Yloe]

@SeaTac

「一堆说是 nit comment 」

请问这个是什么意思 没太看懂