几种常用的代码重构方法

1、Martin Fowler 的著作 Refactoring: Improving the Design of Existing Code .Extract Method(提取函数)样例代码：public void PrintOwing( double amount)Prin tBa nn er();/ print detailsConsole.WriteLine( "name:" + _name);Console.WriteLine( "amount:" + _amount);重构为：public void PrintOwing( double amou

2、nt) Prin tBa nn er();Prin tDetails(amou nt); public void PrintDetails( double amount) / print detailsConsole.WriteLine( "name:" + _name);Console.WriteLine( "amount:" + _amount);思想：让每个函数只做一件事，抽取能够被组织在一起的代码，并单独抽取为一个函数。 如果函数的粒度小，被复用的机会就更大。我常看到初学 ASP.NET的同学喜欢把所有的代码全部写在Page_Load事件中。我们

3、暂且不论分层架构，光看页面本身，如果所有的代码都放在一个函数中，那么可重用性就几乎为0。打个比方，你的页面上有个GridView，你希望打开网页的时候给它绑定数据，于是你讲这段代码放在了 Page_Load中，当 然，Page_Load还会做除此以外的很多处理。然而，在你 相应页面的其他事件的时候(比如点击Button、处理用户输入等)，仍然需要重新绑定GridView的数据。这时候你就非常希望不要重写一遍数据绑定的逻辑(如果重写一个逻辑 很多次，那以后要改的话就得改很多处)。于是，我们就应该把数据绑定逻辑提取到一个单 独的函数中。比如叫 BindData()，然后分别在 Page_Load事

4、件，和其他你需要再次绑定数 据的事件中调用BindData()。PS:有时候需要重新绑定数据，单独写GridViewl.DataBi nd();是不行的。Visual Studio中做这样的重构很简单：选中你要提取的代码片段，然后点右键-> 重构-> 提取方法，键入一个新方法名称即可。VS会自动处理该片段与已有函数的调用关系。关于函数的命名，我建议大家用动宾短语，阅读的时候会感觉比较自然。2.I nli ne Temp(内联临时变量)样例代码：double basePrice = anOrder .BasePrice(); return (basePrice >1000);重

5、构为：return注意：重构前要保证这个临时变量只被赋值一次，所以当遇到for等循环语句时，要注意被内联的变量的值是否会在循环中被改变。有时在执行for之前定义固定的临时变量是必要的。尤其是当临时变量取自某对象的属性，而这个对象将在for循环中被更改。3. Replace Temp with Query(以查询取代临时变量)样例代码：double basePrice = _quantity * _itemPrice;if(basePrice > 1000)return basePrice * 0.95;elsereturn basePrice * 0.98;重构为：if(BasePric

6、e() >1000)return BasePrice() * 0.95 ;elsereturn BasePrice() * 0.98 ;double BasePrice()return _quantity *temPrice;其实对于这个重构，我还是有些疑惑的。我们发现，BasePrice()会被计算多次，但书上说不用担心性能问题，不过我对此还是保持怀疑。毕竟有些计算开销是很大的，并且执行多次可能产生额外的影响，所以我建议大家使用Replace Temp with Query手法的时候要谨慎，如果给一个临时变量复制的操作开销很大，尤其是需要和数据库交互，我不建议采用该手法重构。4. Re

7、place Nested Co nditio nal with Guard Clauses(以卫语句嵌套条件表达式)样例代码：double GetPayAmount()double result;if(_isDead) result = deadAm oun t();elseif(_isSeparated)result = separatedAm oun t(); elseif(_isRetired)result = retiredAm oun t();elseresult = no rmalPayAm oun t();return result;重构为double GetPayAmount(

8、)if(_isDead)return deadAmount();if(_isSeparated)return separatedAmount();if(_isRetired)return retiredAmount();return normalPayAmount();这个重构手法也是我用的很多的，在初学编程的时候，很多人都会写出很多ifelse嵌套的代码，其实很多时候，else并不是必须的。关于ifelse如何取决，关键看你对各分支的 重视程度。摘录书上的原话：根据我的经验，条件表达式通常有两种表现形式。第一种形式是：所有分支都属于正常行为。第二种形式则是：条件表达式提供的答案中只有一种是正

9、常行为，其他都不是常见的情况。这两类表达式有不同的用途，这一点应该通过代码表现出来。如果两条分支都是正常行为，就应该用形如ifelse的条件表达 式；如果某个条件及其罕见，就应该单独检查该条件，并在该条件为真时立刻从函数中返回。这样的单独检查常常被称为卫语句”插一句题外话，我平时写程序，遇到不正常情况需要提示，我通常会在if里throw 一个exception，这些if是并行的，不是嵌套的，最后在catch里捕获消息并弹窗提示，我不知道这种做法是否好，求高手指点:)(引入解释性变量)"MAC") > - 1) && (browser .ToUpper(

10、).IndexOf(0)"IE") > - 1)5n troduce Explai ning Variable 样例代码：if(platform.ToUpper().l ndexOf(&& was In itialized() && resize > / do something重构为：bool isMacOs = platform.ToUpper().I ndexOf( "MAC") > - 1;bool isIE = browser .ToUpper().IndexOf( "IE"

11、) > - 1;bool wasResized = resize >0;if(isMacOs && isIE && wasI ni tialized() && wasResized) / do something这种重构手法的目的显而易见，不多叙述了。6.我自己常用的代码重构片段1.关于 if 和 return/原始片段int row = DbHelperSql.ExecuteSql(sql);if(row) >1return true ;elsereturn false;/这样写可以省略一对括号int row = DbHelperSql.ExecuteSql(sql);if(row) >1return true ;return false;/逻辑是这样，直接return 一个表达式int row = DbHelperSql.ExecuteSql(sql);return (row >1);/ 应用"Replace Temp with Query" 手法return DbHelperSql.ExecuteSql(sql) >1;2. 二目表达式/如果碰到非