ich wollte ein Globales Objekt erstellen, dafür habe ich mich für Singleton entschieden.

Aber wenn ich folgenden Code

ZufallsGenerator.h:

#ifndef ZUFALLSGENERATOR_H_
#define ZUFALLSGENERATOR_H_

#include <iostream>
#include <time.h>
#include <stdlib.h>

class ZufallsGenerator
{
	static ZufallsGenerator* m_generator;

	ZufallsGenerator();
	~ZufallsGenerator();

	public:
		static ZufallsGenerator* holeGenerator();
		int holeZufallsInt() const;

};

#endif /*ZUFALLSGENERATOR_H_*/

ZufallsGenerator.cpp:

#include "ZufallsGenerator.h"

ZufallsGenerator::ZufallsGenerator()
{
	time_t t;
	srand(static_cast<unsigned int>(time(&t)));
	rand();
}

ZufallsGenerator::~ZufallsGenerator()
{
}

ZufallsGenerator* ZufallsGenerator::holeGenerator() {
	return((ZufallsGenerator::m_generator) ? ZufallsGenerator::m_generator : ZufallsGenerator::m_generator=new ZufallsGenerator);
}
int ZufallsGenerator::holeZufallsInt() const {
	return(rand());
}

Und noch die main.cpp:

#include "ZufallsGenerator.h"

using namespace std;

int main()
{
	ZufallsGenerator* zg = ZufallsGenerator::holeGenerator();
	cout << zg->holeZufallsInt() << endl;
	cout << zg->holeZufallsInt() << endl;

	return 0;
}

schreibe.

Bekomm ich Folgende Fehlermeldung, bin leicht am Verzweifeln:

/tmp/ccenxSNS.o: In function ZufallsGenerator::holeGenerator()': ZufallsGenerator.cpp:(.text+0x1be): undefined reference toZufallsGenerator::m_generator'
ZufallsGenerator.cpp:(.text+0x1de): undefined reference to ZufallsGenerator::m_generator' ZufallsGenerator.cpp:(.text+0x1e3): undefined reference toZufallsGenerator::m_generator'
ZufallsGenerator.cpp:(.text+0x1ed): undefined reference to `ZufallsGenerator::m_generator'
collect2: ld returned 1 exit status

Vielen Dank schonmal

MfG

Patrick

ZufallsGenerator* ZufallsGenerator::m_generator = 0;

Allerdings würde ich abraten deinen Code zu benutzen. Das ist so schreklich..

1. Erzeugst du ein Memoryleak durch das fehlende delete auf das new

2. Ist auch, wenn du irgendwo den Speicher freigibst nicht Exceptionsicher.

3. Ist das ganze auch nicht generisch. Wenn du jetzt ein weiteres Singleton brauchst, musst du das Verhalten erneut programmieren, was auch nicht wirklich der Sinn ist..

Schau dir am besten einmal ein paar Singleton Implementierungen an. Z.B reicht das Meyers Singleton meist:
http://www.devarticles.com/c/a/Cplusplus/C-plus-plus-In-Theory-The-Singleton-Pattern-Part-I/4/

1. Erzeugst du ein Memoryleak durch das fehlende delete auf das new

Und das stört warum? Es kann ja nicht auflaufen. Es sind nur 32 Bytes, die bei Prozessende dann doch verschwinden.

2. Ist auch, wenn du irgendwo den Speicher freigibst nicht Exceptionsicher.

Ist mir gerade nicht aufgefalle. Wo ist das Problem genau?

3. Ist das ganze auch nicht generisch. Wenn du jetzt ein weiteres Singleton brauchst, musst du das Verhalten erneut programmieren, was auch nicht wirklich der Sinn ist..

Sehe ich nicht so. Es muß nicht allese generisch sein.

Schau dir am besten einmal ein paar Singleton Implementierungen an. Z.B reicht das Meyers Singleton meist:

Guter Tip.

ich wollte ein Globales Objekt erstellen, dafür habe ich mich für Singleton entschieden.

Dürfte Unfug sein. Deine Signletonverwaltung kostet mehr Rechenzeit und Aufwand und Speicher und bringt keine Sicherheit oder Übersicht dazu und ist in der main schwieriger zu bedienen.

#include <iostream>
#include <time.h> //<ctime>
#include <stdlib.h> //<cstdlib>

ZufallsGenerator::ZufallsGenerator()
{
	time_t t;
	srand(static_cast<unsigned int>(time(&t)));
	rand();
}

ZufallsGenerator::ZufallsGenerator()
{
	srand(time(0));
	rand();
}

Nett, daß Du das Muster srand();rand(); kennst. Nur sehr wenige tun das.

ZufallsGenerator::~ZufallsGenerator()
{
}

Hätte ich in der Klasse gemacht.

ZufallsGenerator* ZufallsGenerator::holeGenerator() {
	return((ZufallsGenerator::m_generator) ? ZufallsGenerator::m_generator : ZufallsGenerator::m_generator=new ZufallsGenerator);
}

Was soll das denn?

ZufallsGenerator* ZufallsGenerator::holeGenerator() {
	if(m_generator==0)
		m_generator=new ZufallsGenerator;
	return (((((((ZufallsGenerator::m_generator)))))));
}

Merke: return ist keine Funktion und bekommt keine Funktionsaufrufklammern.

int ZufallsGenerator::holeZufallsInt() const {
	return(rand());
}

Merke: return ist keine Funktion und bekommt keine Funktionsaufrufklammern.

Leider tut dein Generator gar nichts machen. Sinnvoll wird das erst, wenn er eigene Daten hat.Wenn Du also zwei Generatoren anlegen könntest, die dann auch zwei unabhängige Folgen liefern. Und genau dann verbietet sich wieder der SIngleton, weil Du auch brauchst, daß Du mehrere Generatoren haben kannst. Willst Du wirklich genau einen haben, hau ihn doch schnell als globale Variable hin. Aber jederzeit muß man sich für eine Aufgabe einen lokalen Generator holen dürfen.

drakon schrieb:

1. Erzeugst du ein Memoryleak durch das fehlende delete auf das new

Und das stört warum? Es kann ja nicht auflaufen. Es sind nur 32 Bytes, die bei Prozessende dann doch verschwinden.

?! - Wenn du das für sauber hälst..

volkard schrieb:

2. Ist auch, wenn du irgendwo den Speicher freigibst nicht Exceptionsicher.

Ist mir gerade nicht aufgefalle. Wo ist das Problem genau?

Naja. Wo räumt er denn den Speicher wieder auf? - Wenn überhaupt irgendwo am Ende einer Funktion und wenn eine Exception fliegt wird es (wahrscheinlich) nicht aufgeräumt. Klar, muss nicht sein. Mit genug Erfahrung geht es auch anderst, allerdings wäre mir persönlich das zu umständlich für so etwas, wie ein Singleton wo es bessere Lösungen gibt.

volkard schrieb:

3. Ist das ganze auch nicht generisch. Wenn du jetzt ein weiteres Singleton brauchst, musst du das Verhalten erneut programmieren, was auch nicht wirklich der Sinn ist..

Sehe ich nicht so. Es muß nicht allese generisch sein.

Nicht alles, aber ich finde, dass sich hier Vererbung sehr eignet.

Nicht alles, aber ich finde, dass sich hier Vererbung sehr eignet.

Daß im Kern einen Meyers-Singleton wohl sein wird, da sind wir uns einig, nehme ich an.

Die Zeilen

ZufallsGenerator* ZufallsGenerator::getInstance(){
  static ZufallsGenerator zuf;
  return &zuf;
}

tun keinem weh.

Willst Du die nur templatisieren mit

T* Singleton::getInstance(){
  static T sing;
  return &sing;
}

und reinmixen mit

class ZufallsGenerator::public Singleton<ZufallsGenerator>

?

Die Zeilen

ZufallsGenerator* ZufallsGenerator::getInstance(){
  static ZufallsGenerator zuf;
  return &zuf;
}

tun keinem weh.

Wobei ich mich wiederum frage warum man die Instanzanfrage public machen sollte (und dann stellt sich die Frage ob eine Referenz nicht sauberer wäre). Ich verwende (wenn ich den zum Singleton greife) nach außen hin eine statische Schnittstelle (Nach meiner Variante wäre ZufallsGenerator::holeZufallsInt eine statische Methode die sich intern die Instanz holt).

Dazu kommt halt nocht, dass ich es extrem sexy finde, wenn ein Singleton alleine dadurch machen kann, dass ich eine Vererbung anfüge. Es kommt wohl eher selten vor, dass ein Singleton notwendig ist, aber wenn, dann hat sich das mit einer Zeile erledigt.

Aber der Punkt ist nicht ohne Grund als letzter gekommen. Ist ja schlussendlich nur eine Entscheidung, die zu treffen ist, aber keinen groben Fehler darstellt. (natürlich können Punkte 1 und 2 auch gewollt sein, was aber eher unwahrscheinlich ist)

Diese Klasse hatte ich nur als Beispiel getest.

Ich wollte den Singleton dazu nutzen um auf eine PostgreSQl-Datenbank zu zugreifen.

Ich wollte verhindern, dass mehrere Zugriffe auf die Datenbank gemacht werden. Aus diesem Grund wollte ich eine Globale-Klasse als Datenbank-Verbindung.

In dieser Klasse habe ich dann noch die Abfrage vereinfachen, diese habe ich mit folgenden Code realisiert.

PGresult* Database::get_data(vector<string> sel,string table)
{
	if(!(this->initialze))
	{
		cout << "Database ist nicht initialsiert!" << endl;
	}
	else {
		PGresult *SQLExec;
		string get_data;
		get_data.append("select ");
		for(unsigned i=0;i<sel.size();i++) {
			get_data.append(sel[i]);
			if (i != sel.size() - 1)
				get_data.append(" ,");
		}
		get_data.append(" from ");
		get_data.append(table);
		SQLExec = PQexec(this->conn, get_data.c_str());
		return SQLExec;
}

Gruß
Patrick