重构函数调用-Remove Setting Method移除设置函数十

重构函数调用-Remove Setting Method移除设置函数十

1.移除设置函数

1.1.使用场景

类中的某个字段应该在对象创建时被设值，然后就不再改变。去掉该字段的所有设值函数。

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

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

1.2.如何做

• 检查设值函数被使用的情况，看它是否只被构造函数调用,或者被构造函数所调用的另一个函数调用。
• 修改构造函数，使其直接访问设值函数所针对的那个变量。
• 如果某个子类通过设值函数给超类的某个private字段设了值，那么你就不能这样修改。这种情况下你应该试着在超类中提供一个protected函数（最好是构造函数）来给这些字段设值。不论你怎么做，都不要给超类中的函数起一个与设值函数混淆的名字。
• 编译，测试。
• 移除这个设值函数，将它所针对的字段设为final。[1]
• 编译，测试。

1.3.示例

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;
  }

如果子类需要对超类的private变量赋初值，情况就比较麻烦一些

class InterestAccount extends Account...
  private double _interestRate;
  InterestAccount (String id, double rate) {
      setId(id);
      _interestRate = rate;
  }

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

class InterestAccount...
  InterestAccount (String id, double rate) {
      super(id);
      _interestRate = rate;
  }

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

class InterestAccount...
  InterestAccount (String id, double rate) {
      initializeId(id);
      _interestRate = rate;
  }

另一种需要考虑的情况就是对一个集合设值

class Person {
  Vector getCourses() {
      return _courses;
  }
  void setCourses(Vector arg) {
     _courses = arg;
  }
  private Vector _courses;

在这里，我希望将设值函数替换为add操作和remove操作。我已经在Encapsulate Collection (208)中谈到了这一点。