下面是一个比较微妙也比较常见的反面的例子,其中涉及可变的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的时候都创建这些实例,如果这个方法被频繁的调用,将会显著的提高性能。