这是一个创建于 401 天前的主题，其中的信息可能已经有所发展或是发生改变。
这段时间看了王争的《设计模式之美》的课程。 个人觉得里面提到的谷歌 CodeReview 很有帮助。
- 确实有的时候需要另一双眼睛看自己的代码。
正文中这样总结：
让国内大部分 IT 从业人士认识到 Code Review 的重要性，形成 Code Review 的技术文化，可能还需要一个漫长的时间。不过，我特别希望，你在学完专栏之后，能够意识到 Code Review 的重要性。有朝一日，当你成了领导，有了话语权、影响力之后，能够推动在团队、公司内进行 Code Review ，甚至为 Code Review 在整个国内技术圈中发扬光大贡献一份力量。
大家会有 Code Review 吗？ 程度会到多少呢？范围会针对那些呢？
第 1 条附言 · 2024-01-03 19:51:48 +08:00
大概总结一下
- 没时间 或者 公司不给时间。
- 没必要，只有能过测试、能跑就行。
对于中小公司来说 确实 Code Review 没太多价值，双赢的局面很难成型， 不如进一步压榨员工时间。
115 条回复 • 2024-02-24 23:47:54 +08:00
 |
101
jaycelhz 2024-01-04 13:36:32 +08:00
一个月几百块钱 review 什么啊，写好了给你加钱吗，老板只看页面美不美观和能不能跑
|
 |
102
Lee2019 2024-01-04 13:42:37 +08:00
我们 devops 组要求研发 mr 必须经过两人以上 codereview 并限制了必须 approve 之后才可以合入。
但是实际执行下来，各个项目的 maintainer 都是直接 approve 了，不管什么烂代码都给合入 |
 |
103
cgrass 2024-01-04 13:51:19 +08:00
看公司了，忙起来的时候 ，review 就是个奢侈的活，人和代码有一个能跑起来就可以。大点公司是有 review ，但是 reeview 多了，就会变成应付式的任务
|
 |
104
holouser 2024-01-04 14:00:46 +08:00
重要但不紧急的事情，最后都不会做
|
 |
105
8355 2024-01-04 14:06:56 +08:00
review 不是目的，团队的人才是根本问题，长期写烂代码的人根本是缺乏敬畏之心，对故障本身态度就是无所谓的，现在做管理久了容忍度高了很多，本质上如果功能和健壮度足够，稍微烂一点可以忍受。
|
 |
106
intmax2147483647 2024-01-04 15:00:59 +08:00
看了一圈评论都不想回国了，啥破环境
|
 |
107
amwyyyy 2024-01-04 15:18:05 +08:00
自己 review 自己的算吗，毕竟项目只有我在开发
|
 |
108
evalcony 2024-01-04 15:22:55 +08:00
在有些公司里，领导会看见你有时间做 CR 而认为你工作量不饱和。
|
|
109
evalcony 2024-01-04 15:29:43 +08:00
在公司里做事，做的事情本身要有制度性保障，才有可持续性。没有制度性保障的事情，很快就会被蚕食干净。
这里的制度，既有明面上白纸黑字的制度，也有“老板的意志”但不落成白纸黑字，但最终是归结于“老板的意志”。 老板的个人意志才是什么事情能做、什么事情做不了、什么事情只是说说而已的决定性原因。 所以 CR 就算千好万好，只要老板个人不支持，不在制度上保障 CR 能够实行，那谈 CR 就是扯淡。 |
 |
110
TOUJOURSER 2024-01-04 15:30:24 +08:00
敏捷开发，今天需求明晚上线，测试通过直接生产跑，哪有时间 review
|
 |
111
collen 2024-01-04 15:59:48 +08:00
拉下屎（输出一下情绪）
纯属吃饱了，又要快速迭代，又要代码标准，还要对接多个项目 互联网的核心一直都是能用就行 新手的时候看着一堆新技术，新标准，现在的屎山改了可以效率提升多少，什么代码提升，标准提升，完全是放屁 我要是谷歌的 moeny 有那么高薪资，或者一个项目时间本来 4 天，你不强行压时间让我 3 天做完，你给我 30 天，我无聊了自己能迭代十几个版本，一版比一版优雅 什么 cr ，什么代码检查，什么垃圾玩意，代码就应该写的又乱又差让人看不懂，直到烂到不符合需求自己升级，和历史王朝一样，自己轮回替换，啥也没有，能跑不出错就行了，还检查，检查能让我写一辈子写到 60 岁吗，要有那个规矩我天天检查，完全没屁用，都是大厂那群被开除的垃圾，到其他生态环境破坏环境罢了，我能工资就是市场觉得我有需求，我不能就是我不符合市场，那群出来的一坨带着被开除的茅坑，把其他生态也改成茅坑罢了， 没什么 p 用，能跑就行了， |
 |
112
ooo4 2024-01-04 17:05:10 +08:00
没有流程，所有人都能 merge 到 main 上
|
 |
113
ymz 2024-01-04 17:33:37 +08:00
@astray1988 换下行....
|
Select Language