分享
 
 
 

重构的快乐

王朝other·作者佚名  2006-01-09
窄屏简体版  字體: |||超大  

今儿个真是高兴,成功把一个140来行,前后大概4页多的函数重构为50多行的函数(如果算上添加的辅助函数、以及重构出的函数的话一共是80多行),而且用新的、更好的方法代替了原来的(这是为什么要重构的原因)。

原来的函数可以工作,但极其冗长,逻辑很难理解。我首先把每一句都加上了详细注释,然后运行了两遍,合并了一些冗余的逻辑,理清了思路。然后就是写一个接口相同的新函数来代替它。

这里我用了个小技巧,就是用同样的接口定义一个新函数,老函数并不删除,而是在老函数顶部把调用传递到新函数。这样如果测试有问题,可以简单注释然后使用老函数。

重构的过程比较麻烦,我本想在原来的代码基础上修改,但它的逻辑实在有点乱,而且太长。所以就在自己理解的基础上把逻辑框架用注释的形式写了出来。然后就是填空,每个注释不超过两行代码。多的代码就抽出来做成函数。当最终写完以后我竟然惊讶的发现,我的新逻辑中也有两个地方明显重复,从代码看都完全一样,于是很高兴的又对逻辑进行了一次重构,更加的清晰了。

现在的代码中,逻辑部分只有1页,完全能够清晰地自说明。虽然仍然有继续重构直到函数不超过1页的余地,但我觉得这样已经够了。后面不管是我自己还是别人来维护这段代码都很快理解程序逻辑而且很容易对其进行改动了。

如果不先重构,理清逻辑的话,我可能会再度陷在100多行的代码里不可自拔。

 
 
 
免责声明:本文为网络用户发布,其观点仅代表作者个人观点,与本站无关,本站仅提供信息存储服务。文中陈述内容未经本站证实,其真实性、完整性、及时性本站不作任何保证或承诺,请读者仅作参考,并请自行核实相关内容。
2023年上半年GDP全球前十五强
 百态   2023-10-24
美众议院议长启动对拜登的弹劾调查
 百态   2023-09-13
上海、济南、武汉等多地出现不明坠落物
 探索   2023-09-06
印度或要将国名改为“巴拉特”
 百态   2023-09-06
男子为女友送行,买票不登机被捕
 百态   2023-08-20
手机地震预警功能怎么开?
 干货   2023-08-06
女子4年卖2套房花700多万做美容:不但没变美脸,面部还出现变形
 百态   2023-08-04
住户一楼被水淹 还冲来8头猪
 百态   2023-07-31
女子体内爬出大量瓜子状活虫
 百态   2023-07-25
地球连续35年收到神秘规律性信号,网友:不要回答!
 探索   2023-07-21
全球镓价格本周大涨27%
 探索   2023-07-09
钱都流向了那些不缺钱的人,苦都留给了能吃苦的人
 探索   2023-07-02
倩女手游刀客魅者强控制(强混乱强眩晕强睡眠)和对应控制抗性的关系
 百态   2020-08-20
美国5月9日最新疫情:美国确诊人数突破131万
 百态   2020-05-09
荷兰政府宣布将集体辞职
 干货   2020-04-30
倩女幽魂手游师徒任务情义春秋猜成语答案逍遥观:鹏程万里
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案神机营:射石饮羽
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案昆仑山:拔刀相助
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案天工阁:鬼斧神工
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案丝路古道:单枪匹马
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:与虎谋皮
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:李代桃僵
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:指鹿为马
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案金陵:小鸟依人
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案金陵:千金买邻
 干货   2019-11-12
 
推荐阅读
 
 
 
>>返回首頁<<
 
靜靜地坐在廢墟上,四周的荒凉一望無際,忽然覺得,淒涼也很美
© 2005- 王朝網路 版權所有