现在的位置: 首页 > 综合 > 正文

必会重构技巧(二):使用多态替换条件

2012年01月31日 ⁄ 综合 ⁄ 共 3605字 ⁄ 字号 评论关闭

 

使用多态替换条件:指在进行类型检查和执行某些类型操作时,最好将算法封装在类中,并且使用多态来对代码中的调用进行抽象
举例理解:看定义可能比较迷糊,其实说的简单一点,对于使用分支语句并且分支条件是和类型检查相关的程序段,如 if(type == typeof(TypeA)){...}else if(type == typeof(TypeB)){...},可以把{...}中的Code,尝试放到if的条件中去。然后通过检查Type就可以直接返回需要的东东了,这样做可以利用已有的继承层次进行计算,比较便于维护。如果还是觉得说的太抽象,可以看看下面的代码感觉一下。
项目实例:用WPF做一个网游的客户端Demo,里面需要对商品,邮件,物品栏做分页操作。于是手动写了几个分页的类。开始是把分页的计算方法都写在了事件里面的,每一个Button绑定一个事件,每次需要修改或者使用分页的时候,都要找到相关类进行修改,复制,各个方法的耦合程度大增,程序可读性,复用性和可维护性都不太好。虽然这个项目是很久之前做的了,但这里既然想起来了,觉得还是可以尝试用这种重构方法,效果如何大家自己看看吧。
先来看看原始的未经过重构的代码:

  使用多态替换条件:指在进行类型检查和执行某些类型操作时,最好将算法封装在类中,并且使用多态来对代码中的调用进行抽象

  举例理解:看定义可能比较迷糊,其实说的简单一点,对于使用分支语句并且分支条件是和类型检查相关的程序段,如 if(type == typeof(TypeA)){...}else if(type == typeof(TypeB)){...},可以把{...}中的Code,尝试放到if的条件中去。然后通过检查Type就可以直接返回需要的东东了,这样做可以利用已有的继承层次进行计算,比较便于维护。如果还是觉得说的太抽象,可以看看下面的代码感觉一下。

  项目实例:用WPF做一个网游的客户端Demo,里面需要对商品,邮件,物品栏做分页操作。于是手动写了几个分页的类。开始是把分页的计算方法都写在了事件里面的,每一个Button绑定一个事件,每次需要修改或者使用分页的时候,都要找到相关类进行修改,复制,各个方法的耦合程度大增,程序可读性,复用性和可维护性都不太好。虽然这个项目是很久之前做的了,但这里既然想起来了,觉得还是可以尝试用这种重构方法,效果如何大家自己看看吧。
  先来看看原始的未经过重构的代码:

 

原始代码

//x:目标页数索引值 y:每页显示记录个数 这里不Care你是如何取到这几个值的和对这两个Int值合法性的验证
protected void btnFirstPage_Click(object sender, EventArgs e)
{
Button btn
= sender as Button;
if (btn != null)
{
repDataList.DataSource
= Getdata(x,y); //Get Date from DB
}
}
protected void btnPrePage_Click(object sender, EventArgs e)
{
Button btn
= sender as Button;
if (btn != null)
{
repDataList.DataSource
= Getdata(x, y); //Get Date from DB
}
}
protected void btnNextPage_Click(object sender, EventArgs e)
{
Button btn
= sender as Button;
if (btn != null)
{
repDataList.DataSource
= Getdata(x, y); //Get Date from DB
}
}
protected void btnlastPage_Click(object sender, EventArgs e)
{
Button btn
= sender as Button;
if (btn != null)
{
repDataList.DataSource
= Getdata(x, y); //Get Date from DB
}
}
private DataSet Getdata(int targetPageIndex, int numberPerPage) //Get Date from DB
{
throw new NotImplementedException(); //Here some code to retrive Data from DB
}

 

  上面的几段代码可以实现功能,但是却存在以下几点问题。
    (1)如果有多个页面需要使用分页,同样的代码需要复制多次,复用性差
    (2)第一个问题造成了第二个问题,如果需要修改某段方法,那所有的相关页面都要修改,可维护性差
    (3)相似逻辑的方法写在了多个事件中,可读性差
  为了解决这三个问题,现在重构后的代码如下:
重构后

public abstract class Paging
{
public int TargetPageIndex { get; set; }
public int NumberPerPage { get; set; }
public abstract DataSet DataList { get; }
}
public class FirstPage : Paging
{
public FirstPage(int targetPageIndex, int numberPerPage)
{
TargetPageIndex
= targetPageIndex;
NumberPerPage
= numberPerPage;
}
public override DataSet DataList
{
get
{
return Getdata(TargetPageIndex, NumberPerPage);
}
}
}
public class PrePage : Paging
{
public PrePage(int targetPageIndex, int numberPerPage)
{
TargetPageIndex
= targetPageIndex;
NumberPerPage
= numberPerPage;
}
public override DataSet DataList
{
get
{
return Getdata(TargetPageIndex, NumberPerPage);
}
}
}
public class NextPage : Paging
{
public NextPage(int targetPageIndex, int numberPerPage)
{
TargetPageIndex
= targetPageIndex;
NumberPerPage
= numberPerPage;
}
public override DataSet DataList
{
get
{
return Getdata(TargetPageIndex, NumberPerPage);
}
}
}
public class LastPage : Paging
{
public LastPage(int targetPageIndex, int numberPerPage)
{
TargetPageIndex
= targetPageIndex;
NumberPerPage
= numberPerPage;
}
public override DataSet DataList
{
get
{
return Getdata(TargetPageIndex, NumberPerPage);
}
}
}

 

以上代码可以独立出一个Paging的类,略加修改就可以对应任意的DataSource。

前台调用代码

protected void btnPaging_Click(object sender, EventArgs e)
{
Button btn
= sender as Button;
if (btn == null) return;
Paging page;
switch (btn.CommandArgument)
{
case "FirstPage":
page
= new FirstPage(x,y);
repDataList.DataSource
= page.DataList;
break;
case "PrePage":
page
= new PrePage(x, y);
repDataList.DataSource
= page.DataList;
break;
case "NextPage":
page
= new NextPage(x, y);
repDataList.DataSource
= page.DataList;
break;
case "LastPage":
page
= new LastPage(x, y);
repDataList.DataSource
= page.DataList;
break;

}
}

 

个人感觉,对于本例中的应用,好处并不是很明显,也许是我写的有问题,也许是我选择例子有问题。尽管如此,重构的思想体现出来了,那就是把算法封装到多态中。此种重构在对于含类型判断条件的复杂算法分支的应用上,效果还是比较显著的。

抱歉!评论已关闭.