下面是一个比较微妙也比较常见的反面的例子,其中涉及可变的Date对象,他们的值一旦计算出来之后就不再变化。这个类建立了一个模型:其中有一个人,并有一个isBabyBoomer方法,用来检验这个人是否为一个BabyBoomer,换句话说,就是检验这个人是否出生于1946年至1964年之间。
public class person {
private final Date birthDate = null;
public boolean isBabyBoomer() {
Calendar calendar = Calendar.getInstance(TimeZone.getTimeZone("GMT"));
calendar.set(1946, Calendar.JANUARY,1,0,0,0);
Date boomStart = calendar.getTime();
calendar.set(1965, Calendar.JANUARY,1,0,0,0);
Date boomEnd = calendar.getTime();
return birthDate.compareTo(boomStart) >= 0 &&
birthDate.compareTo(boomEnd) < 0;
}
}
isBabyBoomer每次被调用的时候,都会新建一个Calendar ,一个TimeZone和两个Date实例,这是不必要的。下面的版本用一个静态的初始净化器,避免了这种效率低下的情况:
public class person {
private final Date birthDate = null;
/**
* the starting and ending dates of the baby boom
*/
private static final Date BOOM_START;
private static final Date BOOM_END;
static {
Calendar calendar = Calendar.getInstance(TimeZone.getTimeZone("GMT"));
calendar.set(1946, Calendar.JANUARY,1,0,0,0);
BOOM_START = calendar.getTime();
calendar.set(1965, Calendar.JANUARY,1,0,0,0);
BOOM_END = calendar.getTime();
}
public boolean isBabyBoom() {
return birthDate.compareTo(BOOM_START)>= 0 &&
birthDate.compareTo(BOOM_END) < 0;
}
}
改进后的person类只有在初始化的时候创建Calendar ,TimeZone,Date实例一次,而不是在每次调用isBabyBoom的时候都创建这些实例,如果这个方法被频繁的调用,将会显著的提高性能。