コンストラクタ以外でインスタンス生成時にのみ値を設定できるようにする方法

Question

###実現したいこと
templateを使ってクラスのインスタンスを生成しているのですが、クラスを定義するときコンストラクタにおまじないのような引数を追加する必要があり、それを回避したいと思っています。

現在使用しているコードです(一部抜粋)

// シーンの親クラス
class Scene{
public:
Scene(const std::string& sceneName, const UINT sceneID) : 
m_SceneName{ sceneName }, m_SceneID{ sceneID } {}

private:
std::string m_SceneName{};
UINT m_SceneID{};
};

// サンプルシーン(ユーザーが定義するクラス)
class SampleScene : public Scene {
public:
// おまじない引数
SampleScene(const std::string& sceneName, const UINT sceneID) :
Scene(sceneName, sceneID) {}
};

// シーンを追加したりするクラス
class SceneManager {
public:
template<class T>
void AddScene(const std::string& sceneName) {
    std::shared_ptr<T> scene = std::make_shared<T>(sceneName, m_Scenes.size());
    m_Scenes[sceneName] = scene;
}

private:
std::map<std::string, std::shared_ptr<Scene>> m_Scenes;
};

###思いついた解決方法
・Sceneクラスに名前とIDを設定するprivateな関数を作成して、friendでSceneManagerにのみ公開する方法

個人的にfriendはコードが一気に汚れるので使いたくありません。。。

Answer 1

コンストラクタは引数なしとし、インスタンス生成後に一度だけm_SceneName,m_SceneIDを設定できるようにしたいということでしょうか。
そうであるならば、未設定の場合にのみ設定できるメソッドを追加すれば達成できるのでは？
以下ではm_SceneNameが最大値の場合を未設定時の値としていますが、最大値も使用したいなら別途フラグを追加で。

// シーンの親クラス
class Scene {
public:
	Scene() : m_SceneID(std::numeric_limits<UINT>::max()) {}
	void SetSceneInfo(const std::string& sceneName, const UINT sceneID)
	{
		if (std::numeric_limits<UINT>::max() != m_SceneID)
		{
			m_SceneName = sceneName;
			m_SceneID = sceneID;
		}
	}

private:
	std::string m_SceneName;
	UINT m_SceneID;
};

// サンプルシーン(ユーザーが定義するクラス)
class SampleScene : public Scene {
public:
	SampleScene() : Scene() {}
};

// シーンを追加したりするクラス
class SceneManager {
public:
	template<class T>
	void AddScene(const std::string& sceneName) {
		std::shared_ptr<Scene> scene = std::make_shared<T>();
		scene->SetSceneInfo(sceneName, m_Scenes.size());
		m_Scenes[sceneName] = scene;
	}

private:
	std::map<std::string, std::shared_ptr<Scene>> m_Scenes;
};

Answer 2

• 「Sceneおよびそれを継承した型ではScene::Scene(const std::string& sceneName, const UINT sceneID)による初期化をしなければならない」という形の設計
• templateでインスタンスを生成するという利点がよくわからない実装

の２つが余計なしがらみを生じさせているという話ならば，（この不自由な形を採るべき特段の理由があるのでもなければ）この両方を見直せばよいのでは．

---

Sceneに本当に必要なのは，名前とIDを「取得できること」なのかもしれません．
であれば，例えば，Sceneは

class Scene
{
public:
  virtual const std::string &GetName() = 0;
  virtual UINT GetID() = 0;
};

みたいな形の方が妥当なのかもしれません．

SceneManager::AddSceneは，tmplateにせずとも，単に
void AddScene( std::shared_ptr<Scene> spScene ) とか void AddScene( std::unique_ptr<Scene> &&upScene ) とかで問題ないかもしれません．

この形であれば，SampleSceneの生成方法の実装形態には何のしがらみも無くなります．
（まぁ，名前とIDの解決手段が何かしら必要なことには変わりはありませんが）

Comments

[yumetodo]

ちょっと寝かせて考えてみましたけど別に今のコードもそんなに悪いと思わないんですよね・・・。

[fana]

どんな形にしようが，どこぞで名前とIDを指定してやる必要があるわけで，それに最も相応しいのはコンストラクタだと思う…んだけども，それが嫌だという話(?)なので，
インスタンス生成～SceneManagerの管理に入る　までの部分をもっと自由にしてやって，管理下に入って以降は名前とIDに関してReadOnlyな型で扱えばそこが書き換えられる心配も無かろう，と．

Answer 3

そのコンストラクタで追加でなにかする気がないなら単に書かなければいいような。

Comments

[suittizihou]

シーン本体にもシーン名とIDを持たせておきたいのですが、名前とIDは生成時に1回セットするだけでそれ以降変えられるとバグが発生してしまうのでコンストラクタで設定していました。