【轉】編寫高品質代碼改善C#程式的157個建議——建議143：方法抽象層級應在同一層次

標籤：重構   封裝   代碼   程式   一個   public   style   post   編寫高品質代碼   

 

建議143：方法抽象層級應在同一層次

看下面代碼：

    class SampleClass    {        public void Init()        {            //本地初始化代碼1            //本地初始化代碼2            RemoteInit();        }        void RemoteInit()        {            //遠程初始化代碼1            //遠程初始化代碼2        }    }

 

 Init方法本意要完成初始化動作，而初始化包括本地初始化和遠程初始化。這段代碼中，Init方法內部代碼的組織圖是本地初始化直接運行在方法內部，而遠程初始化代碼卻被封裝為一個方法在這裡被調用。這顯然是不妥當的，應為本地初始化和遠程初始化的地位是相當的。如果遠程初始化代碼作為方法存在，則本地初始化代碼也應該作為方法存在。

所以，上面代碼應該重構為：

    class SampleClass    {        public void Init()        {            LocalInit();            RemoteInit();        }        void LocalInit()        {            //本地初始化代碼1            //本地初始化代碼2           }        void RemoteInit()        {            //遠程初始化代碼1            //遠程初始化代碼2        }    }

 

 重構後的代碼看上去清晰明了，所有的方法的抽象層級都在一個層次上，使讀者一眼看上去就知道Init方法完成了什麼樣的功能。

 

 

轉自：《編寫高品質代碼改善C#程式的157個建議》陸敏技

【轉】編寫高品質代碼改善C#程式的157個建議——建議143：方法抽象層級應在同一層次