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