前幾天，搬房子時(shí)又拿起《重構(gòu)----改善既有代碼的設(shè)計(jì)》這本書來隨便翻來看下，重構(gòu)在開發(fā)時(shí)，是時(shí)常也經(jīng)常會(huì)使用得到。
她確實(shí)教我們怎樣把寫程序?qū)懞啙崳宄?好明白，好理解，好維護(hù)......今天為博客寫點(diǎn)重構(gòu)相關(guān)。以網(wǎng)友實(shí)際的例子，在論壇上看到的：
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; } } } } }View Code剛開始看到這段代碼時(shí)，一眼看上去就發(fā)現(xiàn)用戶寫得很相同的代碼，即是冗余。用了很多if...else...。這些都是在書所提及需要重構(gòu)的問題點(diǎn)。分幾步來重構(gòu)，先來解決代碼冗余的部分，從if最內(nèi)層開始，Insus.NET已經(jīng)把不同的標(biāo)記出來了，看下圖：
把相同的代碼寫一個(gè)方法，以后一旦下拉列表的SelectedIndex的維護(hù)，只來這個(gè)方法修改即可，不必去每一個(gè)if內(nèi)檢查，或是查找替換。
然后我們就可以去修改上抽離的代碼,Insus.NET把重構(gòu)部分comment掉：
現(xiàn)在我們把comment的代碼刪除，變成下面這個(gè)樣子，代碼簡潔多了：
但是上面經(jīng)首次重構(gòu)之后，if還是太多，學(xué)習(xí)過設(shè)計(jì)模式的話，看到這部分程序，自然想起使用《設(shè)計(jì)模式--中介(Mediator)模式》http://www.companysz.com/insus/p/4134383.html中介者模式來解決。只管傳入str[i]值進(jìn)去即可，用戶不必明白它是怎樣處理與判斷，達(dá)到封裝效果：
運(yùn)用設(shè)計(jì)模式來進(jìn)行重構(gòu)：
最后一個(gè)PRivate int SetSelectIndex(string position)方法，我們也把它移至中介者類別中去了：所留給用戶看到的代碼，即是（刪除comment）：
以面向?qū)ο缶幊虂碚f，上面高亮兩行代碼，應(yīng)該它來自一個(gè)對象，不管你的程序是使用存儲過程還是寫SQL語句，我們盡量不要出現(xiàn)在xxx.aspx.cs代碼頁中：
這樣子，在xxx.aspx.cs代碼，就可以改為：
現(xiàn)在我們回過頭去看中介者SwitchObject類，它最底的私有方法，其實(shí)使用了switch方法，在重構(gòu)之中，也是一個(gè)很壞的使用。其實(shí)switch即是替代多個(gè)if判斷。出現(xiàn)多個(gè)if時(shí)，我們又可以使用中介者模式來解決。
創(chuàng)建一個(gè)中介者類：
再來看看Insus.NET怎樣對SwitchObject類修改：
重構(gòu)到這里，整段程序也改得差不多了，但是在SwitchObject類中，還是有些代碼是很不穩(wěn)定的。有可能常被修改，因此，我們還是把它移走。只留下穩(wěn)定代碼：
移到哪里，Insus.NET想把它移至最原始的位置，在移動(dòng)之前Select()方法，需要小修改一下，期望傳入另外一個(gè)參數(shù)：
回至xxx.aspx.cs的public void show2(string day)方法：
重構(gòu)到這里，你還是看到有一個(gè)if判斷：
似乎改得有點(diǎn)問題，正確來說，當(dāng)傳入的position為空時(shí)，它等于4。而不是獲取的selectedIndex不等于4時(shí)等于4。
重構(gòu)完畢，最終源程序可從下面鏈接下載：http://download.VEVb.com/insus//Refactoring/RefactoringAndMediator_pattern.rar
新聞熱點(diǎn)
疑難解答
圖片精選
