開發(ASP.NET程式)把寫程式碼寫至最有面向物件味道
前幾天，搬房子時又拿起《重構----改善既有程式碼的設計》這本書來隨便翻來看下，重構在開發時，是時常也經常會使用得到。
她確實教我們怎樣把寫程式寫簡潔，清楚 好明白，好理解，好維護......
今天為部落格寫點重構相關。以網友實際的例子，在論壇上看到的：
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還是太多，學習過設計模式的話，看到這部分程式，自然想起使用《
運用設計模式來進行重構：
最後一個private int SetSelectIndex(string position)方法，我們也把它移至中介者類別中去了：
所留給使用者看到的程式碼，即是（刪除comment）：
以面向物件程式設計來說，上面高亮兩行程式碼，應該它來自一個物件，不管你的程式是使用儲存過程還是寫SQL語句，我們儘量不要出現在xxx.aspx.cs內碼表中：
現在我們回過頭去看中介者SwitchObject類，它最底的私有方法，其實使用了switch方法，在重構之中，也是一個很壞的使用。其實switch即是替代多個if判斷。出現多個if時，我們又可以使用中介者模式來解決。
建立一箇中介者類：
再來看看Insus.NET怎樣對SwitchObject類修改：
重構到這裡，整段程式也改得差不多了，但是在SwitchObject類中，還是有些程式碼是很不穩定的。有可能常被修改，因此，我們還是把它移走。只留下穩定程式碼：
移到哪裡，Insus.NET想把它移至最原始的位置，在移動之前Select()方法，需要小修改一下，期望傳入另外一個引數：
回至xxx.aspx.cs的public void show2(string day)方法：
似乎改得有點問題，正確來說，當傳入的position為空時，它等於4。而不是獲取的selectedIndex不等於4時等於4。
重構完畢，最終源程式可從下面連結下載：
http://download.cnblogs.com/insus//Refactoring/RefactoringAndMediator_pattern.rar
轉https://www.cnblogs.com/insus/p/4139346.html