重构既有代码.pptVIP

下载本文档

1
0
约6.41千字
约 17页
2017-05-28 发布于四川
举报
版权申诉

重构既有代码.ppt

1、原创力文档（book118）网站文档一经付费（服务费），不意味着购买了该文档的版权，仅供个人/单位学习、研究之用，不得用于商业用途，未经授权，严禁复制、发行、汇编、翻译或者网络传播等，侵权必究。。
2、本站所有内容均由合作方或网友上传，本站不对文档的完整性、权威性及其观点立场正确性做任何保证或承诺！文档内容仅供研究参考，付费前请自行鉴别。如您付费，意味着您自己接受本站规则且自行承担风险，本站不退款、不进行额外附加服务；查看《如何避免下载的几个坑》。如果您已付费下载过本站文档，您可以点击这里二次下载。
3、如文档侵犯商业秘密、侵犯著作权、侵犯人身权等，请点击“版权申诉”（推荐），也可以打举报电话：400-050-0827(电话支持时间：9:00-18:30)。
4、该文档为VIP文档，如果想要下载，成为VIP会员后，下载免费。
5、成为VIP后，下载本文档将扣除1次下载权益。下载后，不支持退款、换文档。如有疑问请联系我们。
6、成为VIP后，您将拥有八大权益，权益包括：VIP文档下载权益、阅读免打扰、文档格式转换、高级专利检索、专属身份标志、高级客服、多端互通、版权登记。
7、VIP文档为合作方或网友上传，每下载1次，网站将根据用户上传文档的质量评分、类型等，对文档贡献者给予高额补贴、流量扶持。如果你也想贡献VIP文档。上传文档

坏味道 public int discount( int price ) { if( price 1000) { price -= 20; …….. } return price; } public int discount( final int price ) { int ret = price; if( price 1000) { ret -= 20; …….. } return ret ; } 传值参数不能用赋值，职责不明确，应用创建临时变量对其操作坏味道 double temp = 2 * (_height + _width); System.out.println (temp); temp = _height * _width; System.out.println (temp); double perimeter = 2 * (_height + _width); System.out.println (perimeter); double area = _height * _width; System.out.println (area); 职责不明确 ColdRule *newRule = new ColdRule(); newRule-SetOID(oldRule-GetOID()); newRule-SetRegion(oldRule-GetRegion()); newRule-SetRebateRuleID(oldRule-GetRebateRuleID()); newRule-SetBeginCycle(oldRule-GetBeginCycle() + 1); newRule-SetEndCycle(oldRule-GetEndCycle()); newRule-SetMainAcctAmount(oldRule-GetMainAcctAmount()); newRule-SetGiftAcctAmount(oldRule-GetGiftAcctAmoun t()); newRule-SetValidDays(0); newRule-SetGiftAcct(oldRule-GetGiftAcct()); rules-Add(newRule); ColdRule* CreateNewRule(ColdRule oldRule) { ColdRule *newRule = new ColdRule(); newRule-SetOID(oldRule.GetOID()); newRule-SetRegion(oldRule.GetRegion()); newRule-SetRebateRuleID(oldRule.GetRebateRuleID()); newRule-SetBeginCycle(oldRule.GetBeginCycle() + 1); newRule-SetEndCycle(oldRule.GetEndCycle()); newRule-SetMainAcctAmount(oldRule.GetMainAcctAmount()); newRule-SetGiftAcctAmount(oldRule.GetGiftAcctAmount()); newRule-SetValidDays(0); newRule-SetGiftAcct(oldRule.GetGiftAcct()); return newRule; } ….. rules-Add(CreateNewRule(*oldRule)); 引入解释性变量 boolean isMacOs = platform.toUpperCase().indexOf(MAC) -1; boolean isIEBrowser = browser.toUpperCase().indexOf(IE) -1; boolean wasResized = resize 0; if (isMacOs isIEBrowser wasInitialized() wasResized) { // do something } if ( (platform.toUpperCase().indexOf(MAC) -1) (browser.toUpperCase().indexOf(IE) -1) wasInitialized() resize 0 ) { // do some