如何继承糟糕的代码? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
推荐学习书目
Learn Python the Hard Way
Python Sites
PyPI - Python Package Index
http://diveintopython.org/toc/index.html
Pocoo
值得关注的项目
PyPy
Celery
Jinja2
Read the Docs
gevent
pyenv
virtualenv
Stackless Python
Beautiful Soup
结巴中文分词
Green Unicorn
Sentry
Shovel
Pyflakes
pytest
Python 编程
pep8 Checker
Styles
PEP 8
Google Python Style Guide
Code Style from The Hitchhiker's Guide
Sherlockhlt
V2EX    Python

如何继承糟糕的代码?

  •  
  •   Sherlockhlt 2012-11-10 10:24:40 +08:00 4537 次点击
    这是一个创建于 4792 天前的主题,其中的信息可能已经有所发展或是发生改变。
    接手了一个项目,现有代码感觉写得很混乱,很有推倒重写的冲动。。可是原来项目的负责人又不肯。。现在对项目完全没有继续写的欲望了。
    15 条回复    1970-01-01 08:00:00 +08:00
    alexrezit
        1
    alexrezit  
       2012-11-10 10:32:05 +08:00
    一边先将就着改 bug, 与此同时另一边推倒重写, 一个模块一个模块替换到旧项目上去, 然后最后差不多的时候就整个 switch 到新项目.
    sdjl
        2
    sdjl  
       2012-11-10 10:49:56 +08:00
    让负责人自己肯。。。
    xiaket
        3
    xiaket  
       2012-11-10 10:56:57 +08:00
    @alexrezit 正解, 逐个模块地改善改进, 尽量减少升级过程中的烦恼.
    qiukun
        4
    qiukun  
       2012-11-10 11:02:54 +08:00
    能隔离好就继续用吧
    haha1903
        5
    haha1903  
       2012-11-10 11:28:42 +08:00
    @alexrezit
    @xiaket
    推倒重写是不对的,不停的 refactory,同时了解需求,能够完全理解这个项目的时候,你就不会想重写了,只会想把当前的项目变得更好。
    powerfj
        6
    powerfj  
       2012-11-10 11:33:21 +08:00
    还有一个就是不要以为自己写的代码就是好代码,自己写代码的时候写完也找几个人评审一下看看,不然到别人那里你写的又是写的烂的代码
    alexrezit
        7
    alexrezit  
       2012-11-10 11:38:33 +08:00
    @haha1903 如果重写比修改更省时省力的话为什么不推倒重写呢?
    platoo
        8
    platoo  
       2012-11-10 12:13:20 +08:00
    吃过亏的人表示赞成 @haha1903,了解需求,不停的refactory是省时省力的方法。
    且分享一例,当时接受了一个新模块,优化的时候觉得太烂了,一个功能的函数分拆到两个函数里面,导致大段功能重复代码。脑子一热,推倒重写!其实并不是很大的优化:将复用的代码提取,功能耦合的代码合并。结果测试的时候出现了bug,调试了两天。后来询问了周围所有接触过的这个模块的人。发现如果不分开的话,时序上面会紊乱。当时项目时间紧,就将就用了。我不幸中枪。所以,一定要在理解功能的基础上,透彻理解当时的项目背景什么的再修改啊!!
    darktiny
        9
    darktiny  
       2012-11-10 13:01:04 +08:00
    最近也在困扰这个问题
    imom0
        10
    imom0  
       2012-11-10 13:07:47 +08:00
    15楼|prancer80|2012-10-30 03:32:00|只看此ID
    回复|发信|转寄
    给你说一个我同事的例子吧:
    第一个阶段,天天骂祖宗。
    第二个阶段,天天骂娘。
    第三个阶段,天天骂写代码的SB。
    第四个阶段,天天爆炸头,继续啃。
    第五个阶段,天天跟老板要求换岗。
    第六个阶段,继续啃。
    第七个阶段,离职。

    [ 在 creataaa 的大作中提到: ]
    : 新人先后接手两个项目,都是公司积累了好几年的,都是几万行几乎无注释,都是逻辑很复杂、各种线程交错。。读得慢死了。。求经验。。。
    --
    FROM 213.55.76.*
    asoncheng
        11
    asoncheng  
       2012-11-10 18:10:59 +08:00
    只能说循序渐进,各个击破。
    遇到的任何地方的代码都会遇到这样的情况,普遍现象。单独花时间重新写一遍,主管肯定觉得没经济价值,而在中国这样只看重当下经济回报急功近利的背景下,LZ还是好好学习,一点点来吧。
    haha1903
        12
    haha1903  
       2012-11-10 18:23:19 +08:00
    @alexrezit 重写比修改省时省力,还要建立在系统所有的特性都完整,我虽然反对再来一版,但也觉得可以吧。
    问题是,这两个前提是怎么确定的,不客气的说一句,大部分时候觉得重写省时省力的原因,是不想去弄原来“一团乱麻”的代码,说到底就是懒。可是,原来的代码你都不明白,你怎么保证系统可用并且功能完整。
    所以,你首先要去了解旧系统,可能你非常了解旧系统之后,就不会再觉得重写一份省时省力了。
    @Sherlockhlt @alexrezit
    过来人给个建议吧,如果真的有足够的责任感,希望把事情做好。
    那么建议分三步:

    1. 给你要维护的功能补充一些自动化测试代码,无论是 UT 还是 ST
    2. 慢慢看懂,并且按照自己的理解去 refactory 代码,第一步来保证你的 refactory 是正确的。(当然不能完全保证)
    3. 重复前两步
    haha1903
        13
    haha1903  
       2012-11-10 18:25:09 +08:00
    @platoo 支持一把,呵呵
    alexrezit
        14
    alexrezit  
       2012-11-10 18:40:58 +08:00
    @imom0 "在 xxx 的大作总提到" 怎么那么像我们学校 nforum 的风格...
    alexrezit
        15
    alexrezit  
       2012-11-10 18:42:43 +08:00
    @haha1903
    如果你接手的项目里几乎所有的 bug 都是因为编写习惯不好而带来的, 肯定就会认为重写更省力.
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1464 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 25ms UTC 16:48 PVG 00:48 LAX 08:48 JFK 11:48
    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