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

[polyang]

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

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

[timedivision]

@gggxxxx 代码风格不定好，拉代码合并的时候，你 2 个 space，他一个 tab，你咋合并？难道自己不用格式化代码的吗

[Orenoid]

我司是有 CR 的，小型创业公司。
虽然谈不上形式化，但这个流程目前基本上已经变成，以 “应该按我的想法来实现” 为目的了。反正我现在也懒得争论了，对方怎么说我就怎么改……说到底还是话语权的问题，总该有个人让步。

[taowen]

code review 只是一种给反馈的手段而已。如果能尽早提出改进意见，没必要等到 code review 的时候来做。cr 时候提一堆意见去返工，浪费也挺大的。

[weer0026]

小公司表示只有在出现性能问题才会去 code review，新员工刚来也会看看，感觉大部分小公司都这样。

[IsaacYoung]

走形式

[mazyi]

垃圾人做辣鸡事，都是垃圾业务怎么来厉害的技术呢？

[Tonni]

我们公司会，Code Review + CI + Unit Testing，就是产品做的一塌糊涂

[saulshao]

@channingcheng
19# 我想表达的愿意其实是写代码的都不喜欢被人 Review，而参与 code review 的人都应该会喜欢。
所以逻辑上应该所有开发都参与 code review，这其实不是看有多少正面效果，而是增强互相之间的理解和沟通。

[hantsy]

@channingcheng 你从来没用过 CI 吗？？？

[hantsy]

@saulshao 这一点没错。更多应该 Peer to Peer Code Review，建立有效的沟通机制，而不是什么上级 Review 下面的人的代码。

[Cstone]

以前有，几个开发大家约个会议室一起认真评审代码，后来来了个领导，说减少工作时间开大会私下指定人 review 就行，代码评审也就成了走过场，再到现在没人 review 了

[leega0]

小公司就如你所说，开发周期根本没有 review 的时间给你，基本匆匆上线，各种改 bug 再测，循环往复。

[dayeye2006199]

我们公司只有三个人，也有 code review，有 CICD，有测试覆盖需求。这是一个企业文化的问题，对这点我们不妥协。

[lights]

刚毕业在传统 IT 行业的时候，没有 CR
后来跳槽做了两年多的互联网，配合 gitlab 做 CR 感觉很棒，偶尔能学到新姿势，也有利于增强代码维护性
再后来转行做游戏，虽然都是用 SVN，但第一家公司有做简陋的 CR，就是看一遍再用 QQ 或者口头和你沟通
现在依旧做游戏，没有 CR，虽然公司小，但毕竟有不同的人做同一个模块，偶尔会觉得比较蛋疼

[hantsy]

@dayeye2006199 这点没错，不用 CI，不写测试，所有 CR 都会是形式。

那种开会形式讨论，什么盯着纯代码提意见的，我也经历过，没任何实质性的作用。

具体 CR 实施一定要有的放矢，那么在 CI 中运行生成的测试报告，质量报告就是那个“的”，连一个最基本的讨论<<基点>>都没有的话，只能说大家公司都是在走过场，说白了，你们根本就不是在 CR 。

[channingcheng]

@saulshao 可惜我们这边都是只有业务 owner 才 code review，大头兵没有资格 code review

[channingcheng]

@hantsy CI 和重构是两个问题呀

[hantsy]

@channingcheng 你们那不是 CR 。只是普通的抽查工作情况（恰好拿代码挑刺说事），跟以前学校检查宿舍卫生一样，只有学工处干的事。

[hantsy]

@channingcheng 不知道怎么跟你解释你的问题。。。

[Perry]

Unit Testing, Code Review, Integration Testing, Accessibility Testing, Stress Testing, Chaos Testing, Load Testing 等等这些不通过估计进不了大公司的 Production 。