24.需要时使用保护性拷贝
程序员文章站
2024-01-01 09:34:22
...
假设类的客户会尽一切手段来破坏这个类的约束条件,在这样的前提下,必须保护性的设计程序
在大多数情况下,都不希望类中的域通过另一个类的方法直接被修改,通常将非可变的参数修饰为private,但有时候会不小心提供这种修改的“机会”
class Period{
private final Date start;
private final Date end;
public Period(Date start, Date end){
if(start.compareTo(end) > 0){
throw new IllegalArgumentException(start + "after" + end);
}
this.start = start;
this.end = end;
}
public Date getStart(){
return start;
}
public Date getEnd() {
return end;
}
}
public class Main {
public static void main(String[] arg){
Date start = new Date();
Date end = new Date();
Period p = new Period(start, end);
end.setYear(78);
}
}
可以发现,虽然类中域被定义成了final private,但通过构造函数这样初始化,并且Date类本身可变,则可以不通过类来修改私有域。
对于Date域初始化,可以利用getTime()解决
class Period{
private final Date start;
private final Date end;
public Period(Date start, Date end){
this.start = new Date(start.getTime());
this.end = new Date(end.getTime());
if(start.compareTo(end) > 0){
throw new IllegalArgumentException(start + "after" + end);
}
}
public Date getStart(){
return start;
}
public Date getEnd() {
return end;
}
}
这此对于参数的检查放在了后面,这是为了避免检查完成后,到赋值的时之间,其他线程对原始数据进行了修改
同时注意,在初始化的时候,不要使用clone方法,因为你并不知道参数到底是不是Date类本身,还是其子类,如果是子类,那么有可能子类中有被别人添加的静态列表(未进行深复制),别人可以通过静态列表来攻击
这样初始化算完成,但是外部还是可以去改变内部域,例如
Date start = new Date();
Date end = new Date();
Period p = new Period(start, end);
p.getEnd().setYear(78);
这时,我们就可以利用clone了,因为这时候我们已经知道了这个类一定是Date类而非子类
public Date getStart(){
return (Date)(start.clone());
}
public Date getEnd() {
return (Date)(start.clone());
}
所以,在我们提供一个类给别人使用的时候,如果没有文档详细的说明,最好做好保护性的拷贝,否则会在不经意间改变一些不想改变的值