重构一点思考(再谈重构)

重构的一点思考——再谈重构WhatisRefactoringRefactoringistheprocessofchangingasoftwaresysteminsuchawaythatitdoesnotaltertheexternalbehaviorofthecodeyetimprovesitsinternalstructure.--

MartinFowler我们写代码时，只关注它的正确性，只要机器认识，通过测试就OK了

Ourcode=忽略未来的维护人员维护人员看我们的代码时花费更多的时间来理解代码/*xxx2009-10-21shitUAP有bug，不修改，偏要我们通过这种奇淫技巧进行解决问题*如果不增加一下几行废物代码的话，那么画面的filedgroup就会莫名其妙的丢失，原因是调用了*billcardpnl.setBillData(billcardpnl.getBillData());**/SolutionRefactoringTip:Anyfoolcanwritecodethatacomputercanunderstand.Goodprogrammerswritecodethathumanscanunderstand.WhyRefactoringTheoneconstantissoftwaredevelopmentChangeRefactoring持续改进软件设计使代码更易为人所理解任何一个人都能写出计算机可以理解的程序，只有写出人容易理解的程序才是优秀的程序员！帮助发现隐藏的代码缺陷有助于提高编程效率WhenRefactoringTheRuleofThree在添加新功能时进行重构重构程序，使得加入新功能变得简单添加这个功能在修改bug时进行重构在代码评审时进行重构WhenNotRefactoring

现有的程序无法运行,此时应该是重写程序,而不是重构程序到了最后的交付期限重构中的难题Database修改接口难以通过重构手法完成的设计改动无安全需求安全性良好

11BadSmellsDuplicatedCodeLongMethodLargeClassLongParameterListDivergentChangeShortgunSurgeryFeatureEnvyDataClumpsPrimitiveObsessionSwitchStatementsParallelInheritanceHierarchiesLazyClassSpeculativeGeneralityTemporaryFieldMessageChainsMiddleManInappropriateIntimacyAlternativeClasseswithDifferentInterfacesIncompleteLibraryClassDataClassRefusedBequestComments©OscarNierstrasz8.12TypicalRefactoringsClassRefactoringsMethodRefactoringsAttributeRefactoringsadd(sub)classtohierarchyaddmethodtoclassaddvariabletoclassrenameclassrenamemethodrenamevariableremoveclassremovemethodremovevariablepushmethoddownpushvariabledownpushmethoduppullvariableupaddparametertomethodcreateaccessorsmovemethodtocomponentabstractvariableextractcodeinnewmethodDuplicatedCodeSmell同样的代码出现了两次或更多次！！!违反DRY(Don’tRepeatYourself)RefactorExtractmethodExtractmethod可读性重复代码——抽取方法nc.impl.emship.palletlist.PaletlistQueryImplL274L437L580DuplicatedCodeSmell同样的代码出现在不同的子类中！！!违反DRYRefactorExtractmethodPullupfieldPullupmethodPullupmethod两个子类有重复的代码提取重复的代码至父类DuplicatedCodeSmell子类代码类似但不相同Refactor:使用

ExtractMethod

分离出变化的代码与相似的代码

使用

FormTemplateMethod.重复代码erdeli.rule.insert.CheckUniqueForInsertFiltRerdeli.rule.insert.CheckUniqueUpdateRule重复代码——工具类erdeli.action.InterdeliCopyAerdeli.action.InterdeliAddAerdeli.mediator.OrgChangedForInterdeliEditorTODOFormTemplateMethod重复代码——FormTemplateMcplan.replreq.Rewrite20ForYG24Impl项目采购替代申请删除回写采购计划reWriteProcplanNumForYG24(ReplReqVO_111[])替代申请保存回写累加请购单对应的采购计划reWrite20RepNumForYG24(ReplReqVO_111[])cplan.alter.ProcPlanNQAheadAcplan.alter.ProcPlanNQAfterAlert重复代码——FormTemplateMethod1、加锁2、获取回写Map3、查询请购单4、处理请购单行状态5、构造项目计划待更新集合6、更新项目采购计划装备631重复代码统计后果怎么产生Ctrl+C，Ctrl+V,Ctrl+S不同的人写了相同功能的代码你该怎么办？注重设计规范SONAR重构LongMethodLongmethodVSshortmethodLivebestandlongerThelongeraprocedureis,themoredifficultitistounderstandGoodnaming（ifyouhaveagoodnameforamethodyoudon'tneedtolookatthebody.）尽量分解LongMethod注释可能是抽取小方法的信号LongMethodExtractMethodReplaceTempwithQueryDecomposeCondition

33ReplaceTempwithQuery(以查询取代临时变量)doublebasePrice=quantity*itemPrice;if(basePrice>1000){returnbasePrice*0.95;}else{ returnbasePrice*0.98;}

if(basePrice()>1000){returnbasePrice()*0.95;}else{returnbasePrice()*0.98;}privatedoublebasePrice(){

returnquantity*itemPrice;}DecomposeCondition复杂的条件逻辑增加可读性nc.ui.empcd.distrbill.distrbill.validator.SaveSignInNumValidator.checkReplaceCmaterial()LargeClass太多的职责太多的实例变量ExtractClassLongparameterlist使用全局变量吗？其他方法都可以修改破坏封装多线程问题IntroduceParameterObjectPreserveWholeObjectIntroduceParameterObjectPreserveWholeObject其他ReplaceArraywithobjectReplaceMagicNumberwithsymbolicConstantReplaceArraywithobjectnc.ui.emic.accreq.subaccessoryout.action.AccessoryJoinCheckAction.doAction(ActionEvent)ReplaceMagicNumberwithsymbolicConstantTest你敢重构吗？重构工具Unit

Test重构一些建议在开始重构之前，花时间弄清楚复杂的代码逻辑。保存初始代码把要做的事情一条条列出来根据重构风险级别来调整重构方法重构的步伐请小一些利用编译器警告信息附录1得墨忒耳KentBeck说：

查询方法：如果一个对象有很多逻辑都依赖于另一个对象的状态，可能意味着逻辑放错了地方,①得墨忒耳法则说明：使用对象内部的某个属性时，不要用贯穿的方法去直接读取，应该使用一个包装过的读取函数。得墨忒耳法则规定，某个对象中的任何方法内部，都应该只调用以下内容：

1它自身。

2传入该方法的任何参数，但不包括参数对象的属性。

3方法内部创建的任何对象。

4