重構(gòu)一點(diǎn)思考(再談重構(gòu))

重構(gòu)的一點(diǎn)思考——再談重構(gòu)WhatisRefactoringRefactoringistheprocessofchangingasoftwaresysteminsuchawaythatitdoesnotaltertheexternalbehaviorofthecodeyetimprovesitsinternalstructure.--

MartinFowler我們寫代碼時(shí)，只關(guān)注它的正確性，只要機(jī)器認(rèn)識(shí)，通過測(cè)試就OK了

Ourcode=忽略未來的維護(hù)人員維護(hù)人員看我們的代碼時(shí)花費(fèi)更多的時(shí)間來理解代碼/*xxx2009-10-21shitUAP有bug，不修改，偏要我們通過這種奇淫技巧進(jìn)行解決問題*如果不增加一下幾行廢物代碼的話，那么畫面的filedgroup就會(huì)莫名其妙的丟失，原因是調(diào)用了*billcardpnl.setBillData(billcardpnl.getBillData());**/SolutionRefactoringTip:Anyfoolcanwritecodethatacomputercanunderstand.Goodprogrammerswritecodethathumanscanunderstand.WhyRefactoringTheoneconstantissoftwaredevelopmentChangeRefactoring持續(xù)改進(jìn)軟件設(shè)計(jì)使代碼更易為人所理解任何一個(gè)人都能寫出計(jì)算機(jī)可以理解的程序，只有寫出人容易理解的程序才是優(yōu)秀的程序員！幫助發(fā)現(xiàn)隱藏的代碼缺陷有助于提高編程效率WhenRefactoringTheRuleofThree在添加新功能時(shí)進(jìn)行重構(gòu)重構(gòu)程序，使得加入新功能變得簡(jiǎn)單添加這個(gè)功能在修改bug時(shí)進(jìn)行重構(gòu)在代碼評(píng)審時(shí)進(jìn)行重構(gòu)WhenNotRefactoring

現(xiàn)有的程序無法運(yùn)行,此時(shí)應(yīng)該是重寫程序,而不是重構(gòu)程序到了最后的交付期限重構(gòu)中的難題Database修改接口難以通過重構(gòu)手法完成的設(shè)計(jì)改動(dòng)無安全需求安全性良好

11BadSmellsDuplicatedCodeLongMethodLargeClassLongParameterListDivergentChangeShortgunSurgeryFeatureEnvyDataClumpsPrimitiveObsessionSwitchStatementsParallelInheritanceHierarchiesLazyClassSpeculativeGeneralityTemporaryFieldMessageChainsMiddleManInappropriateIntimacyAlternativeClasseswithDifferentInterfacesIncompleteLibraryClassDataClassRefusedBequestComments?OscarNierstrasz8.12TypicalRefactoringsClassRefactoringsMethodRefactoringsAttributeRefactoringsadd(sub)classtohierarchyaddmethodtoclassaddvariabletoclassrenameclassrenamemethodrenamevariableremoveclassremovemethodremovevariablepushmethoddownpushvariabledownpushmethoduppullvariableupaddparametertomethodcreateaccessorsmovemethodtocomponentabstractvariableextractcodeinnewmethodDuplicatedCodeSmell同樣的代碼出現(xiàn)了兩次或更多次！！!違反DRY(Don’tRepeatYourself)RefactorExtractmethodExtractmethod可讀性重復(fù)代碼——抽取方法nc.impl.emship.palletlist.PaletlistQueryImplL274L437L580DuplicatedCodeSmell同樣的代碼出現(xiàn)在不同的子類中??！!違反DRYRefactorExtractmethodPullupfieldPullupmethodPullupmethod兩個(gè)子類有重復(fù)的代碼提取重復(fù)的代碼至父類DuplicatedCodeSmell子類代碼類似但不相同Refactor:使用

ExtractMethod

分離出變化的代碼與相似的代碼

使用

FormTemplateMethod.重復(fù)代碼erdeli.rule.insert.CheckUniqueForInsertFiltRerdeli.rule.insert.CheckUniqueUpdateRule重復(fù)代碼——工具類erdeli.action.InterdeliCopyAerdeli.action.InterdeliAddAerdeli.mediator.OrgChangedForInterdeliEditorTODOFormTemplateMethod重復(fù)代碼——FormTemplateMcplan.replreq.Rewrite20ForYG24Impl項(xiàng)目采購替代申請(qǐng)刪除回寫采購計(jì)劃reWriteProcplanNumForYG24(ReplReqVO_111[])替代申請(qǐng)保存回寫累加請(qǐng)購單對(duì)應(yīng)的采購計(jì)劃reWrite20RepNumForYG24(ReplReqVO_111[])cplan.alter.ProcPlanNQAheadAcplan.alter.ProcPlanNQAfterAlert重復(fù)代碼——FormTemplateMethod1、加鎖2、獲取回寫Map3、查詢請(qǐng)購單4、處理請(qǐng)購單行狀態(tài)5、構(gòu)造項(xiàng)目計(jì)劃待更新集合6、更新項(xiàng)目采購計(jì)劃裝備631重復(fù)代碼統(tǒng)計(jì)后果怎么產(chǎn)生Ctrl+C，Ctrl+V,Ctrl+S不同的人寫了相同功能的代碼你該怎么辦？注重設(shè)計(jì)規(guī)范SONAR重構(gòu)LongMethodLongmethodVSshortmethodLivebestandlongerThelongeraprocedureis,themoredifficultitistounderstandGoodnaming（ifyouhaveagoodnameforamethodyoudon'tneedtolookatthebody.）盡量分解LongMethod注釋可能是抽取小方法的信號(hào)LongMethodExtractMethodReplaceTempwithQueryDecomposeCondition

33ReplaceTempwithQuery(以查詢?nèi)〈R時(shí)變量)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復(fù)雜的條件邏輯增加可讀性nc.ui.empcd.distrbill.distrbill.validator.SaveSignInNumValidator.checkReplaceCmaterial()LargeClass太多的職責(zé)太多的實(shí)例變量ExtractClassLongparameterlist使用全局變量嗎？其他方法都可以修改破壞封裝多線程問題IntroduceParameterObjectPreserveWholeObjectIntroduceParameterObjectPreserveWholeObject其他ReplaceArraywithobjectReplaceMagicNumberwithsymbolicConstantReplaceArraywithobjectnc.ui.emic.accreq.subaccessoryout.action.AccessoryJoinCheckAction.doAction(ActionEvent)ReplaceMagicNumberwithsymbolicConstantTest你敢重構(gòu)嗎？重構(gòu)工具Unit

Test重構(gòu)一些建議在開始重構(gòu)之前，花時(shí)間弄清楚復(fù)雜的代碼邏輯。保存初始代碼把要做的事情一條條列出來根據(jù)重構(gòu)風(fēng)險(xiǎn)級(jí)別來調(diào)整重構(gòu)方法重構(gòu)的步伐請(qǐng)小一些利用編譯器警告信息附錄1得墨忒耳KentBeck說：

查詢方法：如果一個(gè)對(duì)象有很多邏輯都依賴于另一個(gè)對(duì)象的狀態(tài)，可能意味著邏輯放錯(cuò)了地方,①得墨忒耳法則說明：使用對(duì)象內(nèi)部的某個(gè)屬性時(shí)，不要用貫穿的方法去直接讀取，應(yīng)該使用一個(gè)包裝過的讀取函數(shù)。得墨忒耳法則規(guī)定，某個(gè)對(duì)象中的任何方法內(nèi)部，都應(yīng)該只調(diào)用以下內(nèi)容：

1它自身。

2傳入該方法的任何參數(shù)，但不包括參數(shù)對(duì)象的屬性。

3方法內(nèi)部創(chuàng)建的任何對(duì)象。

4