開發(asp.net程序)把寫代碼寫至最有面向對象味道

前幾天，搬房子時又拿起《重構----改善既有代碼的設計》這本書來隨便翻來看下，重構Refactoring在開發時，是時常也經常會使用得到。她確實教我們怎樣把寫程序寫簡潔，清楚 好明白，好理解，好維護......今天為博客寫點重構相關。以網友實際的例子，在論壇上看到的：

public void show2(string day)    {        string[] str = new string[] { "09:00", "11:00", "12:00", "13:00", "14:00", "15:00", "16:00" };        for (int i = 0; i < str.Length; i++)        {            string sql1 = "select position from cwsz where day='" + day + "' and sjd='" + str[i] + "'";            var dt1 = db.ExecuteQuery(sql1);            if (dt1.Rows.Count > 0)            {                if (dt1.Rows[0]["position"] != null)                {                    if (str[i] == "09:00")                    {                        if (dt1.Rows[0]["position"].ToString() == "3")                            dropnine.SelectedIndex = 1;                        else if (dt1.Rows[0]["position"].ToString() == "4")                            dropnine.SelectedIndex = 2;                        else if (dt1.Rows[0]["position"].ToString() == "5")                            dropnine.SelectedIndex = 3;                        else                            dropnine.SelectedIndex = 4;                    }                    if (str[i] == "10:00")                    {                        if (dt1.Rows[0]["position"].ToString() == "3")                            dropten.SelectedIndex = 1;                        else if (dt1.Rows[0]["position"].ToString() == "4")                            dropten.SelectedIndex = 2;                        else if (dt1.Rows[0]["position"].ToString() == "5")                            dropten.SelectedIndex = 3;                        else                            dropten.SelectedIndex = 4;                    }                    if (str[i] == "11:00")                    {                        if (dt1.Rows[0]["position"].ToString() == "3")                            dropele.SelectedIndex = 1;                        else if (dt1.Rows[0]["position"].ToString() == "4")                            dropele.SelectedIndex = 2;                        else if (dt1.Rows[0]["position"].ToString() == "5")                            dropele.SelectedIndex = 3;                        else                            dropele.SelectedIndex = 4;                    }                    if (str[i] == "12:00")                    {                        if (dt1.Rows[0]["position"].ToString() == "3")                            droptw.SelectedIndex = 1;                        else if (dt1.Rows[0]["position"].ToString() == "4")                            droptw.SelectedIndex = 2;                        else if (dt1.Rows[0]["position"].ToString() == "5")                            droptw.SelectedIndex = 3;                        else                            droptw.SelectedIndex = 4;                    }                    if (str[i] == "13:00")                    {                        if (dt1.Rows[0]["position"].ToString() == "3")                            tropthir.SelectedIndex = 1;                        else if (dt1.Rows[0]["position"].ToString() == "4")                            tropthir.SelectedIndex = 2;                        else if (dt1.Rows[0]["position"].ToString() == "5")                            tropthir.SelectedIndex = 3;                        else                            tropthir.SelectedIndex = 4;                    }                    if (str[i] == "14:00")                    {                        if (dt1.Rows[0]["position"].ToString() == "3")                            dropfour.SelectedIndex = 1;                        else if (dt1.Rows[0]["position"].ToString() == "4")                            dropfour.SelectedIndex = 2;                        else if (dt1.Rows[0]["position"].ToString() == "5")                            dropfour.SelectedIndex = 3;                        else                            dropfour.SelectedIndex = 4;                    }                }            }        }    }

剛開始看到這段代碼時，一眼看上去就發現用戶寫得很相同的代碼，即是冗余。用了很多if...else...。這些都是在書所提及需要重構的問題點。分幾步來重構，先來解決代碼冗余的部分，從if最內層開始，Insus.NET已經把不同的標記出來了，看下圖：

把相同的代碼寫一個方法，以后一旦下拉列表的SelectedIndex的維護，只來這個方法修改即可，不必去每一個if內檢查，或是查找替換。然后我們就可以去修改上抽離的代碼,Insus.NET把重構部分comment掉：

現在我們把comment的代碼刪除，變成下面這個樣子，代碼簡潔多了：

但是上面經首次重構之后，if還是太多，學習過設計模式的話，看到這部分程序，自然想起使用《設計模式--中介(Mediator)模式》http://www.49028c.com/insus/p/4134383.html中介者模式來解決。只管傳入str[i]值進去即可，用戶不必明白它是怎樣處理與判斷，達到封裝效果：

運用設計模式來進行重構：最后一個PRivate int SetSelectIndex(string position)方法，我們也把它移至中介者類別中去了：所留給用戶看到的代碼，即是（刪除comment）：以面向對象編程來說，上面高亮兩行代碼，應該它來自一個對象，不管你的程序是使用存儲過程還是寫SQL語句，我們盡量不要出現在xxx.aspx.cs代碼頁中：

這樣子，在xxx.aspx.cs代碼，就可以改為：

現在我們回過頭去看中介者SwitchObject類，它最底的私有方法，其實使用了switch方法，在重構之中，也是一個很壞的使用。其實switch即是替代多個if判斷。出現多個if時，我們又可以使用中介者模式來解決。

創建一個中介者類：再來看看Insus.NET怎樣對SwitchObject類修改：重構到這里，整段程序也改得差不多了，但是在SwitchObject類中，還是有些代碼是很不穩定的。有可能常被修改，因此，我們還是把它移走。只留下穩定代碼：

移到哪里，Insus.NET想把它移至最原始的位置，在移動之前Select()方法，需要小修改一下，期望傳入另外一個參數：

回至xxx.aspx.cs的public void show2(string day)方法：

重構到這里，你還是看到有一個if判斷：

似乎改得有點問題，正確來說，當傳入的position為空時，它等于4。而不是獲取的selectedIndex不等于4時等于4。

重構完畢，最終源程序可從下面鏈接下載：http://download.VEVb.com/insus//Refactoring/RefactoringAndMediator_pattern.rar