重构改善既有代码设计--重构手法18:Self Encapsulate Field (自封装字段)

你直接访问一个值域(field),但与值域之间的耦合关系逐渐变得笨拙。

为这个值域建立取值/设值函数(getting/setting methods),并且只以这些函数来访问值域。

 

private int _low, _high;
boolean includes(int arg) {
return arg >= _low && arg <= _high;
}

==〉

 

private int _low, _high;
boolean includes(int arg) {
return arg >= getLow() && arg <= getHigh();
}
int getLow() {return _low;}
int getHigh() {return _high;}

 

动机

如果你想访问superclass中的一个值域,却又想在subclass中将[对这个变量的访问]改为一个计算后的值,这就是最该使用Self Encapsulate Field(171)的时候。[值域自我封装]只是第一步。完成自我封装之后,你可以在subclass中根据自己的需要随意覆写取值/设值函数(getting/setting methods)。

 

作法

1. 为[待封装值域]建立取值/设值函数(getting/setting methods)。

2. 找出该值域的所有引用点,将它们全部替换为[对于取值/设值函数的调用]。

如果引用点是[读值]值域值,就将它替换为[调用取值函数];如果引用点是[设定]值域值,就将它替换为[调用设值函数]。

你可以暂时为设值域改名,让编译器帮助你查找引用点。

3. 将该值域声明为private。

4. 复查,确保找出所有引用点。

5. 编译,测试。


class IntRange {
private int _low, _high;

boolean includes(int arg) {
return arg >= _low && arg <= _high;
}

void grow(int factor) {
_high = _high * factor;
}

IntRange(int low, int high) {
_low = low;
_high = high;
}
}

为了封装_low和_high这两个值域,我先定义[取值/设值函数](如果此前没有定义的话),并使用它们:
class IntRange {
private int _low, _high;

boolean includes(int arg) {
return arg >= _low && arg <= _high;
}

void grow(int factor) {
SetHigh(getHigh()*factor);
}

int getLow() {
return _low;
}

int getHigh() {
return _high;
}

void setLow(int arg) {
_low = arg;
}

void setHigh(int arg) {
_high = arg;
}
}

使用本项重构时,一般说来,设值函数被认为应该在[对象创建后]才使用,所以初始化过程中的行为有可能与设值函数的行为不同。这种情况下,我允许在构造函数中直接访问值域,要不就是建立另一个独立的初始化函数:

IntRange(int low, int high) {
initialize(low, high);
}
private void initialize(int low, int high) {
_low = low;
_high = high;
}

一旦你拥有一个subclass,上述所有动作的价值就体现出来了。如下所示:

class CappedRange extends IntRange {
CappedRange(int low, int high, int cap) {
super(low, high);
_cap = cap;
}

private int _cap;

int getCap() {
return _cap;
}

int getHigh() {
return Math.min(super.getHigh(), getCap());
}
}
现在,我可以CappedRange class中覆写getHigh(),从而加入对cap的考虑,而不必修改IntRange class的任何行为。

posted on 2017-09-20 20:43  pony1223  阅读(216)  评论(0编辑  收藏  举报

导航