09 | 代码现代化：如何将一个300行的方法重构为3行？

原来《重构》第二版中介绍了这么多代码的坏味道，还没看过，立马买了一本。 目前掌握的发现坏味道的知识大部分来自郑烨老师的专栏《软件设计之美》、《代码整洁之道》、《修改代码的艺术》 现在又多了一个来源：跟着姚琪琳老师学习发现代码的坏味道 如果看过专栏《徐昊 · TDD项目实战70讲》的1~4课后，再看本篇文章会有一种风生水起的感觉！ 之前一直看徐昊老师 inline，不懂其奥义，今天才明白：原来在重构的时候经过 inline 的代码可以摆脱调用方法的限制，想移动到哪都可以

老师，我有一个疑问，最终重构后Performance类看起来也不是单一的吧，calculateAmount和calculateCredits各有一个修改的原因，但感觉amount和credits的计算又应该放到Performance类里，很矛盾，那怎么理解呢？ public class Performance { private int audience; private Play play; public int calculateAmount() { int thisAmount = 40000; if (audience > 30) { thisAmount += 1000 * (audience - 30); } return thisAmount; } public int calculateCredits() { int thisCredits = Math.max(audience - 30, 0); if ("comedy".equals(play.type)) thisCredits += Math.floor((double) audience / 5); return thisCredits; } }

确实挺过瘾的，我不知道自己看了多少遍了，每次看完都身心愉悦，太值了！哪怕吵架了心情不好了，也可以看，O(∩_∩)O哈哈~ public String print(Invoice invoice) { int totalAmount = invoice.calculateTotalAmount(); int volumeCredits = invoice.calculateVolumeCredits(); return resultFormatter.getResult(invoice, totalAmount, volumeCredits); } 到这一步，ResultFormatter对象的getResult方法对Invoice对象也有依恋情结，同样按下F6就移动Invoice了。看起来就更清爽，三个步骤完成： public String print(Invoice invoice) { int totalAmount = invoice.calculateTotalAmount(); int volumeCredits = invoice.calculateVolumeCredits(); return invoice.format(totalAmount, volumeCredits); }

老师能不能也提提非面向对象的实现方式。

老师从拆分阶段，到引入方法对象，然后到消消乐消除了三个方法对象（方法都到Performance和Invoice对象上了），给我们展示了整个重构过程。就本例代码而言，拆分阶段完成后，就可以按F6用IDEA的Move Instance Method重构手法，将仍发散在TheatricalPlayers类里的私有方法分别移动到Performance和Invoice对象上。

方法对象&重构发到领域模型，受益匪浅~

让我明天再看看一遍

这一篇其实没有什么标准的模式，在不同的项目下，其 处理的步骤可能完全不一样，一样的是处理问题的思路，老师提供了这样一个契机展现自己处理问题的想法，但是可能每个人的想法在这个契机上并不是一样的。其实老师说的这些大部分人都会接触过，但是缺乏一个理论性的总结过程，在真实面对的时候，总是觉得不够自信面对，感觉缺少了什么，其实这就是我们所欠缺以及需要积累的。