到底应不应该重构别人写的代码? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
fwrq41251
V2EX    编程

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

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

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

    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     Solana     3194 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 25ms UTC 10:52 PVG 18:52 LAX 02:52 JFK 05:52
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86