31天重構學習筆記29. 去除中間人對象

摘要：由於最近在做重構的項目，所以對重構又重新進行了一遍學習和整理，對31天重構最早接觸是在2009年 10月份，由於當時沒有訂閱Sean Chambers的blog，所以是在國外的社區上閒逛的時候鏈接過去的。記得當時一口氣看完了整個系列並沒有多少感覺，因爲這些基本上項目都在使用，只是我們沒有專門把它標示和整理出來，所以也沒有引起多大的重視。現在突然接手這個重構項目，由於團隊成員技術和經驗參差不齊，所以有必要專門整理一個重構的綱要，當然這個系列也非常適合做新系統的代碼規範參考，只要有代碼的地方，這個重構規範就很有價值。週末也不想出去閒逛，因爲在剛到這個美麗的城市，沒有親戚或者朋友，所以才能靜下心來兩天時間寫完這個重構參考規範。同時也感受了Windows Live writer寫文章的快感。當然重構的整體架構得另當別論（整體架構在我的這篇文章有專門的講解（http://www.cnblogs.com/zenghongliang/archive/2010/06/23/1763438.html）。大的架構設計好了以後，這些重構細節點就成了東風之後的大火，對整個項目也是至關重要。31天重構這個系列和《代碼大全》、《重構：改善既有代碼的設計》比較起來最大的特點就是比較簡單、淺顯易懂。那麼我這些文章也都是學習Sean Chambers的31天重構的筆記整理，所以如果大家對這個筆記有任何異議也可以指出。

具體也可以通過http://www.lostechies.com/blogs/sean_chambers/archive/2009/07/31/31-days-of-refactoring.aspx查看原文。

 

概念：本文中的”去除中間人對象”是指把 在中間關聯而不起任何其他作用的類移除，讓有關係的兩個類直接進行交互。

 

正文：有些時候在我們的代碼會存在一些”幽靈類“，設計模式大師Fowler稱它們爲“中間人”類，“中間人”類除了調用別的對象之外不做任何事情，所以“中間人”類沒有存在的必要，我們可以將它們從代碼中刪除，從而讓交互的兩個類直接關聯。

如下代碼所示，Consumer 類要得到AccountDataProvider 的數據，但中間介入了沒起任何作用的 AccountManager 類來關聯，所以我們應當移除。

using LosTechies.DaysOfRefactoring.PullUpField.After;

namespace LosTechies.DaysOfRefactoring.SampleCode.RemoveMiddleMan.Before
{
    public class Consumer
    {
        public AccountManager AccountManager { get; set; }

        public Consumer(AccountManager accountManager)
        {
            AccountManager = accountManager;
        }

        public void Get(int id)
        {
            Account account = AccountManager.GetAccount(id);
        }
    }

    public class AccountManager
    {
        public AccountDataProvider DataProvider { get; set; }

        public AccountManager(AccountDataProvider dataProvider)
        {
            DataProvider = dataProvider;
        }

        public Account GetAccount(int id)
        {
            return DataProvider.GetAccount(id);
        }
    }

    public class AccountDataProvider
    {
        public Account GetAccount(int id)
        {
            // get account
        }
    }
}

 

重構後的代碼如下所示，Consumer 和AccountDataProvider 直接進行關聯，這樣代碼就簡單了。 

using LosTechies.DaysOfRefactoring.PullUpField.After;

namespace LosTechies.DaysOfRefactoring.SampleCode.RemoveMiddleMan.After
{
    public class Consumer
    {
        public AccountDataProvider AccountDataProvider { get; set; }

        public Consumer(AccountDataProvider dataProvider)
        {
            AccountDataProvider = dataProvider;
        }

        public void Get(int id)
        {
            Account account = AccountDataProvider.GetAccount(id);
        }
    }

    public class AccountDataProvider
    {
        public Account GetAccount(int id)
        {
            // get account
        }
    }
}

 

總結： ”去除中間人對象“很多時候都會很有作用，尤其是在誤用設計模式的代碼中最容易見到，設計模式中的適配器模式和代理模式等都用中間的類是兩者進行關聯，這是比較合理的，因爲中間類做了很多事情，而對於沒有任何作用的中間類應該移除。