大部分公司应该都不会 code review 吧？

[polyang]

以我所在的公司为例，平时项目周期已经很紧张了，根本没时间 code review 。

之前有个需求，从开发到联调再到 showcase，只给你一周的时间，这么短的时间，根本不会考虑什么 code review，只要能把功能实现就不错了，等到做完这个需求，还没等你缓过来，下一个需求就立马来了。

[llllboy]

就是个摆设

[DiverRD]

我一个版本改了 70 多个文件 组长看了 直接放弃 review

[Mirage09]

@Perry 我们组一堆 integration test 挂掉的 pipeline...

[USAA]

code review ？ 这玩意不应该自己来弄吗

[flowerains]

有时间才能互相 code review

比如我们公司如果是压着点做需求，能按时按量把需求作完上线就不错了

[hantsy]

@DiverRD 说明你们项目管理问题非常大。一个 PR 一般只包含一个 Feature，不应该太多，正常一般几个文件吧。如果是 Bug，可能可能几行代码，几个字符。

[hantsy]

@llllboy
@flowerains 跟时间没一点关系，这个和企业项目的工程文化有关。没写过测试，CR 就是摆设，没错。

[stirlingx]

@ferock 都 21 世纪了，鹅厂还用 svn

[onec]

纯纯的摆设

[dfzj]

在中型公司，项目分三个等级，只有 P1 等级才会 code review 。也就是是被定义为核心基础依赖的项目。
一般的业务项目，功能测试过了就发布了。

[irytu]

我们公司 code review 很多都喜欢鸡蛋挑骨头 要说意义么 几乎可以忽略不计

[polyang]

@dfzj 感觉这个有道理。

[pangleon]

楼上很多人没理解一点，CODE REVIEW 真正受益的是说人，他在给你介绍自己代码的时候也是重新梳理自己实现逻辑的流程，这个过程很能帮助他提高自己的思维甚至发现 BUG

[akira]

不做 cr 的产品 确实是能跑能上线，但是后面出问题你要花费数以倍记的时间。 单元测试也好，压力测试也好，代码评审也好，这些都是为了降低风险引入的，做了后续出问题的可能性小，不做后续出问题的可能性大。

出来混 迟早是要还的。

[yexiaoxing]

我们合规要求必须有人 review 才能 checkin 然后发 release 。

[lagoon]

我觉得，先是设计好，然后开发好，再然后才是测试好。

Code Review 肯定是好的，也是有用的。

但许多中小公司的情况是，设计没办法好，开发没办法好，然后领导指望抓测试，指望 Code Review，就能稳住质量。
本末倒置，只是安慰剂。

能不能设计好，取决于领导。能不能开发好，取决于领导。
领导不从自己身上找原因，寄希望于给开发人员找茬来稳住质量。



因此，大部分公司，不会 Code Review，或者只能进行走形式的 Code Review 。

[hugo54]

我所在的产品线的后端，都是 QA 来 CR 。 凡是非自主测试的代码，保准给你一行行 review 。

[vagranth]

我司不光有 review，而且还要求写测试 case 。
我曾经有一个大 patch 被要求拆分成 5 个 patch 慢慢 review 完的经历，要不是那个功能架构做的还可以，拆都拆不开。

[msg7086]

我们是强制 code review，除了其他开发以外，team leader 也要 review 一次，没问题了才能合并。

[Leee]

@dayeye2006199 你们在广州吗？还要人吗😂