二、通过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); …… } |