V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
Morii
V2EX  ›  问与答

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

  •  
  •   Morii · 2021-09-04 14:07:53 +08:00 · 4821 次点击
    这是一个创建于 936 天前的主题,其中的信息可能已经有所发展或是发生改变。
    小弟背景三年 Java 开发经验,刚刚转到新的组里,写了半个月的 Go

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

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

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

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

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

    而不是直接一声不吭的重写.. 朋友们,我该如何调节?
    第 1 条附言  ·  2021-09-05 15:16:03 +08:00
    Hi,感谢各位关注这个帖子,可能是最近的刚转语言压力比较大才选择发帖

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

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

    我当时想法太狭隘了,导致看待这个事情很片面,至此,努力提高自己的输出质量才是首位

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

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

    最后 再次感谢大家的回复
    43 条回复    2021-09-06 14:52:28 +08:00
    GM
        1
    GM  
       2021-09-04 14:12:08 +08:00   ❤️ 16
    一般来说,是问题实在太多,导出是槽点导致无处吐槽,所以干脆直接重写了。
    xuzhzzz
        2
    xuzhzzz  
       2021-09-04 14:14:11 +08:00   ❤️ 8
    你没有最佳实践,别人默默给你提供了一个可能是最佳实践的实践,你还想怎么样啊 好好学不就完了吗
    xlsepiphone
        3
    xlsepiphone  
       2021-09-04 14:16:03 +08:00   ❤️ 30
    “并且补充了单元测试”
    bianzhifu
        4
    bianzhifu  
       2021-09-04 14:16:38 +08:00 via Android   ❤️ 9
    指出问题让你来改都够老大哥重写写几遍了,还能咋办?反复思考被重写部分,找出自己的不足
    Morii
        5
    Morii  
    OP
       2021-09-04 14:18:36 +08:00
    @GM #1

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

    @xuzhzzz #2
    是的,作为新手我写出的代码确实真的不是最佳实践,但能通过 review 以及部署验证应该说明其「可用」
    但是「默默给你提供了一个可能是最佳实践的实践」真的是一个好的行为吗?或者说你会不打招呼去重写同组新人的代码吗?
    wellsc
        6
    wellsc  
       2021-09-04 14:18:49 +08:00
    “并且补充了单元测试”
    msg7086
        7
    msg7086  
       2021-09-04 14:18:57 +08:00   ❤️ 3
    重写一小时,教你两星期,换你自己在他的位子上的时候你也可能会这么做。
    与其去纠结这是哪种滋味,不如把时间花在学习人家的写法上,以后代码质量越来越好,让老大哥再也不需要给你重写才是真的。
    Morii
        8
    Morii  
    OP
       2021-09-04 14:19:15 +08:00
    @xlsepiphone #3 enmm,补充了我没覆盖到的情况,本身我也是有写单元测试的
    Morii
        9
    Morii  
    OP
       2021-09-04 14:20:41 +08:00
    @bianzhifu #4
    @msg7086 #7
    是的,目前还在研究大哥的输出。查漏补缺问题,感谢回复
    zhangshine
        10
    zhangshine  
       2021-09-04 14:21:58 +08:00   ❤️ 1
    这不挺好的学习机会🤓
    JohnH
        11
    JohnH  
       2021-09-04 14:22:24 +08:00   ❤️ 1
    要说确实是应该提前沟通的。我在实际开发中确实遇到这样的情况,但是发现有时沟通成本比直接改写还高。
    至于你说的调节,有两个建议,一个是友善地向其提出意图,比如简单的谈话或玩笑中,另一个就是学习采纳提升。
    anyxchachapoly
        12
    anyxchachapoly  
       2021-09-04 14:24:40 +08:00   ❤️ 4
    带人是很累的,尤其带某领域的新人,带的人越多,消耗精力越大。

    虽然我能理解你前辈的做法,不过我觉得沟通、引导还是很必要的,不然你无法获得真实的进步,可以主动请教下,对你自己的能力提升也有帮助。
    Morii
        13
    Morii  
    OP
       2021-09-04 14:34:32 +08:00   ❤️ 1
    Hi,感谢各位关注这个帖子,可能是最近的刚转语言压力比较大才选择发帖

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

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

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

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

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

    最后 再次感谢大家的回复

    @anyxchachapoly #12
    @JohnH #11
    @zhangshine #10
    @msg7086 #7
    @wellsc #6
    @bianzhifu #4
    @xlsepiphone #3
    @xuzhzzz
    @xuzhzzz #2 #2
    @GM #1
    echo1937
        14
    echo1937  
       2021-09-04 14:34:47 +08:00   ❤️ 3
    工作里讲究一个对事不对人,所以“转 Go 的新人”啊,“吭哧吭哧写完了”这种都是无意义的描述,

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

    你作为后进组人员,可以主动去和当事人进行沟通和学习,以开放和主动的心态去面对这件事情。
    Morii
        15
    Morii  
    OP
       2021-09-04 14:38:26 +08:00
    @echo1937 #14

    感谢回复

    是的,是我当时想法太狭隘了,导致看待这个事情很片面
    msg7086
        16
    msg7086  
       2021-09-04 14:52:41 +08:00
    @Morii
    > 所以我以后也不会去做这样的操作对待其他新人

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

    (在字幕组里待了 15 年看了无数是是非非的人路过。
    blackboom
        17
    blackboom  
       2021-09-04 15:26:59 +08:00   ❤️ 4
    这说明你老大哥不错啊,又没有责怪你,还替你擦屁股。

    你感觉自己做了无用功,不要难过,虚心去请教。
    Bunnyranch
        18
    Bunnyranch  
       2021-09-04 15:36:39 +08:00
    你要想,人家是喜欢自发做事么,给你改起码不是害你,我看这件事整体挺正面的..
    namelosw
        19
    namelosw  
       2021-09-04 15:38:40 +08:00
    一般来说,开发新功能的时候,如果相关的旧代码会有点影响,一般顺路就重构了,很正常
    redford42
        20
    redford42  
       2021-09-04 16:14:10 +08:00
    有的改就偷着乐了
    vindurriel
        21
    vindurriel  
       2021-09-04 16:39:12 +08:00 via iPhone
    找个机会感谢他吧 在你变得比他强之后
    learningman
        22
    learningman  
       2021-09-04 16:43:41 +08:00 via Android
    因为有时间教你,他自己都写完了。。。
    cus
        23
    cus  
       2021-09-04 17:43:58 +08:00 via Android
    ??多好啊 谁写的谁负责呀
    zch693922
        24
    zch693922  
       2021-09-04 18:10:08 +08:00
    同 Java 转 Go,才刚开始... 从 IDEA 换到了 vsCode... 抓瞎中... TAT
    cassyfar
        25
    cassyfar  
       2021-09-04 18:19:42 +08:00
    为什么会认为这种行为有问题呢?即使没 @ 你,我觉得也没任何问题。改良这段代码并不是否定或者贬低写这段代码的人啊。自信点儿吧,没必要在工作里掺入这么多猜忌,心累。
    creanme
        26
    creanme  
       2021-09-04 18:32:19 +08:00
    真好。。。如果每天有人帮我 review 改进代码那真挺棒
    fl2d
        27
    fl2d  
       2021-09-04 18:36:46 +08:00 via iPhone
    我也有过这种情况。一个萌新问我思路,因为确实太简单了不知道怎么讲,于是就写好了给他发去了。
    NonClockworkChen
        28
    NonClockworkChen  
       2021-09-04 18:49:05 +08:00
    人家帮你用最完美的方法擦完屁股,你难道不是应该感谢人家,顺便学一下人家的思路,高情商再请个咖啡吗?
    还在纠结你的代码没了,人家又没拿你工资。。。
    CloudMx
        29
    CloudMx  
       2021-09-04 18:55:06 +08:00
    安安静静苟着学.
    yanzhiling2001
        30
    yanzhiling2001  
       2021-09-04 19:45:49 +08:00
    请他吃饭,表达一下诚意
    icy37785
        31
    icy37785  
       2021-09-04 20:09:25 +08:00 via iPhone
    @zch693922 为什么不从 idea 到 goland
    yaphets666
        32
    yaphets666  
       2021-09-04 23:47:46 +08:00
    学学 28 楼前辈吧,感谢一下再请喝个咖啡是真的.
    akira
        33
    akira  
       2021-09-05 00:14:20 +08:00
    老大哥无偿帮我改代码,那是一种什么精神,
    是我遇到这种情况,赶紧麻溜的看代码怎么改的,看不懂的地方赶紧找老大哥

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

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

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

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




    不至于一无是处就该为了你的面子留着你的辣鸡代码?天下没有这样的道理。你应该庆幸有这样的老大哥帮你改代码,让你获得最直接的范本,有机会从这些细微之处直接获益。
    kingfalse
        41
    kingfalse  
       2021-09-06 08:13:21 +08:00 via Android
    大哥给你擦屁股还不好?非得过来喷你一顿?
    jeeyong
        42
    jeeyong  
       2021-09-06 10:37:49 +08:00
    不要过分关注职场里鸡毛蒜皮的事情.
    大部分同事, 在你离开这家公司后, 你们可能这辈子都不会再见一面.
    大部分同事, 对于你的生老病死并不关心.
    少数关心你的人, 只关心你是否出现意外.
    当你变得更好的时候, 更多的人是嫉妒或者不平衡.

    但这些不是你对其他人冷漠或者漠不关心的理由.
    热情的对待你身边的人.
    适当关心你身边的同事.
    力所能及的帮助他们.
    不要在意一些同事给你穿小鞋或者打你的小报告. 更不要去想怎么报复.
    做一个乐观的人.
    过好自己的生活.
    做你自己.

    欢迎来到熊二的世界, 一切都特别美好.
    LemonK
        43
    LemonK  
       2021-09-06 14:52:28 +08:00
    无非就是他否定你的代码,让你觉得没面子了。
    有些超级自卑敏感的人,你多看他一眼他都觉得你在鄙视他,难道怪你多看他一眼么。
    换位想想,你的老同事 A 给你的新手同事 B 改了代码,你会觉得 A 做错了吗,B 以什么态度应对你觉得合理?
    别太把自己(以及自己的代码)当回事。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   2417 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 37ms · UTC 16:07 · PVG 00:07 · LAX 09:07 · JFK 12:07
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.