二、通过TDD进行重构
“ 发现价值”的过程远远没有结束。通过测试代码，我们从客户的角度来考虑，会发现一些问题。在已经实现的代码中，SmartAssistor类型实现了 Search，Store和List的功能。但这些职责是否真的应该由它承担呢？表面上来看，是这样的。然而根据OO的思想来看，这个 SmartAssistor所承担的责任是否太多了？它和搜索的结果、显示的方式耦合度是否太紧密了？这个设计将实现抽象出来了吗？这些都应该是我们考虑的重点。考虑的时机，可以是设计之初，也可以是重构之时。
在重构的时候，仍然不能放弃TDD，只有它才能保证程序的可靠性，重构的正确性。开始重构吧。
首先从行为来考虑。搜索的功能会很复杂吗？可能会有精确搜索，模糊搜索；可能是在网上搜索，也可能是本机搜索。那么，存储的功能呢？IO的操作是否频繁，存储的要求是否会根据安全级别而逐步升级？再考虑显示，对于个人智能助理来讲，显示的方式需要多样化吗？显然，以上的行为都是复杂的。
再从抽象性考虑。需要把这些行为抽象出来吗？也就是说，这些行为的载体是否会有多种类型？显然，搜索可能会是文件的搜索，可能会是文本的搜索，也可能会是数据库的搜索；存储的格式也会有多种多样，文本文件，xml文件，数据库文件。显示的方式可能会通过浏览器显示，也可能会在WinForm中显示。也许用户要求是带滚动条的文本框，也许只是简单的文本显示。对象的形式很多吧，需要抽象吗？显然是的！
这样考虑之后，我发觉需要重构的东西太多了，应该怎么入手？首先，我们把SmartAssistor的职责先剥离出来，用更单一的对象来完成各自的功能。例如我们可以引入SmartEngine来管理搜索功能。然后，分别将这些对象提炼出各自的接口。还是先写测试代码吧，考虑搜索功能，首先需要将对象分离出来：
[Test] public void TestSearching() { SearchEngine engine = new SearchEngine(); Assert.IsNotNull(engine); SearchResult result1 = new SearchResult(); SearchResult result2 = new SearchResult(); Assert.IsNotNull(result1); Assert.IsNotNull(result2); result1 = engine.ExactSearch(control.Categaries); result2 = engine.BlurSearch(control.Categaries); SearchResult tempResult1 = new SearchResult(control.Categaries,"contents"); SearchResult tempResult2 = new SearchResult(control.Categaries,"more contents"); Assert.AreEqual(tempResult1,result1); Assert.AreEqual(tempResult2,result2); } |
在NUnit中运行测试代码，未能通过。然后在程序中创建SearchEngine类型，并实现ExactSearch和BlurSearch方法。直到在NUnit中运行通过，全部显示绿灯。
接下来抽象出SearchEngine的接口ISearchEngine，并让SearchEngine实现该接口。其中接口方法包括ExactSearch和BlurSearch方法。将前面的测试代码作小小的修改，修改后同样需要在NUnit中运行，保证顺利通过：
[Test] public void TestSearching() { ISearchEngine engine = new SearchEngine(); Assert.IsNotNull(engine); …… } |
考察SearchEngine类型，该类型的对象应该在整个程序中只保留一个对象，因此，应对此采用单例模式。修改测试代码：
[Test] public void TestSearching() { ISearchEngine engine = SearchEngine.Instance; Assert.IsNotNull(engine); …… } |
