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

359 天前
 iceAD

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

正文中这样总结:

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

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

14249 次点击
所在节点    程序员
115 条回复
zogwosh
358 天前
cr 又不算产出谁干?我连 lint 都要关就为了早点搞完早点下班
28Sv0ngQfIE7Yloe
358 天前
@SeaTac #22

明白了 谢谢
smirkcat
358 天前
自己开发,自己看,bug 自己处理,有没有都一样
SoviaPhilo
358 天前
@burymme11 解决不了的, 产品设计和业务架构是产品部门的护城河,怎么可能容你置喙

等业务功能一多,很多业务细节完全没人控的住的

比起 code review , 还不如有意识地推动领域驱动设计, 减少心智成本
Shanee
358 天前
一直有 code review ,一般提交 review 之前先要通过工具保证一些基本的代码质量例如 sonar ,checkmax ,跑完 pipeline ,sonar 的覆盖率到 80%以上,checkmarx 没有问题,才能提 PR 给人去 review ,不同的 reviewer 会 review 粒度和力度都不同,会反复 comment-》 challenge-》 resolved
54xavier
358 天前
换了三家公司,重来没有 reviewer 过,不过自己的代码习惯还好,每次看别人的代码都是地铁老人手机脸
nhhjhwiner
358 天前
业务需求没,技术需求有
fqy12300
358 天前
我一般都是自己写代码,自己测试,自己提交,自己合并,自己部署,自己发布。我还是挺相信我自己的。
Sigrdirfa
358 天前
有点还是好的,至少能让大家知道你写了屎山,至少在 cr 变质之前可以有效防止大家写屎山。
iovekkk
358 天前
不仅有,还接入了 CHATGPT4 ,AI Review 给出的内容,又详细又准确
MicroSharpAnt
358 天前
上家公司有,而且觉得 leader 审核代码对我帮助很大。现在在一家小公司,从没 review 代码的,而且 git 大家都随便 push ,版本管理也很乱。不接手某些人的功能还好,一接手就觉得头疼,屎山就一点点堆积起来了
NoString
358 天前
我一个人手上十几个项目,哪来人 review 。基本都处于随地大小便的状态
huyomi
358 天前
上家公司有,被大佬 review 过代码提升会很大
twogoods
358 天前
review 确实有这种感觉,其实真没必要的东西硬是要挑点出来,直接点不会显得你水平低
supuwoerc
358 天前
@iovekkk 不合规吧,我们搞过机器人,然后不然用,会泄漏代码
pkoukk
358 天前
会看新来的应届生的,其它基本不看
magicls
358 天前
cr 是基操吧,而且 google 还有 presubmit-verify 机制
darkengine
358 天前
我左手一个 git merge ,右手一个 git push --force ,丝滑
somebody1
358 天前
同一家公司干了六年了,code review 的次数没超过三次。

国内头部 to b 网络安全企业。
kingjpa
358 天前
我代码都是 提交自动 webhook 到生产环境的,别说审查代码了, 我连 commit 都懒得写,每次回滚都是看时间

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1005441

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX