类之中有一个数值类型码,但它并不影响类的行为。以一个新的类替换该数值类型码
在以C为基础的编程语言中,类型码或枚举值很常见。如果带着一个有意义的符号名,类型码的可读性还是不错的。问题在于,符号名终究只是个别名,编译器看见的、进行类型检验的,还是背后那个数值。任何接受类型码作为参数的函数,所期望的实际上是一个数值,无法强制使用符号名。这会大大降低代码的可读性,从而成为bug之源。
如果把那样的数值换成一个类,编译器就可以对这个类进行类型检验。只要为这个类提供工厂函数,你就可以始终保证只有合法的实例才会被创建出来,而且它们都会被传递给正确的宿主对象。
但是,在使用Replace Type Code with Class (218)之前,你应该先考虑类型码的其他替换方式。只有当类型码是纯粹数据时(也就是类型码不会在switch语句中引起行为变化时),你才能以类来取代它。
Java只能以整数作为switch语句的判断依据,不能使用任意类,因此那种情况下不能够以类替换类型码。更重要的是:任何switch语句都应该运用Replace Conditional with Polymorphism (255)去掉。为了进行那样的重构,你首先必须运用Replace Type Code with Subclasses (223)或Replace Type Code with State/Strategy (227),把类型码处理掉。
即使一个类型码不会因其数值的不同而引起行为上的差异,宿主类中的某些行为还是有可能更适合置放于类型码类中,因此你还应该留意是否有必要使用Move Method (142)将一两个函数搬过去。
每个人都拥有四种血型中的一种。我们以Person 来表示「人」,以其中的type code 表示「血型」:
class Person {public static final int O = 0;public static final int A = 1;public static final int B = 2;public static final int AB = 3;private int _bloodGroup;public Person (int bloodGroup) {_bloodGroup = bloodGroup;}public void setBloodGroup(int arg) {_bloodGroup = arg;}public int getBloodGroup() {return _bloodGroup;}}
首先,我建立一个新的BloodGroup class,用以表示「血型」,并在这个实体中保存原本的type code 数值:
class BloodGroup {public static final BloodGroup O = new BloodGroup(0);public static final BloodGroup A = new BloodGroup(1);public static final BloodGroup B = new BloodGroup(2);public static final BloodGroup AB = new BloodGroup(3);private static final BloodGroup[] _values = {O, A, B, AB};private final int _code;private BloodGroup (int code ) {_code = code;}public int getCode() {return _code;}public static BloodGroup code(int arg) {return _values[arg];}}
然后,我把Person 中的type code 改为使用BloodGroup class
class Person {public static final int O = BloodGroup.O.getCode();public static final int A = BloodGroup.A.getCode();public static final int B = BloodGroup.B.getCode();public static final int AB = BloodGroup.AB.getCode();private BloodGroup _bloodGroup;public Person (int bloodGroup) {_bloodGroup = BloodGroup.code(bloodGroup);}public int getBloodGroup() {return _bloodGroup.getCode();}public void setBloodGroup(int arg) {_bloodGroup = BloodGroup.code (arg);}}
现在,我因为BloodGroup class 而拥有了运行期检验能力。为了真正从这些改变中获利,我还必须修改Person 的用户,让它们以BloodGroup 对象表示type code,而不再使用整数。
首先,我使用Rename Method 修改type code 访问函数的名称,说明当前情况:
class Person...public int getBloodGroupCode() {return _bloodGroup.getCode();}
然后我为Person 加入一个新的取值函数(getter),其中使用BloodGroup
public BloodGroup getBloodGroup() {return _bloodGroup;}
另外,我还要建立新的构造函数和设值函数(setter),让它们也使用BloodGroup :
public Person (BloodGroup bloodGroup ) {_bloodGroup = bloodGroup;}public void setBloodGroup(BloodGroup arg) {_bloodGroup = arg;}
现在,我要继续处理Person 用户。此时应该注意,每次只处理一个用户,这样才可以保持小步前进。每个用户需要的修改方式可能不同,这使得修改过程更加棘手。 对Person 的static 变量的所有引用点也需要修改。因此,下列代码:
Person thePerson = new Person(Person.A)
就变成了:
Person thePerson = new Person(BloodGroup.A);
调用取值函数(getter)
必须改为调用新取值函数
。因此,下列代码:
thePerson.getBloodGroupCode()
变成了 :
thePerson.getBloodGroup().getCode()
修改完毕Person 的所有用户之后,我就可以删掉原本使用整数型别的那些旧的取值函数、构造函数、静态变量和设值函数了:
class Person ...public static final int O = BloodGroup.O.getCode();public static final int A = BloodGroup.A.getCode();public static final int B = BloodGroup.B.getCode();public static final int AB = BloodGroup.AB.getCode();public Person (int bloodGroup) {_bloodGroup = BloodGroup.code(bloodGroup);} public int getBloodGroup() {return _bloodGroup.getCode();}public void setBloodGroup(int arg) {_bloodGroup = BloodGroup.code (arg);}
我还可以将BloodGroup 中使用整数型别的函数声明为private (因为再没有人会使用它们了)
class BloodGroup...private int getCode() {return _code;}private static BloodGroup code(int arg) {return _values[arg];}
上一篇:CSS实现文字凹凸效果