2005年09月24日

Singleton パターンではインスタンスを一つ作ってそれを使いまわす．そのために，そのインスタンスを取得する関数でインスタンスを作ってあるかを調べ，あったらそれをかえす，なければ新しく作るという動作をさせるのが一番楽．で，コンストラクタはプライベートにしておくと．

しかしながらとあるプログラマは次のようなコードを書いて，うまく動かないと騒いでいた．

AClass* AClass::getInstance(){
	static bool isFirst = false;
	if(!isFirst){
		instance = new AClass();
	}
	return instance;
}

isFirst が false のままだし...　あなたインスタンス取得するたびに新しいの作るんですか？　Singleton とか言ってたのにインスタンスはひとつじゃないんですね．そもそも isFirst などというフラグを持たずに if(!instance) と書いていれば間違いがなかったものを．

とりあえず今後の参考のためにこの事実を記しておこう．ちなみに彼はコンストラクタすらもプライベートでなかったという...　（まあ，このソース以外では正しいコードを書いていたそうだが．）