简化函数调用之十 :Remove Setting Method(移除设置函数)

你的class 中的某个值域,应该在对象初创时被设值,然后就不再改变。

去掉该值域的所有设值函数(setter)。

动机(Motivation)

如果你为某个值域提供了设值函数(setter),这就暗示这个值域值可以被改变。如果你不希望在对象初创之后此值域还有机会被改变,那就不要为它提供设值函数 (同时并将该值域设为final )。这样你的意图会更加清晰,并且往往可以排除其值被修改的可能性——这种可能性往往是非常大的。

如果你保留了间接访问变量的方法,就可能经常有程序员盲目使用它们[Beck]。这些人甚至会在构造函数中使用设值函数!我猜想他们或许是为了代码的一致性,但却忽视了设值函数往后可能带来的混淆。

作法(Mechanics)

·检查设值函数(setter)被使用的情况,看它是否只被构造函数调用,或者被构造函数所调用的另一个函数调用。

·修改构造函数,使其直接访问设值函数所针对的那个变量。
Ø如果某个subclass 通过设值函数给superclass 的某个private 值域设了值,那么你就不能这样修改。这种情况下你应该试着在superclass 中提供一个protected 函数(最好是构造函数)来给这些值域设值。不论你怎么做,都不要给superclass 中的函数起一个与设值函数混淆的名字。

·编译,测试。

·移除这个设值函数,将它所计对的值域设为final 。

·编译,测试。

范例:(Example)

译注:本书英文版网站上的勘误网页(www.refactoring.com/errata.html)显示,本页程序有些问题,惟因前后颇有牵连,故勘误表上并未明确条列代码之修改。请读者自行上网査阅理解。

下面是一个简单例子:

class Account {

  private String _id;

  Account (String id) {

      setId(id);

  }

  void setId (String arg) {

     _id = arg;

  }

以上代码可修改为:

class Account {

  private final String _id;

  Account (String id) {

     _id = id;

  }

问题可能以数种不同的形式出现。首先,你可能会在设值函数中对引数做运算:

class Account {

  private String _id;

  Account (String id) {

      setId(id);

  }

  void setId (String arg) {

     _id = "ZZ" + arg;

  }

如果对引数的修改很简单(就像上面这样)而且又只有一个构造函数,我可以直接在构造函数中做相同的修改。如果修改很复杂,或者有一个以上的函数调用它,我就需要提供一个独立函数。我需要为新函数起个好名字,清楚表达该函数的用途:

class Account {

  private final String _id;

  Account (String id) {

      initializeId(id);

  }

  void initializeId (String arg) {

     _id = "ZZ" + arg;

  }

如果subclass 需要对superclass 的private 变量赋初值,情况就比较麻烦一些:

class InterestAccount extends Account...

  private double _interestRate;

  InterestAccount (String id, double rate) {

      setId(id);

      _interestRate = rate;

  }

问题是我无法在InterestAccount() 中直接访问id 变量。最好的解决方法是使用superclass 构造函数:

class InterestAccount...

  InterestAccount (String id, double rate) {

      super(id);

      _interestRate = rate;

  }

如果不能那样做,那么使用一个命名良好的函数就是最好的选择:

class InterestAccount...

  InterestAccount (String id, double rate) {

      initializeId(id);

      _interestRate = rate;

  }

另一种需要考虑的情况就是对一个群集(collections)设值:

class Person {

  Vector getCourses() {

      return _courses;

  }

  void setCourses(Vector arg) {

     _courses = arg;

  }

  private Vector _courses;

在这里,我希望将设值函数替换为"add"操作加上"remove"操作。我己经在 Encapsulate Collection 中谈到了这一点。

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值