V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
fwrq41251
V2EX  ›  编程

到底应不应该重构别人写的代码?

  •  
  •   fwrq41251 · 2016-06-22 10:48:29 +08:00 · 4483 次点击
    这是一个创建于 3079 天前的主题,其中的信息可能已经有所发展或是发生改变。

    按照童子军原则来说,看到有问题的代码就应该马上修正它。 但是实际上重构别人的代码可能会让别人觉得没有尊重他的劳动成果,特别是,重构一个比自己更年长的人的代码。

    13 条回复    2016-07-20 06:57:11 +08:00
    wjfz
        1
    wjfz  
       2016-06-22 10:54:15 +08:00
    这要看你们公司的氛围
    yangqi
        2
    yangqi  
       2016-06-22 10:56:31 +08:00
    不应该,浪费时间浪费资源,很多情况觉得可以重构的其实是自作聪明的行为。
    awanabe
        3
    awanabe  
       2016-06-22 11:11:55 +08:00   ❤️ 1
    当你因为不了解业务, 没有在系统的讨论下重构之后 踩了坑,你就知道线上故障都是你们年轻人的
    chilaoqi
        4
    chilaoqi  
       2016-06-22 11:21:06 +08:00
    到底应不应该帮隔壁老王媳妇生个像我孩子。
    MajestySolor
        5
    MajestySolor  
       2016-06-22 11:23:30 +08:00
    楼主肯定以前没试过
    亲身体验,四个字,自讨苦吃
    chilaoqi
        6
    chilaoqi  
       2016-06-22 11:24:19 +08:00
    老王年纪比我大。
    老王孩子(视代码如己出)太讨厌太烦人啦。
    这孩子从小这样,以后肯定危害社会。(这垃圾代码上线一定崩)
    为了公司利益。。。

    老王会 neng 死你的,哈哈
    fwrq41251
        7
    fwrq41251  
    OP
       2016-06-22 11:45:05 +08:00
    @yangqi 这次是真的应该重构的地方,因为之前这个模块都是我负责的,而且我已经写过很多遍了(每次重写都会比之前多一些心得)。之前因为出差了一个多月,同事才在新项目里写了这一个模块。
    ChiangDi
        8
    ChiangDi  
       2016-06-22 12:12:45 +08:00 via Android
    特别是,看着长者的代码写得非常烂
    est
        9
    est  
       2016-06-22 12:24:21 +08:00
    重构代码要 3 倍的精力。
    twtiyb
        10
    twtiyb  
       2016-06-22 12:41:43 +08:00
    2 个人平级的话,不重构。
    zxgngl
        11
    zxgngl  
       2016-06-22 17:14:12 +08:00
    崇尚技术,以人为本。
    yuekcc
        12
    yuekcc  
       2016-06-26 13:11:57 +08:00 via Android
    我也遇到了。

    长者代码很垃圾,沟通未果,果断重构。为什么?因为我才是被烂搞死的那个!
    skinheadbob
        13
    skinheadbob  
       2016-07-20 06:57:11 +08:00 via iPhone
    跟老板打好招呼,重构前先写足 test case
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3522 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 24ms · UTC 04:50 · PVG 12:50 · LAX 20:50 · JFK 23:50
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.