这是一个创建于 1016 天前的主题，其中的信息可能已经有所发展或是发生改变。
我有点讨厌 review 同事代码，想听听大家的想法。
 |
1
deplivesb 2022-02-17 14:38:05 +08:00  1
我还可以，既可以学习学习大佬的代码，还能偶尔看看窒息操作。对自己挺好的
|
 |
2
timelessg 2022-02-17 14:38:21 +08:00 via Android
💩🙅，🎂🉑
|
 |
3
Leonard 2022-02-17 14:38:28 +08:00
不喜欢，typo 太多了，看得心烦
|
 |
4
jones2000 2022-02-17 14:38:36 +08:00
可以摸鱼聊天，很好呀。
|
 |
6
wolfie 2022-02-17 15:04:00 +08:00
8 成都是 💩
|
 |
7
wakaka 2022-02-17 15:07:46 +08:00
penta kill
|
 |
8
ppllss 2022-02-17 15:09:36 +08:00 16
帮同事 review
- 这写的什么垃圾？ - 啊？ for 遍历不知道用 steam ？ - 啊？什么勾把？ BeanUtil.copy 不就行了？为什么要 set ？ - 啊？代码这不是上删的代码嘛？ - 啊？什么鬼方法都不知道拆分嘛？ - 啊？什么垃圾代码？谁写的，原来是我一年前写的，那没事了 |
 |
9
golangLover 2022-02-17 15:11:39 +08:00 via Android
喜欢，我喜欢挑剔其他人
|
 |
10
tabris17 2022-02-17 15:12:24 +08:00
有种别人拉的屎自己再吃一遍的感觉
|
 |
12
yazinnnn 2022-02-17 15:14:38 +08:00
自己两年前写的代码已经完全看不懂了，还看别人的。。。
|
 |
13
leeyuzhe 2022-02-17 15:25:07 +08:00
看谁的都是 shit ，包括自己的
|
 |
14
jinliming2 2022-02-17 15:29:47 +08:00 via iPhone
@ppllss 所以 for 循环要登录 steam 么……
stream ？还是 iterator ？ |
 |
15
Pastsong 2022-02-17 15:35:36 +08:00 3
 |
 |
16
littlewing 2022-02-17 15:47:13 +08:00
@Jihua 不，看到自己写的代码都是💩，然后改一遍又一遍，改完发现还是💩，然后看看别人开源的优秀代码净化下心灵
|
 |
18
iyaozhen 2022-02-17 15:53:27 +08:00
喜欢，这样我就能喷开发了 （逃
|
 |
19
SkipToMyLou 2022-02-17 16:15:47 +08:00
只有互相监督、互相 review 代码，大家才能保持良好的编码风格和习惯。
大佬 review 别人代码，可以提前发现问题 规避线上事故 像我这样的彩笔 review 别人代码，就是抱着学习的心态， 学习别人的思路和设计方案。偶尔还能发现大佬的一些不小心的错误 |
 |
20
libook 2022-02-17 16:24:29 +08:00
我们之前是团队 review ，6 人以内一个团队，周期要尽可能短，比如一天一次，这样每个人的代码可能只需要 5-10 分钟就可以 review 完了。
我是觉得从别人代码里找到问题挺有成就感的，同时也可以从各个人的观点中获得更多思路。 |
 |
21
liian2019 2022-02-17 16:28:57 +08:00
给自己一条活路
|
 |
22
lizon 2022-02-17 16:36:11 +08:00 via Android
等你同事的代码上生产了，过了半年他跑路了，你接手了，然后半夜出问题了，领导要你起床马上修复并且上线，还要顺带热修数据，你就会后悔，这么 sb 的写法一眼就能看出问题，当初为什么没有 review 代码提前发现
review 能提前发现处理一些 sb 问题 sb 写法 |
 |
24
MegrezZhu 2022-02-17 16:43:01 +08:00
挺喜欢的
|
 |
25
iovekkk 2022-02-17 16:56:03 +08:00
review 同事代码，那肯定是非常不喜欢，毕竟会浪费自己时间，平时的任务排期也不会专门为这件事去预留时间
不过我们公司，review 代码，一般分两种 日常提交：因为硬性规定所有代码都需要同事 review ，一般都是看看代码规范，以及有没有一眼就能看出来的错误，如果提交后代码依然出了 bug ，review 代码的人是没有任何责任的 重要功能开发完毕：这个要拉个会议，整组人一起 review 的，这个也主要是 review 代码结构和核心逻辑，太细节的东西也不会去看 |
 |
26
wanguorui123 2022-02-17 16:59:48 +08:00
看多了大多数是：💩⛰️
|
 |
27
yuhuan66666 2022-02-17 17:03:32 +08:00
上个需求写完 上面要求 找同部门 review 代码 然后 大家一脸懵逼的看着我 讲了一个半小时没讲完 完全不知道 只能看出读写问题 具体为啥这么写 他们不了解业务完全不知道。
讲得我口干舌燥的，最后变成了 小黄鸭编程，他们是那群鸭子 |
 |
28
vayci 2022-02-17 17:08:19 +08:00 9
看代码量。
少量代码：这写的不好，那可以优化 几十上百个变动：过 |
 |
29
ffw5b7 2022-02-17 17:12:26 +08:00 via Android
@ppllss BeanUtils 有问题，转换类型不一致，预编译 /运行不一定会报错，各个包的不一样。从效率来说 get/set 最快。 嫌麻烦可以使用插件生成。
|
 |
31
RedisMasterNode 2022-02-17 17:34:56 +08:00
喜欢，而且部门应该鼓励做这个事情，例如按 review comment 量来予以奖励
|
 |
32
OldDriver8848 2022-02-17 18:00:50 +08:00
你们竟然还有时间看别人的代码，自己代码写不完，每天干到 11 点
|
 |
33
janus77 2022-02-17 18:37:41 +08:00
一般只看有 bug 的地方，别的地方不说话
|
 |
34
Chad0000 2022-02-17 18:52:51 +08:00
你这个“帮”字用得不太恰当，一般是“迫于”：为了完成任务。
|
 |
35
redford42 2022-02-17 21:49:26 +08:00 2
迫于改造老项目，硬着头皮看
边看还要做笔记：写尼玛代码，2000 行的函数写论文吧你 |
 |
36
dingdong 2022-02-17 22:38:59 +08:00
团队有 review 代码的氛围，那还是相当享受的
|
 |
37
nonwill 2022-02-17 22:57:39 +08:00
最近刚把一高手大虾给杠爆了：
https://github.com/goldendict/goldendict/pull/1447 |
 |
38
YuTengjing 2022-02-18 02:16:52 +08:00
我还是蛮愿意的，不 review 的话，一方面不知道他们负责的那块业务大概是怎么实现的，另一方面都是写一个仓库的代码，它要是写的不规范，自己写那块逻辑也难受，另外还可以促进彼此编码能力提高。
|
|
39
YuTengjing 2022-02-18 02:18:31 +08:00
你不 review 同事代码帮他指出问题，大概率以后还会发生经常碰到同样的问题。
|
 |
40
nanjoyoshino 2022-02-18 02:37:28 +08:00
还行，有 bug 了会说，别的基本不吭声，好的地方算是学到了，不好的地方当没看见了
|
 |
41
dayeye2006199 2022-02-18 04:39:42 +08:00
code review 可以缓慢提升水平均值，同时绝对可以降低方差
|
 |
43
tqccc 2022-02-18 09:20:15 +08:00 via Android
刚入职的时候会读公司一些关键中间件，基础服务的代码和配置，比如消息 /定时任务相关的，了解它们做了什么。。。出问题时候方便排查。。当然也会读到大佬的代码而目瞪口呆
|
 |
44
zw1one 2022-02-18 09:36:42 +08:00
好代码可以看。垃圾代码能跑就行，谁爱看谁看
|
 |
45
SD10 2022-02-18 09:45:04 +08:00 via iPhone
小 PR 可以 一下子几十个 除非是全新的，否则不看
|
 |
46
lawsiki 2022-02-18 10:08:28 +08:00 1
都是业务垃圾代码，没人想看
|
 |
48
pkwenda 2022-02-18 10:34:23 +08:00
作为 owner 不论多少还是要看的
|
 |
49
hejw19970413 2022-02-18 10:39:43 +08:00
看是看 但是不会说
|
 |
50
Renco 2022-02-18 10:45:55 +08:00
这写的什么垃圾代码，看看 git 提交记录是谁写的。噢是我写的。
|
 |
52
tianzhiya 2022-02-18 10:49:50 +08:00
自己几年前写的代码都不想看，更不要说是别人的代码了。。。
|
|
54
nonwill 2022-02-18 11:21:57 +08:00
@RainCats 那位乌克兰大哥中文应该也不错，但氛围都被 pr 主的多国英文给搞😵了......别说，中文都说不好英文也不行的真是多......
|
 |
55
snickers 2022-02-18 11:49:11 +08:00
不算 review 吧 ，更像是看帖子 ，但有问题也会说
|
 |
56
wangyzj 2022-02-18 11:51:26 +08:00
挑重点看
|
 |
57
monospace 2022-02-18 11:55:38 +08:00
我狠起来连自己的代码都很少 review ，还给别人 review ？
|
 |
58
xuanbg 2022-02-18 12:00:11 +08:00
看代码风格。风格恶臭的，3 行都看不下去。风格好的，看起来赏心悦目。
|
 |
59
sorshion 2022-02-18 12:00:35 +08:00
不喜欢，眼不见为净，说了，还怪你，屁事多
|
 |
60
Daiwf 2022-02-18 14:11:13 +08:00
我也不喜欢 review 主要原因
1 、小公司，我们技术都一般通过 review 同事代码也学不到太多东西 2 、业务堆砌，你 review 给出了重构意见。重构之后，千年老代码本来跑的好好的，结果重构完之后出个问题算谁头上？ 总结，没有任何好处。 |
 |
61
Cloutain 2022-02-18 14:35:41 +08:00
对着自己拉的屎还能勉强进行，对着别人拉的屎。。。。
|
 |
62
yuezk 2022-02-18 16:29:11 +08:00
我们公司每一行代码都是要 review 的。我个人喜欢 review 也喜欢被 review 。
Review 代码只对代码不对人，每个人都是从 0 过来的，能帮助同事并提升代码质量是最终目的。当然了，自己能从别人的 review 的意见中学到东西也很开心。 我们还在招人，感兴趣的朋友可以加我聊一下 /t/834762 |
 |
63
ideaspad 2022-02-19 11:07:42 +08:00
眼不见心不烦
|
Select Language