Neden threadsafe olmadığına dair sorunuzu cevaplamak için, ilk çağrının yapıcıyı çağırması instance()gerektiği için değil Singleton s. İş parçacığı güvenli olmak için bu kritik bir bölümde gerçekleşmelidir ve standartta kritik bir bölümün alınmasına gerek yoktur (bugüne kadar standart dişler üzerinde tamamen sessizdir). Derleyiciler bunu genellikle statik bir booleanın basit bir kontrolünü ve artışını kullanarak uygular - ancak kritik bir bölümde değil. Aşağıdaki sözde kod gibi bir şey:
static Singleton& instance()
{
static bool initialized = false;
static char s[sizeof( Singleton)];
if (!initialized) {
initialized = true;
new( &s) Singleton(); // call placement new on s to construct it
}
return (*(reinterpret_cast<Singleton*>( &s)));
}
İşte basit bir iş parçacığı için güvenli Singleton (Windows için). Biz derleyici otomatik olarak başlatmak zorunda böylece, Windows CRITICAL_SECTION nesnesi için basit bir sınıf sarmalayıcı kullanır CRITICAL_SECTIONönce main()denir. İdeal olarak, kritik bölüm tutulduğunda ortaya çıkabilecek istisnalarla başa çıkabilen gerçek bir RAII kritik bölüm sınıfı kullanılır, ancak bu, bu cevabın kapsamı dışındadır.
Temel işlem, bir örneği Singletonistendiğinde bir kilit alındığında, gerekiyorsa Singleton'ın oluşturulması, daha sonra kilidin serbest bırakılması ve Singleton referansının geri döndürülmesidir.
#include <windows.h>
class CritSection : public CRITICAL_SECTION
{
public:
CritSection() {
InitializeCriticalSection( this);
}
~CritSection() {
DeleteCriticalSection( this);
}
private:
// disable copy and assignment of CritSection
CritSection( CritSection const&);
CritSection& operator=( CritSection const&);
};
class Singleton
{
public:
static Singleton& instance();
private:
// don't allow public construct/destruct
Singleton();
~Singleton();
// disable copy & assignment
Singleton( Singleton const&);
Singleton& operator=( Singleton const&);
static CritSection instance_lock;
};
CritSection Singleton::instance_lock; // definition for Singleton's lock
// it's initialized before main() is called
Singleton::Singleton()
{
}
Singleton& Singleton::instance()
{
// check to see if we need to create the Singleton
EnterCriticalSection( &instance_lock);
static Singleton s;
LeaveCriticalSection( &instance_lock);
return s;
}
Adam - "daha iyi bir küresel olmak" için çok saçma.
(Bazı hataların geçmesine izin vermediysem) bu uygulamanın ana dezavantajları:
- eğer
new Singleton()atar, kilit serbest olmayacak. Bu, burada sahip olduğum basit yerine gerçek bir RAII kilit nesnesi kullanılarak düzeltilebilir. Bu, kilit için platformdan bağımsız bir sargı sağlamak için Boost gibi bir şey kullanırsanız, işleri taşınabilir hale getirmeye de yardımcı olabilir.
- Bu, Singleton örneği
main()çağrıldıktan sonra iş parçacığı güvenliğini garanti eder - daha önce çağırırsanız (statik nesnenin başlatılmasında olduğu gibi), başlatılmayabileceğinden işler çalışmayabilir CRITICAL_SECTION.
- her örnek talep edildiğinde bir kilit alınmalıdır. Dediğim gibi, bu basit bir iş parçacığı güvenli bir uygulamadır. Daha iyi bir taneye ihtiyacınız varsa (veya çift kontrol kilidi tekniği gibi şeylerin neden kusurlu olduğunu bilmek istiyorsanız), Groo'nun cevabındaki bağlantılı makalelere bakın .