質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.33%

Singletonのテンプレートクラス化試作で出る参照エラーが解決出来ない

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,864

Iwan

score 27

C++の中でSTL、デザインパターンを勉強中でその一環としてSingletonをテンプレートクラス化して
使ってみる。と言う事をサイト等参考に試作してみたのですが以下のエラーが出ており
調べたりしているのですが解決できず苦慮しており教えて頂きたく思います。

エラー    LNK1120    1 件の未解決の外部参照    DEV_01    C:\MyFile\Sandbox\DEV_01\x64\Debug\DEV_01.exe    1    
エラー    LNK2019    未解決の外部シンボル "private: __cdecl SampleObject::SampleObject(void)" (??0SampleObject@@AEAA@XZ) が関数 "public: static class SampleObject * __cdecl Singleton<class SampleObject>::getSingletonPtr(void)" (?getSingletonPtr@?$Singleton@VSampleObject@@@@SAPEAVSampleObject@@XZ) で参照されました。    DEV_01    C:\MyFile\Sandbox\DEV_01\AppSystem.obj    1

【使用環境】
Windows10
VisualStudio2017
DXライブラリ

【サンプルのソースコード】

<SampleSingleton.h>

template <class _Ty>
class Singleton
//class Singleton final    // 派生クラスの定義禁止
{
public:
    typedef _Ty    InstanceType;

public:
/*
    static void create(){
        if (_pInstance == nullptr) {
            _pInstance = new InstanceType();
        }
    }
    static void destroy(){
        if (_pInstance != nullptr) {
            delete _pInstance;
            _pInstance = nullptr;
        }
    }
*/
    static InstanceType& getInstanceRef( void ){
        return *_pInstance;
    }
    static InstanceType* getInstancePtr( void ){
        return _pInstance;
    }
    static InstanceType* getSingletonPtr( void ){
        if ( !_pInstance ) { _pInstance = new InstanceType(); }
        return _pInstance;
    }

    // 生成済み?
    static bool isCreate() { return _pInstance != nullptr; };
    // 破棄済み?
    static bool isDestroy() { return _pInstance == nullptr; };

private:
    static InstanceType * _pInstance;        //!< 唯一のインスタンス

private:
    // 禁止
    Singleton() {}
    virtual ~Singleton() {}

    Singleton(const Singleton&);        /** コピーコンストラクタ(使用禁止) */
    Singleton& operator=(const Singleton&);    /** 代入演算子(使用禁止) */
};

template <class _Ty>
typename Singleton<_Ty>::InstanceType* Singleton<_Ty>::_pInstance = nullptr;
<SampleObject.h>

class SampleObject// : public Singleton<SampleObject >
{
    // SampleObjectのインスタンスを生成する時に、コンストラクタが呼べなかった為friend指定
    // シングルトンのテンプレート引数に自身を指定したものだけに生成をゆるす
    friend Singleton<SampleObject>;

public:
    int     m_UseEnable;

private:
    // コンストラクタ部分をpublicにしてしまうと、SampleObject  container; でインスタンスつくれてしまう
    // 「そのクラスのインスタンスが1つしか生成されないことを保証することができる」
    // と言うのに反するのでprivateにする必要がある。
    SampleObject();            // 外部でのインスタンス作成は禁止
    virtual ~SampleObject();

};

inline SampleObject* createObject( void )
{
    return Singleton<SampleObject>::getSingletonPtr();
}
<main.cpp>
#include "SampleObject.h"
#include "DxLib.h"

using CSampleObject= Singleton<SampleObject>;

int WINAPI    WinMain( HINSTANCE    hInstance,
                       HINSTANCE    hPrevInstance,
                       LPSTR        lpCmdLine,
                       int          nCmdShow )
{
  // DXライブラリ関係の記述は省略
    auto result = Singleton<SampleObject>::getSingletonPtr();

  // DXライブラリ関係の記述は省略
}

記載してあるコメントで間違っているのがあったり何書いてるのか不明な箇所等
あれば指摘して頂ければ。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

こんにちは。

    SampleObject();            // 外部でのインスタンス作成は禁止

デフォルト・コンストラクタを宣言していますが、実体の定義がないですよ。
getSingletonPtrから呼び出しているので必要です。

yumetodoさんのは default 指定して実体生成をコンパイラへ指示しているからエラーにならないのだと思います。


しかし、ご提示の実装は任意のクラスをシングルトン化するものですけど、そもそもシングルトンの考え方が「インスタンスを1つしか生成できないことを保証する」目的で使うので、これは意味がないように感じます。
class SampleObject : public Singleton<SampleObject>のようにしてCRTPを併用すると目的を果たすと思います。(boostでよく使われているパターンです。)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/28 16:07

    リンク先のCRTPサイトを参考にソースを修正しエラーは解消されました。
    その中で分からない点がありまして
    protected:
    // コンストラクタをprotectedに定義する
    Singleton()
    {
    std::cout << TYPENAME(tDerived) << "()\n";
    }
    と書いてあるのですがprotected:にする理由は何ですか?

    キャンセル

  • 2018/11/28 17:18

    試しにprivateにしてみればわかると思いますが、基底クラスのコンストラクタを呼べなくなってしまいますよね?

    キャンセル

  • 2018/11/28 17:42

    Iwanさん
    yumetodoさんのフォローの通りです。
    明示的に書いてなくても派生クラスのコンストラクタは基底クラスのコンストラクタを必ず呼びますが、privateだと呼べません。
    このようなケースでは派生先を限定したくないのでfriend指定もできません。
    外部から使うことを想定しないのでpublicにするのはあまりよくないでしょう。
    なのでprotectedです。

    キャンセル

+2

>static InstanceType * _pInstance;        //!< 唯一のインスタンス

これをC++17のinline変数にしてあげたほうがいいのと無駄にデストラクタを記述しない以外は特に問題ないと思うんですけどねぇ・・・。それで手元ではエラーないので

https://wandbox.org/permlink/109bKnmQQoD9iDPL

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.33%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る