关闭

软件开发—重构

发表于:2014-5-09 11:16

字体: | 上一篇 | 下一篇 | 我要投稿

 作者:cmmi    来源:51Testing软件测试网采编

  重构是对软件内部结构的一种调整,目的是在不改变软件之可察性前提下,提高其可理解性,降低其修改成本。关于重构的至理明言如下:
  任何一个傻瓜都能写出计算器可以理解的代码,唯有写出人类容易理解的代码,才是优秀的程序员;
  事不过三,三则重构;
  当你接获bug提报,请先撰写一个单元测试来揭发这个bug;
  当你感觉需要撰写注释,请先尝试重构,试着让所有的注释变得多余;
  当你发现自己需要为程序增加一个特性,而代码结构使你无法方便的这样做,就先重构那个程序;
  重构之前,必须建立一套可靠的测试机制;
  写软件就像种树,优秀的程序员挖成小坑后随及填好,继续挖下一个,只会产生一系列小坑,不会有大坑,菜鸟则不会意识到所挖的坑正在变大,还是不停的挖,直到自己掉进大坑,爬不出来,陷入无尽的痛苦深渊;
  开发时间越长,越能体会垃圾代码的痛苦,却不知道如何改进;
  Kent Beck:我不是一个伟大的程序员,我只是个有着一些优秀习惯的好程序员而已;
  变量(Variable)
  不要定义一个临时变量多次重复使用,临时变量定义仍然应该可以自解释,从变量名称能够很好的理解变量的含义和作用。在定义一个临时变量后需要有一段业务逻辑才能够完成对临时变量的赋值的时候,可以考虑将这段逻辑抽取到一个独立的方法。
doublegetPrice(){
int basePrice = _quantity* _itemPrice;
double discountFactor;
if (basePrice > 1000) discountFactor = 0.95;
else discountFactor = 0.98;
return basePrice * discountFactor;
}
  重构为:
double getPrice(){
return basePrice()* discountFactor();
}
private int basePrice(){
return _quantity* _itemPrice;
}
private double discountFactor(){
if (basePrice()> 1000) return0.95;
else return 0.98;
}
  当遇到复杂的表达式的时候,需要引入解释变量,因为复杂的表达式很难进行自解释。
if ((platform.toUpperCase().indexOf("MAC")> -1)&&
(browser.toUpperCase().indexOf("IE")> -1)&&
wasInitialized()&& resize> 0 )
{
// do something
}
  重构为:
final booleanisMacOs    = platform.toUpperCase().indexOf("MAC")>-1;
final boolean isIEBrowser =browser.toUpperCase().indexOf("IE") > -1;
final booleanwasResized  = resize >0;
if (isMacOs&& isIEBrowser&& wasInitialized()&& wasResized){
// do something
}
21/212>
《2023软件测试行业现状调查报告》独家发布~

关注51Testing

联系我们

快捷面板 站点地图 联系我们 广告服务 关于我们 站长统计 发展历程

法律顾问:上海兰迪律师事务所 项棋律师
版权所有 上海博为峰软件技术股份有限公司 Copyright©51testing.com 2003-2024
投诉及意见反馈:webmaster@51testing.com; 业务联系:service@51testing.com 021-64471599-8017

沪ICP备05003035号

沪公网安备 31010102002173号