首页   注册   登录
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
Coding
V2EX  ›  职场话题

出差回来发现代码被重构了!

  •  
  •   dayoushen · 352 天前 · 10125 次点击
    这是一个创建于 352 天前的主题,其中的信息可能已经有所发展或是发生改变。

    题主属于两个产品线,出差一周搞 A 产品线事情回来,一个星期快过去了,发现 B 产品线提交了一个新版本,本着好奇的心理看看更新了什么,最后自己完成的功能被重构了,心理 MMP。首先被重构了,还是很开心,终于少了一个包袱,不用调高校提供的 SDK,他们每次更新版本都不如旧版本好用,总是缺东西;其次发现组长在毫无征兆的情况下,被代码重构了,才意思到有很大的信任危机。问问大伙有没有相关经验,如何调整心态进入下一个产品线。

    第 1 条附言  ·  350 天前
    发现没有表达清楚,这里的重构不是指小规模的模块重写,而是整个程序的重写。

    测试是个妹子,听说测起来很痛苦,因为只给了一个脚本没有配套的工具,之前我是提供了整套的测试工具,包括压力测试工具;

    比较了新旧代码,从逻辑上基本一致,新代码用了 qt 和一些新的组件,旧代码是用原生 linux 的 c++构建。
    从整体设计上看新代码确实是比较先进,但上手会难一些,可以学习一下。


    尴尬的是,下个月开始要继续和组长奋斗在另外一个新的产品线,一个月后再来理解一下现在的心情。
    72 回复  |  直到 2018-12-29 18:59:51 +08:00
        1
    jinhan13789991   352 天前
    准备简历吧
    另外多看看设计模式
        2
    eamon666   352 天前
    总结下自己的不足,然后复习复习 明年换一个好公司吧。
        3
    ssynhtn   352 天前   ♥ 7
    我就是那种一声不吭就重构别人的代码的人, 一般当我这样做的时候, 说明:
    1, 我对这个项目还有热情,
    2, 分配给我的任务太少, 闲的,
    3, 实在是看不上原来的代码, 感觉写的像屎
    以上三个条件缺一不可
        4
    Phariel   352 天前 via iPhone
    谁重构谁背锅 不是挺好
        5
    jinhan13789991   352 天前
    上面其实是玩笑话,我之前一直想重构代码,领导还不让呢
        6
    gzlock   352 天前 via Android
    只要公司还在就不是大问题
    有人出差途中公司倒了的
        7
    imn1   352 天前
    两个词没能理解具体含义:组长、自己完成的功能

    首先这些功能不是整个产品线吧?只是部分 class 或模块?
    其次,组长不是项目经理吧?被下属私改不知照是问题,但如果上面还有项目经理,是他下令重构的不知照(原创建人外出)很正常
        8
    hhhsuan   352 天前   ♥ 1
    我看了不爽的代码都是立刻动手重构,才不管是谁写的,老子没心情在那些垃圾代码上修修补补。
        9
    bobuick   352 天前
    这么玻璃心。要是定个流程规定重构前都要征得同意,估计又要被骂傻逼玩意啥都流程。
    代码怎么样没点数么
    要不代码烂的不行了,要么需求变太多了不得不改了。
        10
    wangxiaoaer   352 天前
    1 重构的代码跟你之前比起来有没有进步?

    2 代码不是你的个人资产,是公司的,有权限的开发人员都可以改“你的”代码,不要玻璃心。
        11
    libook   352 天前   ♥ 2
    别人没经过我的允许改我的代码,给我的感觉像是做销售别人抢了我的客户是一样的,这是职业道德的问题。

    不商量就改别人的代码的心理是什么样的呢?
    1. 我自认为我了解这个人做的业务,我觉得我可以改他的代码实现我的目的。(我这样试过,结果证明我对他的业务不够了解,修改完的代码造成了严重的问题,以后不问清楚再也不敢改了。)
    2. 我觉得这个人是 SB,我表现我的能力的时候到了。(抢饭碗?)
    3. 这个人联系不上,问题很紧急必须马上解决,没有任何其他人能解决这个问题,只能我来了,后果责任我来承担,一旦联系上这个人马上跟他说具体情况,以求理解。(个人能接受的合理理由。)

    我想说,一个团队首先的互相尊重,每个人都认为每个人都是团队不可缺少的一份子,每个人也都能参与到团队所有工作中,这才是合作关系的体现;团队内人与人之间如果不是合作关系,那么差不多就只能是竞争关系或没关系了。
        12
    misaka19000   352 天前 via Android   ♥ 1
    要是我开心还来不及,谁重构,谁负责,出了问题谁背锅😀
        13
    libook   352 天前   ♥ 2
    不管出于什么原因,不告知就修改其实就是在否定原开发人员的存在意义,遇到这样的事情郁闷的人也通常是因为自己失去了存在感。代码质量差也得解决这个人的问题,要么提出意见商量并改正,要么除名,而不是只解决代码的问题,不然这个人的代码质量永远都是差的,永远都事后帮他解决质量问题显然得不偿失。团队成员的成长也是很重要的。
        14
    dayoushen   352 天前
    @imn1 产品线 B 没有产品经理,部门经理提出想法组长带着两个人干;程序是一个嵌入式 AI 板子上的 http 小后台,分成 3 个模块,被重构的是 http 服务。据了解部门经理没有重构的意思。1 月份这个产品线 B 估计要被放弃掉,人员会被抽离到另外一个产品线,所以在这个节骨眼上被重构也是让我很惊讶。
        15
    pusidun   352 天前
    代码还分你的我的?
    在我电脑上就是我的 XD
        16
    orangeade   352 天前 via Android
    我的项目被重构好几个了,还挺开心的
        17
    imn1   352 天前
    @dayoushen
    那就当下面的人在练习,宽容一下
    跟他交待一两句,大意是将来去别的公司高就,这样做会吃亏的
        18
    dayoushen   352 天前   ♥ 2
    谢谢大家的关心和批评,我是从传统行业出来的,对代码的理解上是可读性胜过技术先进性,可能没被他们认可,不过至少代码在线上已近跑了 3-4 个月了,所以被重构了还是感觉有点失落,虽然这样也能少了一件操心的事情。
    不想那么多了,还是好好提高自己。
        19
    k9982874   352 天前
    我会问一下原因,然后对比一下自己的代码,吸收对方的优点。
        20
    myhot21   352 天前
    最喜欢别人重构我的代码,然后不再需要我维护那部分代码。
        21
    dayoushen   352 天前
    @imn1 谢谢,看得开。
        22
    laike9m   352 天前 via Android
    惊了,重构代码居然不需要之前写的人 review。。。这种公司还是离职吧
        23
    GDC   352 天前 via iPhone
    @libook

    如果写得太垃圾 确实没有存在的意义,同事一场还撕破脸皮跟你说你写得太垃圾?多次被人重构自己心里没点数?如 3 楼说的,如果不是太垃圾,是不会一声不吭去重构的,这是缺一不可的条件。

    你说的那种尊重 沟通 告知,是建立在还认为你的代码由自己改改就能更好的基础上,这种情况没谁闲着还去动别人代码的。

    别扯什么团队每个人都缺一不可,不过都是个挨踢民工罢了,对公司来说更不会缺一不可,所以不要自欺欺人,讲什么职场道德,什么抢你客户,无论什么行业什么岗位,提升自己的能力水平才是王道
        24
    lzhCoooder   352 天前
    换了一个部门,上任领导被踢出开发队伍,新任领导让我把上任留下的所以代码重构...花了两个月,难免不让人多想为啥突然给他换岗位
        25
    Maboroshii   352 天前 via Android
    我最近也重构了一份代码。没办法啊,让我在之前的上面改实在是下不去手。
        26
    LewisW   352 天前 via iPhone
    @ssynhtn 一声不吭就重构?迟早被测试打死
        27
    reus   352 天前   ♥ 2
    @libook 你们写的代码都属于公司的代码,不存在什么“你的代码”
        28
    LowBi   352 天前
    如果重构的代码比原来的好,那是支持的了,对于我这种菜鸡,多参考别人写代码的风格习惯及思维是最好不过了
        29
    Macolor21   352 天前
    想起我之前的代码
    if(xxxx) throw ....
        30
    Macolor21   352 天前
    想起我之前的代码
    if(xxxx) throw ....
        31
    Macolor21   352 天前
    想起我之前的代码 WTF,CTRL + ENTER 竟然是提交,不好意思,影响看帖体验了
    if(xxxx) throw ....
    if(yyyy) throw ....
    被改成
    if(xxxx) {
    if(yyyy) {
    ........N 个 if throw
    }
    }
    ..然后我就申请离职了。
        32
    publicvoid   352 天前
    代码轻易不要重构,除非你很懂这块的业务,不然 bug 无限多。。。
        33
    kkkkkrua   352 天前
    看重构的好不好吧,不好可以准备走了。
        34
    ian19znj   351 天前
    既然 review 和 test 都过了,也 check in 进去了,说明重构的没啥问题啊,就不要在意啦。
    话说你们公司是不是规模不大呀?要是大的公司,千百号开发,几百万的代码,每次重构还要逐个征得同意,不累死啊?
    至于以后出问题,那是重构跟审核的锅。
        35
    amumu666   351 天前
    这是对你代码的否定,心里毫无波澜才是假的。
    也许出差找不到你人才让别的同事改这块业务,接盘的同事也是心里 mmp,谁愿意改别人的代码啊。
        36
    jimrok   351 天前
    忍无可忍我才会重构某个别人的代码,通常我会指明代码有什么问题。如果我做这个事,估计这个人我不会再放在重要位置上了。
        37
    specita   351 天前
    我一般是先告诉别人我的想法,看他会不会自觉去优化,不会的话,我还是会手痒优化的。。。
        38
    Keyes   351 天前
    @jinhan13789991 +1,不过最近研发老大和大部门经理终于忍受不了我的碎碎念,把重构提上日程了,要坚持啊盆友
        39
    Keyes   351 天前
    @Macolor21 hhhhh 所谓的“嵌套层次不够”
        40
    signalas1   351 天前
    不要随意重构别人代码,特别是复杂度的部分。。虽然一般写业务搬砖没什么复杂度..
        41
    jinhan13789991   351 天前
    @Keyes #38 我已经从那家离职了,因为健身,比较准时下班,同事都涨薪了我没有~ 和领导提了下涨薪,还被劝退(就是那种说你表现不行,公司给你机会,你要好好表现,决口不提辞退),后面投简历找到更好的就跳槽了。
        42
    zsen   351 天前   ♥ 1
    想起前公司,前端组长跟我吐槽:他的代码被后端组长给改了。
    后端同学说是“重构”,前端同学说是“改”。

    然后没多久,前端同学走了。
        43
    babedoll   351 天前
    说实话直接重构别人代码跟不打招呼直接用别人东西一样,怎么说也要事先给你讲一下吧?
    看不起你的写的代码?有 bug 不好用也要说一声啊。。。看楼上才知道程序员大部分情商都很低。。

    楼主淡定,准备简历把
        44
    Damon4V   351 天前
    没通知下是过分了,准备简历吧,但近期经济形势不好,找到后再辞职;
        45
    fkdog   351 天前   ♥ 1
    程序员自我感觉良好系列之“不是我写的代码都是屎”。

    然后自己 xjb 重构一通发现自己的实现并没有比原来好到哪里去。
        46
    yiqiao   351 天前
    @Macolor21 回归本心,面条式代码嵌套走起。
        47
    xinyewdz   351 天前
    最近接手了几个项目,原来的人调到别的项目上了。我把接手的项目,都重构了一遍。
        48
    jzmws   351 天前   ♥ 1
    @Macolor21 if 后面都是要带大括号的,即使只有一行的时候也是要带的,这样方便后期的维护.
    if 里面套 if 这种就 .
        49
    SimbaPeng   351 天前   ♥ 1
    假如你是做销售的,你的客户被人抢了,你还乐呵呵的说“客户都是公司的,不存在你的客户”?

    楼上都是这种孤儿逻辑?
        50
    Macolor21   351 天前 via Android
    @jzmws
    打字的时候省略了,平时我的习惯不省略大括号
        51
    canxden   351 天前
    刚重构代码. 然后看到这个帖子, 看到之前的文件创建者是 boss....感觉....
        52
    sm0king   351 天前
    哦. . .
    前两天,遇到类似情况,同事把我代码改了,而且还改出 bug 了。
    然后给我提了个 bug 过来. . .
    后来我改了,只备注了一下原因,然后被怼,很长时间不搭理我。
    这几天,他很多需求做不完了,而,我的早做完了,接下来要帮他做需求。
    嗯,能咋滴,还不是一样做?
        53
    gouchaoer2   351 天前
    今日最佳
        54
    nimabibi   351 天前 via Android
    改完坑不用自己填就行了
        55
    victor   351 天前
    @jinhan13789991 咋了兄弟,发完帖被领导发现了?赶紧补救一下
        56
    windfarer   351 天前 via Android
        57
    jinhan13789991   351 天前
    @victor 离职快半年了都~ 后面又被俩公司坑,半个月前入职了现在这个公司。新公司很好很 nice,曾经怀疑是个人问题,为啥俩公司自己都不适应,后面才知道那俩公司实在是坑!
        58
    learnshare   351 天前
    能重构还是不错的,多数代码通常没有这个机会
    另外重构比推翻重来好,项目越大越没人敢动

    可以深入了解一下为何重构,以及重构之后比之前好在哪里
    最难受的不是被重构,而是全部代码代码被统一格式化
        59
    iugo   351 天前
    看看别人重构过的代码, 能否从中学到什么. 如果有学到, 就挺好. 如果没有学到, 也能理解对方的思路, 对以后沟通有好处.
        60
    sampeng   351 天前
    巴不得被重构。一来跳出来一个人背锅,以前里面埋的任何雷都和你没啥关系。另一来,等于多一次机会看看别人的思路。写代码不能固步自封
        61
    dengtongcai   351 天前 via iPhone
    重构的好就学习,不好就别人背锅呗…
        62
    xuanbg   351 天前
    楼主应该是感觉自己被否定了,所以不开心。说实话,被否定没什么,但你要知道为什么被否定。否则可能会接二连三地被否定,就有点尴尬了。总之,多经历一些挫折不是坏事,得到成长就是赚到。
        63
    skypyb   351 天前
    我就偶尔会重构同事的代码啊...
    有的一个方法上百行,一个类上千行。其中无数个 if 嵌套,可读性吊差。我没事做的时候给提取一下拆分一下用设计模式重写一下咋了。
    一问为啥这么写就说写逻辑的时候写嗨了,反正跑的挺 6 没事。也是不知道该说什么:)
        64
    qoras   351 天前
    重构都有一定风险的, 指不定出什么坑, 没有收益还是不要做了.
    有这功夫还不如看看书提升自己
        65
    log4geek   351 天前
    组长重构你的代码,还不告诉你,说明你写的真的好烂。
    正常来说,组长发现你的代码有问题的时候,会先给你指出来,让你自己去重构的。
        66
    zhouyut001   351 天前
    好说,谁重构,谁背锅
        67
    wleexi   351 天前
    借楼问一句,
    重构的后的的代码 让测试同学测试 他们总是不高兴 ,认为没事找事做。。这要怎么办
        68
    hasbug   351 天前
    他重构,还让你填坑,那就说不过去了。
        69
    connection   351 天前
    如果是逻辑上的重构,代码优化上的重构,正当的应该是一起探讨。
    但是如果只是代码风格的重构,把别人的代码风格改变成自己的
    这样子我觉得是无用且不尊重的
        70
    xfriday   350 天前   ♥ 1
    @libook 你真玻璃心,代码易于维护谁高兴重构,你以为重构代码的人还能发奖金?
        71
    zgl263885   350 天前 via iPhone
    组长实在是看不下去你的代码了。我也重构过小弟的代码,刚开始我会跟他讲这个应该怎么样,那个应该怎么样,后来实在不好意思再说他,只好自己默默给重写一遍。代码审查时候我们也很无奈的,管的话小年轻心里不痛快,不管的话屎一样的代码实在吃不下饭!虽然我算不上重度代码洁癖,但也不是啥都能看的下去啊!
        72
    dayoushen   350 天前
    最后谢谢大家的关心
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2303 人在线   最高记录 5043   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.3 · 32ms · UTC 07:44 · PVG 15:44 · LAX 23:44 · JFK 02:44
    ♥ Do have faith in what you're doing.