写代码写至最有面向对象味道
前几天,搬房子时又拿起《重构----改善既有代码的设计》这本书来随便翻来看下,重构在开发时,是时常也经常会使用得到。
她确实教我们怎样把写程序写简洁,清楚 好明白,好理解,好维护......
今天为博客写点重构相关。以网友实际的例子,在论坛上看到的:
![](https://images.cnblogs.com/OutliningIndicators/ExpandedBlockStart.gif)
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; } } } } }
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代码页中:
现在我们回过头去看中介者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