![重构一点思考(再谈重构)_第1页](http://file4.renrendoc.com/view/041ca1e6d089ad5c9895004ec3ba1bbd/041ca1e6d089ad5c9895004ec3ba1bbd1.gif)
![重构一点思考(再谈重构)_第2页](http://file4.renrendoc.com/view/041ca1e6d089ad5c9895004ec3ba1bbd/041ca1e6d089ad5c9895004ec3ba1bbd2.gif)
![重构一点思考(再谈重构)_第3页](http://file4.renrendoc.com/view/041ca1e6d089ad5c9895004ec3ba1bbd/041ca1e6d089ad5c9895004ec3ba1bbd3.gif)
![重构一点思考(再谈重构)_第4页](http://file4.renrendoc.com/view/041ca1e6d089ad5c9895004ec3ba1bbd/041ca1e6d089ad5c9895004ec3ba1bbd4.gif)
![重构一点思考(再谈重构)_第5页](http://file4.renrendoc.com/view/041ca1e6d089ad5c9895004ec3ba1bbd/041ca1e6d089ad5c9895004ec3ba1bbd5.gif)
版权说明:本文档由用户提供并上传,收益归属内容提供方,若内容存在侵权,请进行举报或认领
文档简介
重构的一点思考——再谈重构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
温馨提示
- 1. 本站所有资源如无特殊说明,都需要本地电脑安装OFFICE2007和PDF阅读器。图纸软件为CAD,CAXA,PROE,UG,SolidWorks等.压缩文件请下载最新的WinRAR软件解压。
- 2. 本站的文档不包含任何第三方提供的附件图纸等,如果需要附件,请联系上传者。文件的所有权益归上传用户所有。
- 3. 本站RAR压缩包中若带图纸,网页内容里面会有图纸预览,若没有图纸预览就没有图纸。
- 4. 未经权益所有人同意不得将文件中的内容挪作商业或盈利用途。
- 5. 人人文库网仅提供信息存储空间,仅对用户上传内容的表现方式做保护处理,对用户上传分享的文档内容本身不做任何修改或编辑,并不能对任何下载内容负责。
- 6. 下载文件中如有侵权或不适当内容,请与我们联系,我们立即纠正。
- 7. 本站不保证下载资源的准确性、安全性和完整性, 同时也不承担用户因使用这些下载资源对自己和他人造成任何形式的伤害或损失。
最新文档
- 员工劳动争议管理办法
- 台式皂液器项目投资计划书
- 《燃气流量计体积修正仪GBT+36242-2018》详细解读
- 《同一首歌》音乐课件
- 《给动物分类》课件
- 日本工业物流房地产市场前景及投资研究报告-培训课件外文版2024.5恒大碧桂园万达万科
- 2024年河北高考生物试卷真题评析及答案详解(精校打印)
- 第06讲 《消息二则》-新八年级语文暑假提升自学课讲义(统编版)(解析版)
- 唤醒教育规划与实施方案(3篇模板)
- 化学教育教学研究(3篇模板)
- 分析仪表基础课件
- 商务英语写作之货运通知
- 普及人民代表大会制度知识竞赛试题库(1000题和答案)
- 幼儿园中班语言绘本《章鱼先生卖雨伞》课件
- 小学生汉语拼音田字格练习纸蓝打印版
- 你好,无废校园主题班会
- 痛风的规范治疗课件
- 《小学语文关于整本书阅读的策略研究》课题研究方案
- (骨筋膜室综合征)骨科小讲课课件
- 物理前沿科技
- 当下的力量读后感市公开课获奖课件
评论
0/150
提交评论