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

[Morii]

小弟背景三年 Java 开发经验，刚刚转到新的组里，写了半个月的 Go

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

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

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

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

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

而不是直接一声不吭的重写.. 朋友们，我该如何调节？

[GM]

一般来说，是问题实在太多，导出是槽点导致无处吐槽，所以干脆直接重写了。

[xuzhzzz]

你没有最佳实践，别人默默给你提供了一个可能是最佳实践的实践，你还想怎么样啊 好好学不就完了吗

[xlsepiphone]

“并且补充了单元测试”

[bianzhifu]

指出问题让你来改都够老大哥重写写几遍了，还能咋办？反复思考被重写部分，找出自己的不足

[Morii]

@GM #1

啊 补充一下，在 review 流程中，已经改了许多问题了，不至于是「无处处理」，所以才有此疑虑，毕竟也不是刚入行的小白，输出的代码是否能满足需求以及有没有质量问题自己还是能察觉到的。

@xuzhzzz #2
是的，作为新手我写出的代码确实真的不是最佳实践，但能通过 review 以及部署验证应该说明其「可用」
但是「默默给你提供了一个可能是最佳实践的实践」真的是一个好的行为吗？或者说你会不打招呼去重写同组新人的代码吗？

[wellsc]

“并且补充了单元测试”

[msg7086]

重写一小时，教你两星期，换你自己在他的位子上的时候你也可能会这么做。
与其去纠结这是哪种滋味，不如把时间花在学习人家的写法上，以后代码质量越来越好，让老大哥再也不需要给你重写才是真的。

[Morii]

@xlsepiphone #3 enmm，补充了我没覆盖到的情况，本身我也是有写单元测试的

[Morii]

@bianzhifu #4
@msg7086 #7
是的，目前还在研究大哥的输出。查漏补缺问题，感谢回复

[zhangshine]

这不挺好的学习机会🤓

[JohnH]

要说确实是应该提前沟通的。我在实际开发中确实遇到这样的情况，但是发现有时沟通成本比直接改写还高。
至于你说的调节，有两个建议，一个是友善地向其提出意图，比如简单的谈话或玩笑中，另一个就是学习采纳提升。

[anyxchachapoly]

带人是很累的，尤其带某领域的新人，带的人越多，消耗精力越大。

虽然我能理解你前辈的做法，不过我觉得沟通、引导还是很必要的，不然你无法获得真实的进步，可以主动请教下，对你自己的能力提升也有帮助。

[Morii]

Hi，感谢各位关注这个帖子，可能是最近的刚转语言压力比较大才选择发帖

关于各位的建议，我每一条都认真看了，谢谢大家

另外，刚才去看代码发现，改写我代码的大哥在提交 Merge Request 时 @了我，

是我一定程度上误会他了，至此，努力提高自己的输出质量才是首位

但是我觉得，有些沟通是有必要的，己所不欲勿施于人，我不能要求其他人按照我的标准去做事

但是最开始确实给我带来了一些困扰以及负面情绪，所以我以后也不会去做这样的操作对待其他新人

最后 再次感谢大家的回复

@anyxchachapoly #12
@JohnH #11
@zhangshine #10
@msg7086 #7
@wellsc #6
@bianzhifu #4
@xlsepiphone #3
@xuzhzzz
@xuzhzzz #2 #2
@GM #1

[echo1937]

工作里讲究一个对事不对人，所以“转 Go 的新人”啊，“吭哧吭哧写完了”这种都是无意义的描述，

“指出问题我来改”并不是天经地义的，工作中重构别人代码或者被别人重构自己的代码是家常便饭，

你作为后进组人员，可以主动去和当事人进行沟通和学习，以开放和主动的心态去面对这件事情。

[Morii]

@echo1937 #14

感谢回复

是的，是我当时想法太狭隘了，导致看待这个事情很片面

[msg7086]

@Morii
> 所以我以后也不会去做这样的操作对待其他新人

想法是美好的，现实是骨感的。当你遇到一个小弟的时候，你可以不厌其烦手把手带他。当你遇到十个新人的时候，你就不想动脑子了。当你遇到一百个新人的时候，心里就只想着，去™的带新人（

（在字幕组里待了 15 年看了无数是是非非的人路过。

[blackboom]

这说明你老大哥不错啊，又没有责怪你，还替你擦屁股。

你感觉自己做了无用功，不要难过，虚心去请教。

[Bunnyranch]

你要想,人家是喜欢自发做事么,给你改起码不是害你,我看这件事整体挺正面的..

[namelosw]

一般来说，开发新功能的时候，如果相关的旧代码会有点影响，一般顺路就重构了，很正常

[redford42]

有的改就偷着乐了