Remove Setting Method (移除设值函数)

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

143720_wiEi_134516.png                                         

动机: 

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

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

做法:

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

2.修改构造函数,使其直接访问设值函数所针对的那个变量。

à如果某个子类通过设值函数给超类的某个private字段设了值,那么你就不能这样修改。这种情况下你应该试着在超类中提供一个protected函数(最好是构造函数)来给这些字段设值。不论你怎么做,都不要给超类中的函数起一个与设值函数混淆的名字。

3. 编译,测试。

4.移除这个设值函数,将它所针对的字段设为final

5.编译,测试。

范例:

下面是一个简单的例子:

class Account{
    private String _id;
    Account(String id){
        setId(id);
    }
    void setId(String arg){
        _id = arg;
    }
}

以上代码可修改为:

class Account{
    private 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 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, doulbe rate){
        super(id);
        _interestRate = rate;
    }

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

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

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

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

    在这里,我们应该将设值函数替换为add操作和remove操作,Encapsulate Collection中谈到了这一点

转载于:https://my.oschina.net/u/134516/blog/220682

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值