シングルトンパターン深く議論

Ensure a class only has one instance, and provide a global point of access to it

意味がわかりづらいかもしれないですが、ポイントとしては二つあります。
・システム全体にクラスのインスタンスが一つだけある
・このインスタンスのオブジェクトはグローバルアクセス可能なインターフェイスがある

確かにシングルトンパターンが簡単だと思う人が多いです。しかし実際は簡単なのでしょうか？以下の疑問を聞かせてください。
・スタティックと比べたら、何の違いがあるか
・正しく実装できるか

上記の疑問について、見た感じは簡単そうですが簡単かどうか深く検討しましょう。

// Using Singleton
public class AppConfig {
    public int appId;
    private static AppConfig self;

    public static AppConfig getAppConfigInstance() {
        if(self == null) {
            self = new AppConfig();
        }
        return self;
    }
}
config = AppConfig.getAppConfigInstance();
appId = config.appId;

// Using static
public class AppConfig {
    public static int appId;
}
appId = AppConfig.appId;

上記の実装を見ると、シングルトンは一箇所だけでアクセスであることがわかるが、スタティックもそうだろうね。シングルトンと比べたら、スタティックの方は実装は簡単だと思いますが、基本的な違いは以下となります。

Life-time
・シングルトンの場合はオブジェクトの初期化〜プログラムの終了
・スタティックの場合はクラスがロードされる時〜プログラムの終了
Abstraction
・シングルトンの場合は可能
・スタティックの場合は不可
つまり、基本的にシングルトンを使った方が多いかと思います。なぜかというのは例えば上記の例により、AppConfigだけではなく、WindowsAppConfigが必要なのであれば、スタティックを使ったら、実装が大変だと思って、シングルトンを使ったら、改修が少なくなります。ではシングルトンを良く理解して使いましょう。

Ensure a class only has one instance, and provide a global point of access to it

正しい実装方は二つポイントを守らないといけません。
・システム全体にクラスのインスタンスが一つだけある(1)
・このインスタンスのオブジェクトはグローバルアクセス可能なインターフェイスがある(2)
確かに簡単だと思う人が多いですよね。以下のルールに守れば良いだろうね。
（１）「constructor」は「private」で作って、自由にオブジェクトを作ることは不可能となる
（２）ひとつのスタティックメソッドだけで戻り値としてオブジェクトを返す

public class AppConfig {
    public int appId;
    private static AppConfig self;

    private AppConfig() {
        appId = new Random().nextInt() % 100;
    }

    public static AppConfig getAppConfigInstance() {
        if (self == null) {
            self = new AppConfig();
        }
        return self;
    }
}

public class Main {

    public static void main(String[] args) {
        AppConfig androidConfig = AppConfig.getAppConfigInstance();
        AppConfig iOSAppConfig = AppConfig.getAppConfigInstance();

        // The same values was printed
        System.out.println(androidConfig.appId);
        System.out.println(iOSAppConfig.appId);
    }
}

正しいかな？（笑）。これは正しくないですよー。以下のソースコードを見ましょう

public static void main(String[] args) {
        AppConfig config = AppConfig.getInstance();
        System.out.println(config.appId);

        AppConfig appConfig = null;
        try {
            Class<AppConfig> clazz = AppConfig.class;
            Constructor<AppConfig> constructor = clazz.getDeclaredConstructor();
            constructor.setAccessible(true);
            appConfig = constructor.newInstance();
        } catch (NoSuchMethodException | InvocationTargetException | IllegalAccessException | InstantiationException e) {
            e.printStackTrace();
        }
        // The different value was printed
        System.out.println(appConfig.appId);
    }

ああ、通常に違う値が表示されますよね。やっぱり正しくないですね。なお、上記の実装方には、二つオブジェクトが作られるので、「constructor」にて改修だけで良いかなと思います。

import java.util.Random;

public class AppConfig {
    public int appId;
    private static AppConfig self;

    private AppConfig() {
        if (self != null) {
            throw new UnsupportedOperationException("Use getAppConfigInstance() instead.");
        }
        appId = new Random().nextInt() % 100;
    }

    public static AppConfig getAppConfigInstance() {
        if (self == null) {
            self = new AppConfig();
        }
        return self;
    }
}

正しいかな？（笑）。これも正しくありません。マルチスレッドで実装して見ましょうか？

public class Main {

    public static void main(String[] args) {
        Thread threadOne = new Thread(new Runnable() {
            @Override
            public void run() {
                AppConfig config = AppConfig.getAppConfigInstance();
                System.out.println(config.appId);
            }
        });

        Thread threadTwo = new Thread(new Runnable() {
            @Override
            public void run() {
                AppConfig config = AppConfig.getAppConfigInstance();
                System.out.println(config.appId);
            }
        });

        threadOne.start();
        threadTwo.start();
    }
}

数回くらい実施すると、違う値が表示されますね。まだ正しくないですよね。どこが間違いかな思ってて、「threadOne」「threadTwo」にて同時に「if(self == null)」実施すると、もちろん二つオブジェクトが作られますよ。なので、オブジェクトの作る際に同期することが必要となります。

import java.util.Random;

public class AppConfig {
    private static volatile AppConfig self;
    public int appId;

    private AppConfig() {
        if (self != null) {
            throw new UnsupportedOperationException("Use getAppConfigInstance() instead.");
        }
        appId = new Random().nextInt() % 100;
    }

    public static synchronized AppConfig getAppConfigInstance() {
        if (self == null) {
            self = new AppConfig();
        }
        return self;
    }
}

ここまで見ると、シングルトンはやっぱり難しいかもしれません。では、正しい実装方を紹介させていただきます。

import java.util.Random;

public class AppConfig {
    private static volatile AppConfig self;
    public int appId;

    private AppConfig() {
        if (self != null) {
            throw new UnsupportedOperationException("Use getAppConfigInstance() instead.");
        }
        appId = new Random().nextInt() % 100;
    }

    public static synchronized AppConfig getAppConfigInstance() {
        if (self == null) {
            self = new AppConfig();
        }
        return self;
    }
}