31天重构学习笔记

31天重构学习笔记1.封装集合2010-06-2803:46by圣殿骑士,3050visits,网摘,收藏,编辑摘要：由于最近在做重构的项目，所以对重构又重新进行了一遍学习和整理，对31天重构最早接触是在2009年10月份，由于当时没有订阅SeanChambers的blog，所以是在国外的社区上闲逛的时候链接过去的。记得当时一口气看完了整个系列并没有多少感觉，因为这些基本上项目都在使用，只是我们没有专门把它标示和整理出来，所以也没有引起多大的重视。现在突然接手这个重构项目，由于团队成员技术和经验参差不齐，所以有必要专门整理一个重构的纲要，当然这个系列也非常适合做新系统的代码规范参考，只要有代码的地方，这个重构规范就很有价值。周末也不想出去闲逛，因为在刚到这个美丽的城市，没有亲戚或者朋友，所以才能静下心来两天时间写完这个重构参考规范。同时也感受了WindowsLivewriter写文章的快感。当然重构的整体架构得另当别论（整体架构在我的这篇文章有专门的讲解（）。大的架构设计好了以后，这些重构细节点就成了东风之后的大火，对整个项目也是至关重要。31天重构这个系列和《代码大全》、《重构：改善既有代码的设计》比较起来最大的特点就是比较简单、浅显易懂。那么我这些文章也都是学习SeanChambers的31天重构的笔记整理，所以如果大家对这个笔记有任何异议也可以指出。具体也可以通过查看原文。概念：本文所讲的封装集合就是把集合进行封装，只提供调用端需要的接口。正文：在很多时候，我们都不希望把一些不必要的操作暴露给调用端，只需要给它所需要的操作或数据就行，那么做法就是封装。这个重构在微软的代码库也经常遇到。比如最经典的属性对字段的封装就是一个很好的例子，那么下面我们将看到对集合的封装，如下代码所示，调用端只需要一个集合的信息，而我们则提供了一个IList的集合，大家都知道IList具有对集合的所有操作，所以这会带来很多隐患，最好的做法就是对它进行重构。usingSystem.Collections.Generic;namespaceLosTechies.DaysOfRefactoring.EncapsulateCollection.Before{publicclassOrder{privateListOrderLine_orderLines;privatedouble_orderTotal;publicIListOrderLineOrderLines{get{return_orderLines;}}publicvoidAddOrderLine(OrderLineorderLine){_orderTotal+=orderLine.Total;_orderLines.Add(orderLine);}publicvoidRemoveOrderLine(OrderLineorderLine){orderLine=_orderLines.Find(o=o==orderLine);if(orderLine==null)return;_orderTotal-=orderLine.Total;_orderLines.Remove(orderLine);}}publicclassOrderLine{publicdoubleTotal{get;privateset;}}}那么重构之后，我们把IList换成了IEnumerable，大家都知道只包括一个返回值为IEnumerator的GetEnumerator()方法，所以这样只能遍历取出它的值，而不能对这个集合做出改变，这正是我们所需要的结果，具体代码如下：usingSystem.Collections.Generic;namespaceLosTechies.DaysOfRefactoring.EncapsulateCollection.After{publicclassOrder{privateListOrderLine_orderLines;privatedouble_orderTotal;publicIEnumerableOrderLineOrderLines{get{return_orderLines;}}publicvoidAddOrderLine(OrderLineorderLine){_orderTotal+=orderLine.Total;_orderLines.Add(orderLine);}publicvoidRemoveOrderLine(OrderLineorderLine){orderLine=_orderLines.Find(o=o==orderLine);if(orderLine==null)return;_orderTotal-=orderLine.Total;_orderLines.Remove(orderLine);}}publicclassOrderLine{publicdoubleTotal{get;privateset;}}}总结：这个例子很容易让我们想到以前系统间耦合常喜欢用数据库。每个系统都会操作数据库，并且有些系统还会对数据库的表结构或字段进行修改，那么这很容易就会造成维护的地狱，很明智的一个做法就是使用SOA来隔开这些耦合，让一些只需要数据展示的系统得到自己需要的数据即可。2.移动方法2010-06-2803:49by圣殿骑士,1358visits,网摘,收藏,编辑摘要：由于最近在做重构的项目，所以对重构又重新进行了一遍学习和整理，对31天重构最早接触是在2009年10月份，由于当时没有订阅SeanChambers的blog，所以是在国外的社区上闲逛的时候链接过去的。记得当时一口气看完了整个系列并没有多少感觉，因为这些基本上项目都在使用，只是我们没有专门把它标示和整理出来，所以也没有引起多大的重视。现在突然接手这个重构项目，由于团队成员技术和经验参差不齐，所以有必要专门整理一个重构的纲要，当然这个系列也非常适合做新系统的代码规范参考，只要有代码的地方，这个重构规范就很有价值。周末也不想出去闲逛，因为在刚到这个美丽的城市，没有亲戚或者朋友，所以才能静下心来两天时间写完这个重构参考规范。同时也感受了WindowsLivewriter写文章的快感。当然重构的整体架构得另当别论（整体架构在我的这篇文章有专门的讲解（）。大的架构设计好了以后，这些重构细节点就成了东风之后的大火，对整个项目也是至关重要。31天重构这个系列和《代码大全》、《重构：改善既有代码的设计》比较起来最大的特点就是比较简单、浅显易懂。那么我这些文章也都是学习SeanChambers的31天重构的笔记整理，所以如果大家对这个笔记有任何异议也可以指出。具体也可以通过查看原文。概念：本文所讲的移动方法就是方法放在合适的位置（通常指放在合适的类中）。正文：移动方法是一个很简单也很常见的重构，只要是系统就会存在很多类，那么类里面包括很多方法，如果一个方法经常被另外一个类使用（比本身的类使用还多）或者这个方法本身就不应该放在这个类里面，那么这个适合应该考虑把它移到合适的类中。代码如下：namespaceLosTechies.DaysOfRefactoring.MoveMethod.Before{publicclassBankAccount{publicBankAccount(intaccountAge,intcreditScore,AccountInterestaccountInterest){AccountAge=accountAge;CreditScore=creditScore;AccountInterest=accountInterest;}publicintAccountAge{get;privateset;}publicintCreditScore{get;privateset;}publicAccountInterestAccountInterest{get;privateset;}publicdoubleCalculateInterestRate(){if(CreditScore800)return0.02;if(AccountAge10)return0.03;return0.05;}}publicclassAccountInterest{publicBankAccountAccount{get;privateset;}publicAccountInterest(BankAccountaccount){Account=account;}publicdoubleInterestRate{get{returnAccount.CalculateInterestRate();}}publicboolIntroductoryRate{get{returnAccount.CalculateInterestRate()0.05;}}}}移动以后大家可以看到BankAccount类的职责也单一，同时CalculateInterestRate也放到了经常使用且适合它的类中了，所以此重构是一个比较好的重构，能让整个代码变得更加合理。namespaceLosTechies.DaysOfRefactoring.MoveMethod.After{publicclassAccountInterest{publicBankAccountAccount{get;privateset;}publicAccountInterest(BankAccountaccount){Account=account;}publicdoubleInterestRate{get{returnCalculateInterestRate();}}publicboolIntroductoryRate{get{returnCalculateInterestRate()0.05;}}publicdoubleCalculateInterestRate(){if(Account.CreditScore800)return0.02;if(Account.AccountAge10)return0.03;return0.05;}}}namespaceLosTechies.DaysOfRefactoring.MoveMethod.After{publicclassBankAccount{publicBankAccount(intaccountAge,intcreditScore,AccountInterestaccountInterest){AccountAge=accountAge;CreditScore=creditScore;AccountInterest=accountInterest;}publicintAccountAge{get;privateset;}publicintCreditScore{get;privateset;}publicAccountInterestAccountInterest{get;privateset;}}}总结：这个重构法则在很多时候能让我们把代码组织的结构调整得更合理，同时也能给以后的维护带来方便。3.提升方法2010-06-2803:52by圣殿骑士,1