转 Go 的新人写了个功能被组里老大哥不打招呼重写了,我该怎么办

2021-09-04 14:07:53 +08:00
 Morii
小弟背景三年 Java 开发经验,刚刚转到新的组里,写了半个月的 Go

起因是刚进组,分了个上报数据的工作,吭哧吭哧写完了

因为是第一次用 Go 从头开发,所以遇到了很多问题,幸亏组里有个老大哥 A,在他 Review 流程帮助下

成功合入主分支部署上线,暂时也没出什么岔子

但是昨天发现另一个老大哥 B 直接把我提交的代码的一部分重写了,并且补充了单元测试

心里有些不是滋味,确实我新手输出的质量不是很高,但是我感觉是不是应该指出的问题我来改

而不是直接一声不吭的重写.. 朋友们,我该如何调节?
4859 次点击
所在节点    问与答
43 条回复
vindurriel
2021-09-04 16:39:12 +08:00
找个机会感谢他吧 在你变得比他强之后
learningman
2021-09-04 16:43:41 +08:00
因为有时间教你,他自己都写完了。。。
cus
2021-09-04 17:43:58 +08:00
??多好啊 谁写的谁负责呀
zch693922
2021-09-04 18:10:08 +08:00
同 Java 转 Go,才刚开始... 从 IDEA 换到了 vsCode... 抓瞎中... TAT
cassyfar
2021-09-04 18:19:42 +08:00
为什么会认为这种行为有问题呢?即使没 @ 你,我觉得也没任何问题。改良这段代码并不是否定或者贬低写这段代码的人啊。自信点儿吧,没必要在工作里掺入这么多猜忌,心累。
creanme
2021-09-04 18:32:19 +08:00
真好。。。如果每天有人帮我 review 改进代码那真挺棒
fl2d
2021-09-04 18:36:46 +08:00
我也有过这种情况。一个萌新问我思路,因为确实太简单了不知道怎么讲,于是就写好了给他发去了。
NonClockworkChen
2021-09-04 18:49:05 +08:00
人家帮你用最完美的方法擦完屁股,你难道不是应该感谢人家,顺便学一下人家的思路,高情商再请个咖啡吗?
还在纠结你的代码没了,人家又没拿你工资。。。
CloudMx
2021-09-04 18:55:06 +08:00
安安静静苟着学.
yanzhiling2001
2021-09-04 19:45:49 +08:00
请他吃饭,表达一下诚意
icy37785
2021-09-04 20:09:25 +08:00
@zch693922 为什么不从 idea 到 goland
yaphets666
2021-09-04 23:47:46 +08:00
学学 28 楼前辈吧,感谢一下再请喝个咖啡是真的.
akira
2021-09-05 00:14:20 +08:00
老大哥无偿帮我改代码,那是一种什么精神,
是我遇到这种情况,赶紧麻溜的看代码怎么改的,看不懂的地方赶紧找老大哥

学到手的才是自己的,面子什么的,那是啥
2i2Re2PLMaDnghL
2021-09-05 01:18:33 +08:00
对工作的态度有很多种。
(在不涉及奖励机制的情况下,)多一事不如少一事『别人乐得去做那我就不用做了』是一种,划分明确『我的工作是我的工作』是另一种。
一般地,在弱责任(事后惩罚轻微)的或者动态责任(事后排查事实来确定责任人)的岗位上做多了会倾向前者,在强责任(事后惩罚严重)或静态责任(事前确定责任人)的岗位上做多了会倾向后者。
这里我在形容词上泛用了强弱、动态静态类型的形容词,但其实跟类型系统或类型系统分类没关系。
同样站门岗,保安和军人就完全不同。

楼主处于后者态度罢了,虽然一般开发岗位通常更偏弱动态责任。
但楼主也有意识到自己的能力其实是不足以喊出『我的工作是我的工作』的,那只能说,学就完事儿了。
BeautifulSoap
2021-09-05 01:29:44 +08:00
我靠有别人直接重构你的代码你还不高兴,对于刚学 go 的人来说简直找不到比这更好的学习机会了。直接对照着别人怎么实现功能还有些单元测试,真的能学到很多。
yousabuk
2021-09-05 02:09:30 +08:00
觉得你写的也太菜了,没必要让你改了,让你来还得和你讲明白,还得等你改,时间紧急,就他自己改了呗。

总结:懒得搭理你
Puteulanus
2021-09-05 02:11:58 +08:00
你看一段代码不爽肯定也是你自己重构了提(只要能过 review ),除非明确发现一段代码会引发 bug,或者某个人目前正在做这一块为了避免和他冲突,不然不可能说你觉得这段写得不好去找别人重写的

我们顶多和对方比较熟的话,改完知会一声为什么要改,指出对方的问题帮助对方进步。不过这种事也是在知道对方有进步的意愿的情况下才会去说,到处指出别人的毛病只会降低自己的风评,老大哥 B 跟你不熟不通知你真的非常正常
BBCCBB
2021-09-05 07:57:31 +08:00
高情商: 请他吃个饭, 让他带带你.
lscho
2021-09-05 10:49:02 +08:00
这不得请他吃个饭?熟了之后他就不好意思直接改了,会告诉你问题所在,然后再改。
xuanbg
2021-09-05 11:09:50 +08:00
@Morii 别人重写的部分,如果看得懂的话,下次学着点就好了。看不懂的话,虚心求教也就是了。下面是不好听的话,楼主不想听就别往下看了。想拉黑也由你。




不至于一无是处就该为了你的面子留着你的辣鸡代码?天下没有这样的道理。你应该庆幸有这样的老大哥帮你改代码,让你获得最直接的范本,有机会从这些细微之处直接获益。

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

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

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

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

© 2021 V2EX