《重构-改善既有代码的设计》reading
程序员文章站
2022-06-08 19:25:54
...
《重构--改善既有代码的设计》
【道班工人+电动砸道机+枕木+道渣】
【重构】在不破坏可察功能的前提下,借由搬移、提炼、打散、凝聚……,改善事物的体质。
强化当前的可读性,为将来的扩充性和维护性做准备,乃至于在过程中找出潜伏的臭虫。
【一句古老的工程谚语】如果它可以运行,就不要动它。
以持续不断的重构行为来整理代码。--以重构方式改进软件质量。
【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