欢迎您访问程序员文章站本站旨在为大家提供分享程序员计算机编程知识!
您现在的位置是: 首页

《重构-改善既有代码的设计》reading

程序员文章站 2022-06-08 19:26:18
...

《重构--改善既有代码的设计》

【道班工人+电动砸道机+枕木+道渣】

【重构】在不破坏可察功能的前提下,借由搬移、提炼、打散、凝聚……,改善事物的体质。
强化当前的可读性,为将来的扩充性和维护性做准备,乃至于在过程中找出潜伏的臭虫。

【一句古老的工程谚语】如果它可以运行,就不要动它。

以持续不断的重构行为来整理代码。--以重构方式改进软件质量。

【refactoring】在不改变代码外在行为的前提下,对代码做出修改,以改进程序的内部结构。
重构是一种有纪律的、经过训练的、有条不紊的程序整理方法,可以将整理过程中不小心引入错误的机率降到最低。

你只需要把某个值域从一个class移到另一个class,把某些代码从一个method拉出来构成另一个method,或是在class hierarchy中把某些代码推上推下就行了。聚沙成塔!

快速而随性quick and dirty地设计一个简单的程序并没有错。但如果是复杂系统中具有代表性的一段,那就真的要对这个程序信心动摇了。即便如此,这个程序还能正常工作,所以这只是美学意义上的判断,只是对于丑陋的代码,编译器才不会在乎代码好不好看呢。但是涉及到人,人在乎这些。这个程序也许还没坏掉,但它带来了伤害。它让你的生活比较难过,因为你发现很难完成客户所需的修改。这时候refactoring技术粉墨登场。

【如果你发现自己需要为程序添加一个特性,而代码结构使你无法很方便地那么做,那就先重构那个程序,使特性的添加比较容易进行,然后再添加特性。】


【重构第一步:为即将修改的代码建立一组可靠的测试环境。self-checking】尽管可以避免绝大多数臭虫引入的机会,但是人就可能犯错,所以。。。

【代码区域块愈小,代码的功能就愈容易管理,代码的处理和搬移就愈轻松】
第一个步骤是找出代码的逻辑泥团logical clump,并运用extract method。如提取switch到独立函数中。
找局部变量。任何不会被修改的变量,都可以当成参数传入新函数。会被修改的变量,要格外小心。如果只有一个变量会被修改,可以把它当作返回值。
【重构技术系以微小的步伐修改函数,如果你犯下错误,很容易便可发现它。】refactoring browser

提取出来的函数入参,要更改为人类容易理解的代码,这才是优秀的程序员做的事。【代码应该表现出自己的目的。】

【函数搬家extract method】如果一个函数,仅涉及某一个bean类内的操作,则绝大多数情况下,函数应该放在它所使用的数据的所属object/class内。

尽量把脱离具体业务的小处理流程抽象提炼出来。

寻找局部变量,当作参数传入;

对于临时变量,若该临时变量在被要抽离的部分使用之前已经先有初值,但提炼出来的块中并没有读取该值,则无需将它当作参数传入。只需执行[ 附添赋值动作,appeding assignment,operator+= ]就行了。如:
i++;//临时变量计数器
if((each.getMovie().getCode() == Movie.NEW_RELEASE) && each.getDayRent() > 1)
i++;
原调用点改为:i += each.getRentPoints();
int getRentPoints(){
if((each.getMovie().getCode() == Movie.NEW_RELEASE) && each.getDayRent() > 1)
return 2;
else
return 1;
}

适应新家,重新编译。
适应新家:去掉参数,变更函数名称。改变新函数内容,使他委托delegate新函数。

第二个步骤:找出原代码块中对于旧函数的所有引用reference点,并修改它们。
1)查找引用点:一般情况下,你得在可能运用该函数的所有classes中查找一遍。
2)更换旧函数为新函数

第三个步骤:replace temp with query
函数中,那些接受了赋值操作后,当做常量来使的临时变量。
因为:临时变量往往形成问题,它们会导致大量参数被传来传去,而其实完全没有这种必要。你很容易失去它们的踪迹,尤其在长长的函数之中更是如此。
但是:这样做,会带来性能上的代价。如每次都会经过一定计算。但这很容易被优化,祥看[p69重构与性能]一章。

【去除临时变量】query method
double count = 0
while(rs.hasMoreElement()){
...
count  += each.getPrice();
}
System.out.println(count );
>>
去掉声明,去掉while里的累加;
System.out.println(getTotalCount() );
double getTotalCount(){
double result = 0
while(rs.hasMoreElement()){
...
result  += each.getPrice();
}
return result;
}
注意:大多数重构,都会减少代码总量,但是这次却增加了代码质量,且性能降低。
重构时不用担心这些,优化时你才需要担心它们,但那时候你已处于一个比较有利的位置,有更多选择可以完成有效优化。


注:重构时最好小步前进,如此一来犯错的几率最小。


运用多态polymorphism
在另一个对象的属性基础上运用switch语句,并不是什么好主意。如果不得不使用,也应该在对象自己的数据上使用,而不是在别人的数据上使用。
switch(getMovie().getPrice())-->移入Movie

【继承inheritance的重构】--往往使用state pattern【Gang of Four】
注:对于模式和其名称的选择,反应出你对结构的想法。可能会使用strategy模式升级state。
三个重构准则:
1)运用replace type code with state/strategy将[与型别相依的行为]搬移至state模式内。
2)运用move method 将switch语句移到它所使用的数据的所属object/class内。
3)运用replace conditional with polymorphism去掉switch语句。
reading p68