spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Dieser Code ist furchtbar, vergiss ihn am besten.
Singletons sollte man nicht benutzen, und #define dafür erst recht nicht.
Fehlt die statische Methode CTimer::Get()? Oder ist die unter "Klassen Stuff"?

Das ist kein Singleton (siehe Wikipedia Singleton-Muster), sondern einfach eine Klasse mit einer nicht-statischen Methode. Es macht aber durchaus den Anschein, als sollte es ein Singleton darstellen Dir fehlt dazu aber die Implementierung der statischen Get-Methode. Über diese gibst du das die einzige Instanz dieser Klasse zurück. Sicherstellen, dass es die einzige Instanz ist, kannst Du zum Beispiel, indem Du den Konstruktor privat machst. Dann kann nur die Klasse selbst eine Instanz erzeugen.

Statisch bedeutet, dass die Methode nicht einer Instanz einer Klasse zugeordnet ist. Das Besondere an dieser Methode ist, dass ihre Adresse fest ist, wie bei einer Funktion, daher kann man sie auch über ihre Adresse aufrufen! (Sprich, man muss keine Instanz erzeugen) Das geht bei nicht-statischen Methoden nicht, da die Aufrufadresse vom Aufruf abhängt.

In deinem Fall hast du sie schlicht als nicht-statisch deklariert, daher ist es logisch, dass der Compiler hier meckert. Du müsstest demnach erst eine Instanz dieser Klasse erzeugen und könntest dann über diese die Methode aurufen.

Was Dir der Autor eigentlich zeigen wollte (und ja, es ist grausam), ist, dass du mit Hilfe der Präprozessordirektiven #define Tipparbeit sparen kannst. Der Präprozessor durchläuft deinen Code vor dem Compiler und ist ein reiner "Textverarbeiter". Er ersetzt nachdem er das

#define g_pTimer CTimer::Get ()

gelesen hat überall den String "g_pTimer" in deinem Quelltext durch "CTimer::Get ()". Warum man das gut findet kann ich Dir nicht sagen. Vermutlich weil der Autor es ärgerlich/hässlich findet immer "CTimer::Get ()" zu tippen.

Das Macro

#define g_pTimer CTimer::Get ()

ist dafür gedacht damit die Verwendung des Singletons aussieht als ob man eine globale Variable verwendet.

Was natürlich sehr unschön ist, wenn man sogar zwei Sünden statt nur einer benutzt. Es hätte dort auch gleich eine (schlechte) globale Variable verwendet werden können. Aber wie David schon sagte, am besten diesen ganzen Stuss von dieser Stelle vergessen und nie verwenden.

Ob man Singletons verwenden sollte oder nicht ist immer Geschmackssache. Ich persönlich finde, dass es sinnvolle Anwendungsfälle gibt, sie aber leicht dazu verleiten, sie im Übermaß zu gebrauchen. Man liest immer wieder die Aussage, dass man statt einem Singleton gleich globale Variablen werden kann. Das stimmt leider nicht. Gegenüber einer globalen Variable haben Singletons den entscheidenen Vorteil, dass man sie zustandssicher entwerfen kann.

Ganz plattes Beispiel:
Stell Dir vor Du legst beispielweise den Spielmodus als Ganzzahl ab, von 0 bis 5 z.B. Bei einer globalen Variable hindert dich nichts daran, einfach eine 6 oder -1 hineinzuschreiben und damit dein Programm in einen ungültigen Zustand zu versetzen. Du müsstest dann an allen Stellen, wo Du diese Variable verarbeitest, deinen Code fehlertolerant machen oder dir etwas anderes in der Art einfallen lassen. Zumindest ist es immer eine schlechte Idee, fehlerhafte Eingaben zu akzeptieren und im Nachgang zu versuchen, sie "intelligent" wieder zu korrigieren, das geht in die Hose.

Zitat von »mint«

Ob man Singletons verwenden sollte oder nicht ist immer Geschmackssache.

Nein, eigentlich ist das keine Geschmackssache, denn Singleton hat einen einzigen Zweck. Wenn dieser nicht benötigt wird, sind sie Unfug und nichts weiter als versteckte globale Variablen. Da ist eine globale Variable jederzeit sinnvoller als ein Singleton und man würde sie auch direkt als solche erkennen. Wir reden hier schließlich auch nicht von einem einfachen globalen Integer (denn ein Singleton ist keiner!), sondern von einer Klassen-Instanz. Ob ich die per "Classtype::getInstance()" hole oder direkt über den Namen der globalen Variable verwende, macht funktional gesehen keinen Unterschied. Semantisch betrachtet allerdings schon. Die aufgerufenen Methoden sorgen übrigens für den korrekten Zustand des Objekts und da ist es egal, ob dieses ein Singleton ist oder eine globale Variable. Ein Singleton hat nur einen Zweck, nämlich dass es unter keinen Umständen mehr als eine Instanz geben darf. Wird dieser nicht benötigt, ist das Singleton architektonischer Unsinn und eine globale Variable tut es genauso und das deutlich klarer.

Die statische Methode steckt in TSingleton.

Hast du die singleton.hpp in deinem Projektordner drinnen?