请问大家的公司有代码 review 吗

2018-04-19 22:15:52 +08:00
 holy_sin

大家觉得有必要吗

8979 次点击
所在节点    git
71 条回复
projectzoo
2018-04-20 12:35:03 +08:00

有必要
holy_sin
2018-04-20 13:51:02 +08:00
代码风格不统一 貌似是推行 review 机制的一个阻碍
codeyung
2018-04-20 14:59:26 +08:00
当然有了 //
sgissb1
2018-04-20 15:15:09 +08:00
没有 review,也没有必要。但是作为学习有必要看别人的代码。
review 很费时,搞不好就成茶话会或者秀技大会了,同时可能代码写好写坏有关系吗?会因为写的好点老板多发点钱?人家现在只看结果,只关心快速抢占市场等等。
wweir
2018-04-20 15:41:02 +08:00
有必要,主要审两方面东西:
局部的架构设计
代码审查,包括命名风格之类的检查
boywang004
2018-04-20 15:46:26 +08:00
有。
有必要。
jedihy
2018-04-20 15:52:54 +08:00
不 review 就能 checkin 这不是做宝搞吗
pwf
2018-04-20 15:56:50 +08:00
用 crucible 的 cr,还有组内投屏...
lizz666
2018-04-20 16:03:49 +08:00
老板的意思是最短的时间用最快的方法写出来上线给客户拿钱,等模板开发好,谁还管你代码质量好不好,全是复制粘贴。
我要学技术。。。
yuriko
2018-04-20 16:28:32 +08:00
前东家有
用 gerrit
虽然有必要,但是也不能过分
比如合入到 dev 的 master 分支需要三个人加分,感受下……
asj
2018-04-20 16:28:38 +08:00
@Acheron 人,以及人会犯错误的本性也是偏离代码本质的。在人工智能还没发展到代码自己进化那天前,只能暂时依赖不完美的程序员来写代码。
不论 review,注释,unit test,都是为了补救不完美的程序员,避免他们偏离代码本质。
当然如果你认为自己就代表着最纯粹的本质,那当我没说。

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

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

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

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

© 2021 V2EX