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

[iceAD]

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

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

正文中这样总结：

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

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

大概总结一下

• 没时间 或者 公司不给时间。
• 没必要，只有能过测试、能跑就行。

对于中小公司来说 确实 Code Review 没太多价值，双赢的局面很难成型， 不如进一步压榨员工时间。

[jaycelhz]

一个月几百块钱 review 什么啊，写好了给你加钱吗，老板只看页面美不美观和能不能跑

[Lee2019]

我们 devops 组要求研发 mr 必须经过两人以上 codereview 并限制了必须 approve 之后才可以合入。
但是实际执行下来，各个项目的 maintainer 都是直接 approve 了，不管什么烂代码都给合入

[cgrass]

看公司了，忙起来的时候 ，review 就是个奢侈的活，人和代码有一个能跑起来就可以。大点公司是有 review ，但是 reeview 多了，就会变成应付式的任务

[holouser]

重要但不紧急的事情，最后都不会做

[8355]

review 不是目的，团队的人才是根本问题，长期写烂代码的人根本是缺乏敬畏之心，对故障本身态度就是无所谓的，现在做管理久了容忍度高了很多，本质上如果功能和健壮度足够，稍微烂一点可以忍受。

[intmax2147483647]

看了一圈评论都不想回国了，啥破环境

[amwyyyy]

自己 review 自己的算吗，毕竟项目只有我在开发

[evalcony]

在有些公司里，领导会看见你有时间做 CR 而认为你工作量不饱和。

[evalcony]

在公司里做事，做的事情本身要有制度性保障，才有可持续性。没有制度性保障的事情，很快就会被蚕食干净。
这里的制度，既有明面上白纸黑字的制度，也有“老板的意志”但不落成白纸黑字，但最终是归结于“老板的意志”。
老板的个人意志才是什么事情能做、什么事情做不了、什么事情只是说说而已的决定性原因。
所以 CR 就算千好万好，只要老板个人不支持，不在制度上保障 CR 能够实行，那谈 CR 就是扯淡。

[TOUJOURSER]

敏捷开发，今天需求明晚上线，测试通过直接生产跑，哪有时间 review

[collen]

拉下屎（输出一下情绪）

纯属吃饱了，又要快速迭代，又要代码标准，还要对接多个项目

互联网的核心一直都是能用就行

新手的时候看着一堆新技术，新标准，现在的屎山改了可以效率提升多少，什么代码提升，标准提升，完全是放屁

我要是谷歌的 moeny 有那么高薪资，或者一个项目时间本来 4 天，你不强行压时间让我 3 天做完，你给我 30 天，我无聊了自己能迭代十几个版本，一版比一版优雅

什么 cr ，什么代码检查，什么垃圾玩意，代码就应该写的又乱又差让人看不懂，直到烂到不符合需求自己升级，和历史王朝一样，自己轮回替换，啥也没有，能跑不出错就行了，还检查，检查能让我写一辈子写到 60 岁吗，要有那个规矩我天天检查，完全没屁用，都是大厂那群被开除的垃圾，到其他生态环境破坏环境罢了，我能工资就是市场觉得我有需求，我不能就是我不符合市场，那群出来的一坨带着被开除的茅坑，把其他生态也改成茅坑罢了，

没什么 p 用，能跑就行了，

[linzhe141]

没有流程，所有人都能 merge 到 main 上

[ymz]

@astray1988 换下行....

[Hyvi]

@JounQin 现在公司流动性这么大，可维护、可扩展都是几年后的长远价值，who care

[JounQin]

@Hyvi 你可以不 Care ，但请尊重 Care 的人。