學習之路三十三：重構技巧的學習

最近看了聖殿騎士大哥的重構文章，其中有幾個重構技巧讓我頗有感觸，特此記錄下。

文章地址：31天重構學習筆記重新整理下載

1.封裝集合，返回集合接口類型 

　　這個技巧讓我想起了項目中的代碼，比如：

    public class Check
    {
        private List<Detail> _details = new List<Detail>();

        public IList<Detail> GetDetails()
        {
            return _details;
        }
    }

    public class Detail
    {
        public string Name { get; set; }
    }

 　  如果這樣設計，那么用戶通過調用GetDetails方法之后就可以對你的集合做任意的改動，這樣的做法是非常危險的，所以必須要改進。

　　重構原則：返回可迭代器類型接口來保證對集合的封裝，改進如下：

    public class Check
    {
        private List<Detail> _details = new List<Detail>();

        private Dictionary<int, Detail> _specificDetail = new Dictionary<int, Detail>();

        public IEnumerable<Detail> GetDetails()
        {
            return _details;
        }

        public IEnumerable<KeyValuePair<int, Detail>> GetSpecificDetail()
        {
            return _specificDetail;
        }
    }

    public class Detail
    {
        public string Name { get; set; }
    }

　　迭代器文章入口：迭代器學習之一：使用IEnumerable和IEnumerator接口

 　  我將在項目中運用此重構技巧。

 

2.提取判斷條件作為方法 - 方法名要有意義

　　當遇到復雜的判斷后，如果沒有注釋，很多人第一眼都不會理解這個判斷是做什么的。

　　所以把復雜的判斷條件提取為方法，再取個有意義的名字，那么別人看一眼就明白做什么的，比如：

    public class Check
    {
        public Detail GetDetail { get; set; }

        public void SendMessage()
        {
            //這樣的判斷別人根本不知道是干什么的
            if (GetDetail != null && GetDetail.Number > 0)
            {
                //Send Message
            }

            //重構后的代碼
            //這樣別人就知道這個判斷的作用：判斷當前的信息有沒有提交
            if (HasConfirm(GetDetail))
            {
                //Send Message
            }
        }

        public bool HasConfirm(Detail detail)
        {
            return GetDetail != null && GetDetail.Number > 0;
        }
    }

    public class Detail
    {
        public int Number { get; set; }
    }

 

3.為帶有大量bool參數的方法重新進行拆分組合

　　這個重構手法在項目中還沒有用過，方法帶有大量的bool參數本來就很奇葩，不過如果真的遇到了，我們也有應對方法，如下：

    public class Test
    {
          //如果是這樣的一個方法，你知道它是干什么的嗎，根本就不能理解它的用意
        private void Create(bool isA, bool isB, bool isC)
        { }
    }

 　　重構后的代碼：

    public class Test
    {
        //進行拆分后在重新組合
        public void CreateA()
        {
            this.Create(true, false, false);
        }

        public void CreateB()
        {
            this.Create(false, true, false);
        }

        public void CreateC()
        {
            this.Create(false, false, true);
        }

        private void Create(bool isA, bool isB, bool isC)
        { }
    }

 

 4.避免雙重否定

　　其實這個重構手法我理解為盡量避免使用在if條件里使用 - “!”.

　　因為如果本來你的判斷條件就是表達否定的意思，那么在加上一個否定的判斷，那么就會是雙重否定。

　　那別人理解起來是不是會很抓狂，他會在心理畫個圈圈狠狠的詛咒你的，:-)，比如：

    public class Test
    {
        public void TestOne()
        {
            //本來想表達已經付款了，所以只能加個否定來判斷咯
            //可是這個交別人理解起來真的很抓狂
            if (!NoPayment)
            {
                //TO DO
            }
        }

        public bool NoPayment { get; set; }
    }

 　　重構后的代碼：

    public class Test
    {
        public void TestOne()
        {
            //本來想表達已經付款了，所以只能加個否定來判斷咯
            //可是這個交別人理解起來真的很抓狂
            if (!NoPayment)
            {
                //TO DO
            }

            //重構后的代碼
            //避免使用雙重否定來作為判斷條件
            if (HasPayment)
            {

            }
        }

        public bool NoPayment { get; set; }

        public bool HasPayment { get; set; }
    }

 

5.盡快返回 - 返回值

　　這個重構技巧可以分解復雜的判斷條件，是那些冗余的判斷分支化成簡介的小分支，比如：

    public class TestOne
    {
        private List<Detail> _details;

        public void Display()
        {
            //這么一大長串的判斷條件是不是很惡心
            if (_details != null)
            {
                if (_details.Count > 0)
                {
                    foreach (var item in _details)
                    {
                        if (item.Number > 0)
                        {
                            //TO DO
                        }
                    }
                }
            }
        }
    }

    public class Detail
    {
        public int Number { get; set; }
    }

 　　重構后的代碼：

    public class TestOne
    {
        private List<Detail> _details;

        public void Display()
        {
            //重構之后是不是很清爽呀
            //哈哈
            if (_details == null || _details.Count <= 0)
                return;
            foreach (var item in _details.Where(p => p.Number > 0).Select(p => p))
            {
                //TO DO
            }
        }
    }

    public class Detail
    {
        public int Number { get; set; }
    }

 

以同步至：個人文章目錄索引