Re: Threadsafe singletons

From:

"Victor Bazarov" <v.Abazarov@comAcast.net>

Newsgroups:

comp.lang.c++.moderated,comp.programming.threads

Date:

31 Jul 2006 15:31:45 -0400

Message-ID:

<eaktqn$3b4$1@news.datemas.de>

fluidparody@yahoo.com wrote:

Tell me if I have your idea correct because this isn't working for me.

/////////// singleton.h
class Singleton
{
char *name;

Add here:

Singleton(Singleton const&);
Singleton& operator(Singleton const&);

public:
static Singleton& GetInstance();

char *GetName() const { return name; }
void SetName(char *n) { name = n; }

private:
Singleton();
};

/////////// singleton.cpp
#include "singleton.h"

Singleton::Singleton()
{
}

/////////// main.cpp
#include "singleton.h"

#include <iostream>

Singleton& Singleton::GetInstance()
{
static Singleton s;
return s;
}

static struct InitSingleton
{
InitSingleton()
{
Singleton s = Singleton::GetInstance();

Replase the line above with

Singleton &s = Singleton::GetInstance();

s.SetName("Super Duper");
}
} s_init;

int main()
{
Singleton singleton = Singleton::GetInstance();

Replace the line above with

Singleton & singleton = Singleton::GetInstance();

std::cout << "The singleton's name is: " <<
singleton.GetName() << std::endl;

return 0;
}

I'm not getting a "Super Duper" Singleton

You are simply not getting a singleton with your code.

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[ See http://www.gotw.ca/resources/clcm.htm for info about ]
[ comp.lang.c++.moderated. First time posters: Do this! ]