在大型遗留系统基础上运作重构项目(4)

对象依赖关系复杂

通过上述重构，尤其是改为通过构造函数参数来获得依赖对象之后，在一段时期会发现各个对象的构造函数堆积了大量的参数。例如经过重构之后的UserLoginDlgHelper的构造函数签名如下：

public UserLoginDlgHelper(LoginInfoComp loginInfoComp, 
SessionManager sessionManager, 
LoginMgr loginManager, 
ResCenter resCenter, 
GuiUtil guiUtil)

共有5个依赖对象从构造函数传入，显得构造函数相当臃肿。这实际上反映出对象本身的臃肿：这个对象承担了太多责任，因此需要依赖大量其它对象。过长的构造函数参数列表是一个直观的指标，让我们能够清晰地看到对象依赖的情况。经过观察可以发现，这些依赖对象都分别只在几个方法中被用到，这意味着UserLoginDlgHelper对象本身的职责可以分为几个相对独立的方面，可以被拆分为几个独立的对象。

重构办法：

以依赖对象的使用情况为线索，把大对象拆分成多个小对象。大对象中原有的方法不删除，而是把调用委派给小对象。

逐一修改大对象的使用者，让它们使用拆分出来的小对象。

大对象中的方法无人使用时即可删除，依赖对象无人使用时即可从构造函数参数列表中删除。

重构目标：

每个对象的依赖对象数比较合理。

数据缺乏对象封装

系统中一些数据以原生类型或者简单容器的形式呈现，“对数据的操作”与数据本身脱离，散落在系统各处。这样做的坏处是不容易看清数据代表的含义和处理这些数据的逻辑。特别是在数据结构发生改变时，必须找出所有操作这些数据的地方，同时修改，一旦有遗漏就很容易引入隐晦的错误。

例如“登录信息”在系统中用Vector数据类型表示，其中各个字符串分别代表DS服务器IP、DS服务器端口、认证模式、安全模式等信息。“共有多少个字符串”和“各个位置上的字符串表示什么含义”等信息在ServerInfoTable类中以常量的形式保存：

public static class ServerInfoTable
{
public static int ColumnCount = 6;
public static int IPColumn = 0;
public static int PortColumn = 1;
public static int SecurityModeColumn = 2;
public static int LastLoginUserNameColumn = 3;
public static int IsLastLoginServerColumn = 4;
public static int AuthModeColumn = 5;

系统中共有5个类、十多个地方引用这些常量，还有一些地方直接用魔法数来访问这些信息。如果新增一列数据，需要修改的地方很多，很容易出错。

重构办法：

创建类型来封装原生数据，同时提供对象与原生数据之间的双向转换方法。

找出原生数据的“索引常量”（如前面的ServerInfoTable），以它为线索，逐个方法进行重构，把其中使用原生数据的逻辑改为使用新对象。在方法入口处把原生数据转换为对象，出口处把对象转换回原生数据。

一条调用链重构完成后，即可修改其中各个方法的签名，由传递原生数据改为传递对象。

如果某些操作不涉及其他业务对象，只是操作这组数据，把这样的操作移到新建的类中。

在调用链附近留意检查是否有不通过“索引常量”而用魔法数直接访问数据结构的情况。

重构目标：

与特定数据结构相关的描述和操作都封装在对象内部，“索引常量”和魔法数被删除。

改变数据结构只需要一处修改。

小结

大规模遗留系统的重构一直是困扰众多软件组织的难题。所谓“冰冻三尺非一日之寒”，大型系统中的质量问题是经年累月堆积下来的，要解决这些问题也只能从价值最高的地方入手，耐着性子一点点重新恢复代码质量。经过这个咨询项目，eMAN产品开发团队在原有代码的基础上划分了细粒度的重构story，建立了持续集成环境，并按照共同探索出的节奏，以测试为驱动、以代码质量为导向，不断重构以改进代码质量，并且积累了一些常见问题的解决办法，为大规模遗留系统的重构找出了一条切实可行的路子。

【责任编辑：碧海蓝天 TEL：（010）68476606】