C++テンプレートシングルトンは、DLL AではDLL

Question

にIテンプレートシングルトンを有します。 DLLのC、D及びEは、ロガーを使用して、それは次のようにアクセスされる：

Singleton<Logger>::instance(); 

問題は、各DLLは代わりに同じシングルトンインスタンスを使用する

Singleton<Logger>. 

の独自のコピーをインスタンス化することです。私はこの問題の解決策がexternテンプレートを使用していることを理解しています。それは、C、DおよびEは

extern template class Singleton<Logger>; 

を含める必要がDLLとDLLのBであることは含まれている必要があります

template class Singleton<Logger>; 

は、これはまだ複数のテンプレートインスタンスが作成される原因となります。私はすべてのdllにexternを入れようとしましたが、まだ動作しませんでした。すべてのdllからexternを削除しようとしましたが、それでも動作しませんでした。テンプレートシングルトンを実装する標準的な方法ではありませんか？これを行う正しい方法は何ですか？

Answer 1

私にとっては、シンプトンのテンプレート定義に__declspec(dllexport)を追加するというトリックがあります。テンプレートの実装をクラス定義から分割し、実装をA DLLにのみ含めます。最後に、Singleton<Logger>::instance()を呼び出すダミー関数を作成して、テンプレートをA DLLでインスタンス化するように強制します。

だからあなたのA DLLのヘッダファイルには、あなたがこのようなシングルトンテンプレート定義：

template <class T> 
class __declspec(dllexport) Singleton { 
public: 
    static T &instance(); 
}; 

を次に、あなたのA DLLのCPPファイルにテンプレートの実装を定義し、このようSingleton<Logger>のインスタンス化を強制します：

template <class T> 
T &Singleton<T>::instance() { 
    static T _instance; 
    return _instance; 
}; 

void instantiate_logger() { 
    Singleton<Logger>::instance(); 
} 

私のコンパイラでは、少なくともどこからでもinstantiate_loggerに電話する必要はありません。ちょうどそれが存在すると、コードが強制的に生成されます。したがって、A DLLのエクスポートテーブルをこの時点でダンプすると、Singleton<Logger>::instance()のエントリが表示されます。

C DLLとD DLLでは、Singletonのテンプレート定義でヘッダーファイルをインクルードできますが、テンプレートの実装がないため、コンパイラはそのテンプレート用のコードを作成できません。これは、リンカーがSingleton<Logger>::instance()の未解決の外部について不平を言うことになりますが、それを修正するには、DLLのエクスポートライブラリにリンクするだけで済みます。

最後に、Singleton<Logger>::instance()のコードはDLL Aでのみ実装されているため、複数のインスタンスを持つことはできません。

Answer 2

instance() 
{ 
    if (_instance == NULL) { 
    _instance = new Singleton(); 
    } 

    return _instance; 
} 

のようないくつかの条件はこれが唯一の単一のインスタンスを作成し、それが2回目のコールを得たとき、それは単に古いインスタンスを返しますしてください。

Answer 3

MSDNは

のWin32のDLLは、呼び出し元プロセスのアドレス空間にマッピングされていることを述べています。 デフォルトでは、DLLを使用する各プロセスは、すべての DLLグローバル変数と静的変数の独自のインスタンスを持ちます。あなたのDLLは、他のアプリケーションによってロードされたことの 他のインスタンスとデータを共有する必要がある場合、あなたは次のいずれかの方法 を使用することができます。

Create named data sections using the data_seg pragma. 

Use memory mapped files. See the Win32 documentation about memory mapped files. 

http://msdn.microsoft.com/en-us/library/h90dkhs0%28v=vs.80%29.aspx

Answer 4

私はrefcountedクラスを組み合わせることをお勧めしますと、あなたのLoggerクラスでエクスポートAPI：

class Logger 
{ 
public: 
    Logger() 
    { 
    nRefCount = 1; 
    return; 
    }; 

    ~Logger() 
    { 
    lpPtr = NULL; 
    return; 
    }; 

    VOID AddRef() 
    { 
    InterLockedIncrement(&nRefCount); 
    return; 
    }; 

    VOID Release() 
    { 
    if (InterLockedDecrement(&nRefCount) == 0) 
     delete this; 
    return; 
    }; 

    static Logger* Get() 
    { 
    if (lpPtr == NULL) 
    { 
     //singleton creation lock should be here 
     lpPtr = new Logger(); 
    } 
    return lpPtr; 
    }; 

private: 
    LONG volatile nRefCount; 
    static Logger *lpPtr = NULL; 
}; 

__declspec(dllexport) Logger* GetLogger() 
    { 
    return Logger::Get(); 
    }; 

コードは、いくつかの固定を必要とするが、私はあなたの基本的な考え方を与えることを試みます。

Answer 5

「正しい」方法は、シングルトンを使用しないことです。

他のすべてのコードで同じタイプのインスタンスを使用する場合は、そのコードをそのインスタンスへの参照を関数またはコンストラクタのパラメータとして指定します。

シングルトン（非テンプレート）を使用するのは、グローバル変数を使用する場合とまったく同じですので、避けてください。

テンプレートを使用すると、コードをインスタンス化する方法と、「インスタンス」にアクセスする方法をコンパイラが決定します。あなたが経験している問題は、これとDLLの静的を組み合わせたものです。

生涯の問題（シングルトンを削除するのが確実なのはいつですか？）、スレッドセーフの問題、グローバルな共有アクセスの問題など、シングルトンが悪い理由はたくさんあります。

要するに、物事のインスタンスを1つだけ必要とする場合は、そのインスタンスを1つ作成し、それを必要とするコードに渡します。

Answer 6

ここでは、あなたが構築できる実際的なスケッチの解決策があります。複数のテンプレートがインスタンス化されますが、すべて同じインスタンスオブジェクトを共有します。

メモリリークを回避するには、いくつかの追加コードが必要です（例：void *をboost :: any of shared_ptrなどに置き換える）。 singleton.cppで

singleton.h

で

#if defined(DLL_EXPORTS) 
    #define DLL_API __declspec(dllexport) 
#else 
    #define DLL_API __declspec(dllimport) 
#endif 

template <class T> 
class Singleton 
{ 
public: 
    static T &instance() 
    { 
     T *instance = reinterpret_cast<T *>(details::getInstance(typeid(T))); 
     if (instance == NULL) 
     { 
      instance = new T(); 
      details::setInstance(typeid(T), instance); 
     } 

     return *instance; 
    } 
}; 

namespace details 
{ 

DLL_API void setInstance(const type_info &type, void *singleton); 
DLL_API void *getInstance(const type_info &type); 

} 

。

#include <map> 
#include <string> 

namespace details 
{ 

namespace 
{ 

std::map<std::string, void *> singletons; 

} 

void setInstance(const type_info &type, void *singleton) 
{ 
    singletons[type.name()] = singleton; 
} 

void *getInstance(const type_info &type) 
{ 
    std::map<std::string, void *>::const_iterator iter = singletons.find(type.name()); 
    if (iter == singletons.end()) 
     return NULL; 

    return iter->second; 
} 

} 

私は今より良い方法を考えることができません。インスタンスは、共通の場所に格納する必要があります。

Answer 7

私はあなたの実装では、あなたの問題を考える：

static T _instance; 

私はstatic修飾子は、各DLLのためのあなたのTクラスのインスタンスを1したコードを作成するために、コンパイラが発生することを想定しています。 シングルトンの異なる実装を試してみてください。 Singletoneクラスで静的Tフィールドを作成しようとすることができます。または、クラス内の静的ポインタを持つSingletoneが動作するはずです。このポインタが初期化されます）私はあなたが第二のアプローチを使用することをお勧めしたい、とあなたのBのDLLに、インスタンスのための最初の呼び出し（上よりも

Singletone<Logger>::instance = nullptr; 

を指定します。そして、これはあなたの問題を解決すると思います。

PS。 mutlithreading instancingを手動で処理することを忘れないでください