als Umsteiger von PHP möchte ich zur Zeit die wichtigsten OOP-Patterns testen.
Da es in PHP weder Pointer noch strenge Datentypskontrollen gibt, ist C++ schon ein bisschen komplizierter.

Deshalb wollte ich fragen, ob meine Impletmentierung des Singletons so gut ist.

#include<iostream>
using namespace std;

class Singleton
{
  private:
    static Singleton* instanz;

  protected:
    Singleton() { };
    Singleton(const Singleton&){};

  public:
    static Singleton* getInstance();
};

Singleton* Singleton::instanz = 0;

Singleton* Singleton::getInstance()
{
  if(instanz == 0)
  {
     instanz = new Singleton;
  }
  return instanz;
}

int main()
{
  Singleton * gibtsnureinmal = Singleton::getInstance();
  Singleton * dasistgleichwiegibtsnureinmal = Singleton::getInstance();
  Singleton * unddasauch = Singleton::getInstance();
  if(gibtsnureinmal == unddasauch)
  {
    cout<<"Die beiden Objekte sind die gleichen, nur unter anderen Namen.";
    system("pause");
  }
  return 0;
}

Es funktioniert zumindest...

Simon

Zu berücksichtigen ist, dass dieses Singleton nicht Thread-Safe ist.

Simon

Edit:
Eine umfangreiche Diskussion des Singleton Pattern ist in Modern C++ Design von Andrei Alexandrescu zu finden.

Zu jedem new , gehört auch ein delete ... und wann gibst du diesen Speicher wieder frei? Klar, wird es das Betriebsystem beim Beenden übernehmen, zumindest normalerweise, aber davon sollte man nicht ausgehen!

Daher wäre es besser, wenn du sowas machst:

Singleton& Singleton::getInstance()
{
  static Singleton instance;
  return instance;
}

static garantiert hier, dass bei jedem Methodenaufruf genau das gleiche Objekt verwendet wird, bzw. dass das Objekt erhalten bleibt. Es wird auch erst beim ersten Aufruf initialisiert. Der Speicher wird dann am Ende auch korrekt freigegeben.

Grüssli

Bei diesem Code funktioniert ein Vergleich der beiden Objekte nicht:

#include<iostream>
using namespace std;

class Singleton
{
  private:
    Singleton();
    Singleton(const Singleton&);

  public:
    static Singleton& getInstance();
};

Singleton& Singleton::getInstance()
{
  static Singleton instanz;
  return instanz;
}

int main()
{
  Singleton &gibtsnureinmal = Singleton::getInstance();
  Singleton &unddasauch = Singleton::getInstance();
  if(gibtsnureinmal == unddasauch)
  {
    cout<<"Die beiden Objekte sind die gleichen, nur unter anderen Namen.";
    system("pause");
  }
  return 0;
}

C++-Builder will, dass ich den ==-Operatpr implementiere. Dabei will ich doch nur testen, ob die beiden Objekte die gleichen sind

Was mache ich falsch?

C++-Builder will, dass ich den ==-Operatpr implementiere. Dabei will ich doch nur testen, ob die beiden Objekte die gleichen sind

Hehe, du lieferst dir die Erklärung gleich im eigenen Satz.

Vergleiche von selbst erstellten Klassen erfordern eine Implementierung des entsprechenden Operators. In deinem Falle von operator== .

int main()
{
  Singleton &gibtsnureinmal = Singleton::getInstance();
  Singleton &unddasauch = Singleton::getInstance();

  Singleton *ptr1 = &unddasauch;
  Singleton *ptr2 = &gibtsnureinmal;
  if(ptr1 == ptr2)
  {
    cout<<"Die beiden Objekte sind die gleichen, nur unter anderen Namen.";
    system("pause");
  }
  return 0;
}

Du scheinst ja schon sehr misstrauisch gegenüber C++ eingestellt zu sein. :p

Hast du ein gutes Tutorial für Umsteiger aus anderen Sprachen?
Ich bin von PHP sehr verwöhnt, wenn es Dokumentation angeht 30 Sekunden googlen und das Problem ist gelöst

Zu empfehlen sind hauptsächlich diese zwei:
1. C++ Primer
C++ Primer | ISBN: 9783827326126
2. Thinking in C++ 1&2: http://mindview.net/Books/TICPP/ThinkingInCPP2e.html

Zudem würde ich im Inet noch diese C++ Standardbibliothek Referenz empfehlen:
http://www.cpluslus.com/reference/

Sie ist allerdings nicht ganz vollständig. Fehlen tut allerdings nicht all zu viel und wenn man wirklich eine vollständige Dokumentation möchte, dann gibt es da auch gute Bücher

Grüssli

drakon schrieb:

Also in deinem Falle reicht das Meyers Singleton (von Dravere vorgestellt) völlig.
Wenn du tatsächlich über zerstörung bestimmen willst, dann empfehle ich dir das Phönix Singletonmuster anzuschauen.

Wie kommst Du aufgrund des Threads auf solch recht komplizierte Teile wie das Phönix Singleton? Er hat doch gar nicht nach der Lebensdauer des Singletons gefragt?!
Das Phönix Singleton macht nur Sinn, wenn es beim Beenden des Programms zu irgendwelchen Race Conditions zwischen verschiedenen, voneineinander abhängigen Singletons kommt. Das der TO solche Probleme hat, wage ich mal zu bezweifeln.
(Mal davon abgesehen, dass das P.S. auch keine Möglichkeiten bietet, explizit über den Zerstörungszeitpunkt zu bestimmen)