不要なオブジェクトの生成を避ける

avoid-creating-unnecessary-objects.png

なるべく新たに生成させずに一つのオブジェクトを再利用すること。[1]

ダメな例 良い例
private static final Date birthDate=new Date();
    public boolean isBabyBoomer(){
        Calendar gmtCal=Calendar.getInstance(TimeZone.getTimeZone("GMT"));
         //↑ここがダメ!呼び出すためにインスタンスが生成される
        gmtCal.set(1946,Calendar.JANUARY,1,0,0,0);
        Date boomStart=gmtCal.getTime();
        gmtCal.set(1965, Calendar.JANUARY,1,0,0,0);
        Date boomEnd = gmtCal.getTime();
        return birthDate.compareTo(boomStart)>=0 && birthDate.compareTo(boomEnd)<0;
    }
public class Person {
    private static final Date birthDate=new Date();
    private static final Date BOOM_START;
    private static final Date BOOM_END;
    static{
        Calendar gmtCal=Calendar.getInstance(TimeZone.getTimeZone("GMT"));
        //一回だけnewする。あとはPersonクラスを何個作ろうと同じ参照が使用される
        gmtCal.set(1946,Calendar.JANUARY,1,0,0,0);
        BOOM_START=gmtCal.getTime();
        gmtCal.set(1965, Calendar.JANUARY,1,0,0,0);
        BOOM_END = gmtCal.getTime();
    }
 
    public boolean isBabyBoomer(){
        return birthDate.compareTo(BOOM_START)>=0 && birthDate.compareTo(BOOM_END)<0;
    }
}

Javaのオブジェクト変数はすべてが参照なので、

何回も呼び出す関数の中でオブジェクトをnewしてはならない

なのだ。うーん、知らなかった。
変数のスコープはなるべく少なく、の法則に従うと関数の中でnewしたものは抜けた後自動的にメモリ解放されるのかと思ってた>_<
なので、答えになるメンバ変数を用意してそれを領域として使って使いまわすのがいいんだ!


effective-java

サポートサイト Wikidot.com effective-java