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

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

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

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

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

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

【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