Singleton ohne Boost

hustbaer

Achja, in deiner CriticalSection und SectionLocker Klasse sind natürlich noch Fehler drinnen. Ist mir beim 1. mal ansehen nicht aufgefallen.

CriticalSection braucht keinen parameter im ctor - das CRITICAL_SECTION Member wird ja eh mit InitializeCriticalSection initialisiert.
Die SectionLocker Klasse muss natürlich im ctor nen CriticalSection& Parameter haben und nicht CRITICAL_SECTION& .
Die SectionLocker Klasse darf die CriticalSection natürlich nicht by value speichern, sondern muss ne Referenz/nen Zeiger halten.
EDIT:
Beide Klassen müssen natürlich non-copyable sein.

Sind das Abschreibfehler? Weil eigentlich dürfte das was du hier gepostet hast nichtmal kompilieren.

stuxn

FrankTheFox schrieb:

Hi Folks,

Ich möchte eine "Application-Klasse erstellen, die ich auch von mehr al einem Thead nutzen kann. Also habe ich mit folgendes überlegt:
class Application
{
    public:
        Application(){}
        ~Application();

        static Application &getInstance();

    private:
        Application( const Application& );
        Application& operator=( const Application& );
};

Hey,

dir ist aber schon bewusst was ein Singleton ist oder?
Weil mit deinen gezeigten Code kann man x Objekte von Application erstellen.

2. Warum glaubst du das du ein singleton benoetigst? Viele Leute verwenden das Singleton Pattern falsch. imho bestes Beispiel: logging.
Aber liess dir ab besten ein paar Post im Forum durch bzw. such mal bei google nach singleton pattern

FrankTheFox

stuxn schrieb:

Hey,
...
dir ist aber schon bewusst was ein Singleton ist oder?
Weil mit deinen gezeigten Code kann man x Objekte von Application erstellen.

Ja, dachte ich bis jetzt immer. Ein Singleton lässt nur eine Instanz zu.

stuxn schrieb:

2. Warum glaubst du das du ein singleton benoetigst? Viele Leute verwenden das Singleton Pattern falsch. imho bestes Beispiel: logging.
Aber liess dir ab besten ein paar Post im Forum durch bzw. such mal bei google nach singleton pattern

Ich dachte, das ich nur eine Application-Klasse im kompletten Programm nutzen will. Funktioniert wahrscheinlich ( oder garantiert ) wenn die Application-Klasse global ist. Wobei ich mich frage was am Singleton so böse ist?

Greetz

Th69

s, z.B. Singleton (Entwurfsmuster) #Nachteile

FrankTheFox

Mal aus der Hüfte geschossen:

namespace blablupp {
  namespace {
     Application app;
  }

  Application* get_App() {
      return &app;
  }
}

blablupp::get_App()->funcX();

Ein globaler Namespace... (in der Übersetzungseinheit in der auch WinMain(...).. liegt)

Sollte es auch tun. Oder ist das völliger Schrott. Wäre dann nur noch das Absichern ganz ohne Singleton. Ein Singleton sollte aber auch Vorteile habe. Weil nur Nachteile (Antipattern...)

Greetz

stuxn

FrankTheFox schrieb:

Mal aus der Hüfte geschossen:
namespace blablupp {
  namespace {
     Application app;
  }

  Application* get_App() {
      return &app;
  }
}

blablupp::get_App()->funcX();
Ein globaler Namespace... (in der Übersetzungseinheit in der auch WinMain(...).. liegt)

Sollte es auch tun. Oder ist das völliger Schrott. Wäre dann nur noch das Absichern ganz ohne Singleton. Ein Singleton sollte aber auch Vorteile habe. Weil nur Nachteile (Antipattern...)

Greetz

Was sollte das deiner Meinung nach bewirken? Ich versteh nicht was du machen willst.

Gruessle

stuxn

Falscher button...

FrankTheFox

Es soll eine und nur eine Instanz von der Application-Klasse möglich sein. Weil es nur eine App geben kann. Nun ja, wenn Singleton so böse sind....

hustbaer

Singleton oder normal globale Variable oder "lokale" Variable die über ne Getter-Funktion global erreichbar gemacht wird -- macht fast keinen Unterschied.
Böse sind sie alle (*). Weil die Probleme die von allen gleichermassen verursacht werden viel schwerer wiegen als die Probleme die die eine Variante löst, die andere aber nicht.

Eines der schlimmsten ist, dass es zu wuchernden Abhängigkeiten kommt. Wodurch Wiederverwendbarkeit jeder Art gezielt vernichtet wird. Und mit "jeder Art" meine ich auch so simple Dinge wie Copy & Paste Re-Use.
Hmmm, ich brauch da ne Einstellung in FunktionDieAnsichSchoenWiederverwendbarWaere() . Ach, die hol ich mir gleich in der Funktion aus den globalen Applikations-Settings, macht ja keinen Sinn die überall rumzureichen.
Boom, you're dead.

FrankTheFox schrieb:

Weil es nur eine App geben kann.

Erstmal bestreite ich das grundsätzlich. Und dann ... selbst wenn es nur eine App geben kann, das heisst noch lange nicht dass alle Klassen bzw. Funktionen die man zur Entwicklung dieser App entwickelt auch auf die App Zugriff haben sollten. Weil wenn sie es doch haben, naja, siehe Beispiel oben.

Und dabei geht es nicht nur um Code Re-Use. Es führt ganz allgemein zu schrecklich unübersichtlichen Programmen. Eine weitere schlimme Sache die man damit "schön" machen kann, also wo man in Versuchung gerät es zu machen, weil's ja "so praktisch ist", ist das "setzen und aufrufen" Anti-Pattern. (Das Ding hat offiziell nen anderen Namen, aber der ist mir leider entfallen. Gemeint ist Code ala app.SetSomeOption(option); someClass.PerformSomeOperationThatUsesTheOptionWeJustSet(); - *schauder*)

*:
Ausser wenn sie es nicht sind.
(Manchmal, in bestimmten Fällen, ganz selten, kann so ein Singleton die Wiederverwendbarkeit z.B. sogar verbessern. Aber hey, Ausnahmen gibt es immer. Grundsätzlich gilt aber: das was du gerade machst ist nicht eine dieser Ausnahmen. Es sei denn es ist doch eine, aber es ist eben keine. ;))

FrankTheFox

hustbaer schrieb:

...

FrankTheFox schrieb:

Weil es nur eine App geben kann.

Erstmal bestreite ich das grundsätzlich.
...

Ok. Das mit dem Singleton ist tatsächlich keine gute Idee, weil die beschriebenen Probleme stärker wiegen als die Vorteile. Bislang hatte ich auch keine sinnvolle Verwendung für das Pattern. Was jetzt nicht heißen soll, nun muß das aber in den Kode weil sonst taugt der nix. Aber wenn ich eine Application-Klasse habe, wieso sollte es dann aber im Programm eine weitere geben? Die Klasse nutze ich ja um meinen Anwendungsspezifischen Krams reinzulegen. Darum dachte ich auch, jetzt wäre eine gute Gelegenheit gekommen das Muster zu nutzen.

Sonst mache ich die App (wenn es denn sein muss) global (->Namespace).

Gruß