Singleton vs. Registry - Seite 2

Manko10 · 19.11.2008, 21:44

Vielleicht meint er ja auch eine Fabrikmethode.

Ich denke aber viel eher, dass er vorhat, eine Instanz zu erzeugen und diese per Registry zu speichern.

dr.e. · 19.11.2008, 21:52

Hallo nerere,

Zitat:

Bei mir stellt sich gerade die Frage, wie ich in meinem Projekt den Umgang mit dem Datenbank-Objekt regle. Wie sinnvoll ist es, dafür eine extra "Registry" anzulegen? Gibt es Vorteile gegenüber der Methode, die MySQL-Klasse als Singleton auszulegen?

Ich würde hier nicht mit einer Registry arbeiten, sondern - wie angesprochen - mit einer Factory. Grund: die Datenbank-Abstraktion folgt einem durchgängigen Muster. Weiterhin würde ich aus Performance-Gründen dafür sorgen, dass eine dedizierte Instanz ein Singleton ist, so muss die Verbindung zur Datenbank beispielsweise nur einmal aufgebaut werden.

Im APF ist dazu der connectionManager enthalten, der gemäß einer Konfiguration konkrete Instanzen erzeugt und verwaltet. Das hat den weiteren Vorteil, dass die gelieferten Datenbank-"Treiber" nicht nur pro Verbindung eineindeutig sind, sondern auch die Art der Datenbank noch weiter abstrahiert werden kann. Sicher nicht komplett, aber weitestgehend.

Gegen Registry spricht meiner Ansicht nach auch, dass sich die Teile deiner Software, die sich mit Datenhaltung beschäftigen darauf verlassen müssen, dass deine Registry relevante Datenbank-Zugriffsschichten enthalten. Das erfordert manualles Eingreifen in der Bootstrap-Datei und entkoppelt damit die Anwendungen wieder weniger voneinander, da du Abhängigkeiten nicht getrennt behandeln kannst sondern manuell auflösen musst. Gehst du wie im zweiten Absatz vor, kann jede Komponente / Software selbst entscheiden, welche Datenbank-Verbindung sie nutzt und es ist auf einfache Weise möglich, in den unterschiedlichen Teilen deiner Anwendung sogar ohne globale Abhängigkeiten auf unterschiedliche Datenbanken zuzugreifen.

Gestaltest du den ConnectionKey innerhalb deiner Anwendung noch konfigurierbar, so kannst du sogar pro "Einsatz" der Applikation unterschiedliche Connections nutzen. "Einsatz" meint hier, wo dein Modul eingesetzt ist.

Just my 2 cent.

nerere · 20.11.2008, 20:02

Hallo!

Zu aller erst vielen Dank für die vielen Antworten!

Ich werde nur eine einzige Datenbank-Verbindung und damit auch nur eine einzige Instanz meiner Datenbank-Verbindungsklasse brauchen. Verschiedene Treiber etc. sind eigentlich nicht nötig, da die Datenbank auf jeden Fall MySQL sein wird.

Mir ist leider noch nicht ganz klar, wie eine Factory mir bei dem Problem mit der eindeutigen (einzigen) Instanz weiterhelfen kann. Oder war diese nur darauf bezogen, dass ich damit evtl. verschiedene Treiber verwenden könnte? Eine Factory kümmert sich doch eigentlich gar nicht darum, wieviele Instanzen eines Objektes es bereits gibt.

Zitat:

Zitat von nikosch

Das meinte ich nicht (ginge aber auch) sondern eine Managementklasse, die Datenbanken erzeugt und zurückgibt und in einem statischen Member verwaltet.

Genau an so eine "Managementklasse" dachte ich, als ich von einer Registry sprach. Oder ist das falsch?

Um noch einmal zusammenzufassen, was mein Stand ist: Ich verstehe es so, dass es zwei Methoden gibt, um sicherzustellen, dass z.B. die Datenbankverbindung nur einmal instanziiert wird.

Einmal durch die Klasse selbst: Singleton
Und einmal "global" (und somit evtl. auch für andere Klassen): per Registry

Viele Grüße,
nerere

nikosch · 20.11.2008, 20:15

Ich persönlich finde es stimmiger, statt einer Registry (ja irgendwie nur ein GLOBALS Ersatz) eine DatenbankManagement-Klasse mit statischen Methode zu verwenden. Die brauchst Du nicht instanziieren und kannst trotzdem, bei Bedarf, mehrere Datenbakobjekte verwalten.

Etwas roughes Beispiel:

PHP-Code:

  <?php
// erster Aufruf, ruft intern self::create () auf
$oDB = DatabaseManager::get ('mysql');
$oDB->query ( ... ); // etc.

function anyfunc ()
  {
  $oDB = DatabaseManager::get ('mysql');
  ...
  }

class DatabaseManager
  {
  static $aDatabases = array ();

  static function create ($key)
    {
    $aSetting = array ('mysql' => array (
                                        'mysql' , 
                                        'host' , 
                                        'user' , 
                                        'pass'
                                        ) ,

                       // andere Datenbanken
                       );

    switch ($aSetting[$key][0])
      {
      case 'mysql':
      default:
        self::$aDatabases[$key] = new MysqlDatabase ($aSetting[$key][1] , ...);
        break;
      }
    // ginge auch so:
    // $class = $aSetting[$key][0] . 'Database';
    // self::$aDatabases[$key] = new $class ($aSetting[$key][1] , ...);
    }

  static function get ($key)
    {
    // Quasi-Singleton
    if (false === isset (self::$aDatabases[$key]))
      {
      self::create ($key);
      }
    return (self::$aDatabases[$key]);
    }

  }

Phoscur · 01.12.2008, 16:12

Thema is ja noch frisch, darf ich noch dies anhängen:
Ich empfehle dir auf jeden Fall den Singleton, ob parameterisiert oder nicht. Zudem solltest du, wenn du schon "nur" mit MySQL und vor allem PHP5 arbeitest mysqli verwenden. Dabei gibt es ein kleines Problem mit dem Contructor, hier mein Code:

PHP-Code:

  class DB extends mysqli
{
// [...]
    /**
     * Unique Instance of Database
     * @var DB
     */
    private static $Instances = array();
    /**
     * Database Contructor
     * Returns the unique Database instance
     * @return DB
     */
    public static function getInstance($id=0)
    {
        if (self::$Instances[$id] === NULL)
        {
            self::$singletonacc = true;
            self::$Instances[$id] = new self();
        }
        return self::$Instances[$id];
    }
    protected static $singletonacc = false; # needed due to __construct problem..
    private final function __clone() {}
    /**
     * PROTECTED!
     */
    public function __construct() # due to some PHP "bug" I can't declare the method protected/private
    {
        if (!self::$singletonacc)
        {
            throw new DBException('Use ::getInstance() to get the DB object!');
        }
        $cnfg = new DBConfig();
        @parent::__construct($cnfg->host,$cnfg->user,$cnfg->passwd,$cnfg->dbname);
        if (mysqli_connect_error())
        {
            throw new DBException('Connection failed ['.mysqli_connect_errno().']:'.mysqli_connect_error());
        }
        self::$singletonacc = false;
    }
// [....]
}

Wie du das mit der Config machst, ist natürlich dir überlassen. Dies ist nur eine Möglichkeit. Die "Multi-"Singleton habe ich auch erst seit kurzem, weil ich etwas testen wollte. Ich könnte ihn eigentlich wieder ausbauen, weil ich ihn nicht brauche, aber so stört es ja nicht.

dr.e. · 01.12.2008, 16:56

Zitat:

Die "Multi-"Singleton habe ich auch erst seit kurzem, weil ich etwas testen wollte. Ich könnte ihn eigentlich wieder ausbauen, weil ich ihn nicht brauche, aber so stört es ja nicht.

Ich denke du spielst hier eher auf eine singleton instanzierte Factory an?

Phoscur · 01.12.2008, 17:06

Nein, ich rede nur von einer Datenbankklasse auf mysqli Basis mit Singleton. Das "Multi-" habe ich dran gebaut, weil dieser Singleton mehr als eine einzige Instanz unterstützt.

Der Aufbau oben ist für die meisten Verwendungszwecke unsinnig, weil er zwar mehrere Instanzen, aber nur eine Config erlaubt. Das müsste man ändern, wenn man doch mehrere Verbindungen verwaltet. Mir ging es aber eher darum zu zeigen, wie man mysqli mit singleton verwenden kann.

Dennoch ist dies eindeutig keine Fabrik.

dr.e. · 01.12.2008, 17:08

Zitat:

Das "Multi-" habe ich dran gebaut, weil dieser Singleton mehr als eine einzige Instanz unterstützt.

Deswegen frage ich ja nach, denn das wiederspricht sich etwas... Dass die Implementierung nichts mit dem Factory-Pattern zu tun hat, ist auch mir klar.

Manko10 · 01.12.2008, 18:38

@Doc: seit wann gibt es eigentlich ein Fabric-Muster? Unter Fabric verstehe ich ein Computernetzwerk, aber kein Design-Pattern. Meinst du nicht eher Factory oder Fabrik?

dr.e. · 01.12.2008, 20:21

@Manko: shame on me! Habs ausgebessert...

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Strukturierung größerrer Projekte	#Avedo	Software-Design	19	24.10.2008 11:10
[Erledigt] Singleton Fehler	Andreas	PHP-Fortgeschrittene	3	17.10.2008 16:27
referentieller T_PAAMAYIM_NEKUDOTAYIM	nikosch	PHP-Fortgeschrittene	31	28.05.2008 00:35
Gegenstück zu parent:: oder Singleton abstrakt implement. ??	Melchior	PHP-Fortgeschrittene	13	26.07.2007 21:25
Verständnisfrage Singleton in PHP	ve.ru	PHP Tipps 2006	3	13.06.2006 20:36
Singleton besser als komplett statische Klasse ?	Cmon	PHP-Fortgeschrittene	4	13.06.2006 00:34
Singleton Klasse - falsche Werte		PHP Tipps 2005	20	24.05.2005 12:07
USB-Ports per Registry disablen	Orolhawion	Off-Topic Diskussionen	5	23.08.2004 11:42

19.11.2008, 21:44
Manko10 Supermoderator HD Registriert seit: 16.03.2008 Beiträge: 8.709 PHP-Kenntnisse: Fortgeschritten	Vielleicht meint er ja auch eine Fabrikmethode. Ich denke aber viel eher, dass er vorhat, eine Instanz zu erzeugen und diese per Registry zu speichern. __________________ Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

20.11.2008, 20:15
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.994 PHP-Kenntnisse: Fortgeschritten	Ich persönlich finde es stimmiger, statt einer Registry (ja irgendwie nur ein GLOBALS Ersatz) eine DatenbankManagement-Klasse mit statischen Methode zu verwenden. Die brauchst Du nicht instanziieren und kannst trotzdem, bei Bedarf, mehrere Datenbakobjekte verwalten. Etwas roughes Beispiel: PHP-Code: <?php // erster Aufruf, ruft intern self::create () auf $oDB = DatabaseManager::get ('mysql'); $oDB->query ( ... ); // etc. function anyfunc () { $oDB = DatabaseManager::get ('mysql'); ... } class DatabaseManager { static $aDatabases = array (); static function create ($key) { $aSetting = array ('mysql' => array ( 'mysql' , 'host' , 'user' , 'pass' ) , // andere Datenbanken ); switch ($aSetting[$key][0]) { case 'mysql': default: self::$aDatabases[$key] = new MysqlDatabase ($aSetting[$key][1] , ...); break; } // ginge auch so: // $class = $aSetting[$key][0] . 'Database'; // self::$aDatabases[$key] = new $class ($aSetting[$key][1] , ...); } static function get ($key) { // Quasi-Singleton if (false === isset (self::$aDatabases[$key])) { self::create ($key); } return (self::$aDatabases[$key]); } } __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. -- Geändert von nikosch (20.11.2008 um 20:21 Uhr).

01.12.2008, 16:12
Phoscur Erfahrener Benutzer Registriert seit: 01.12.2008 Beiträge: 450 PHP-Kenntnisse: Fortgeschritten	Thema is ja noch frisch, darf ich noch dies anhängen: Ich empfehle dir auf jeden Fall den Singleton, ob parameterisiert oder nicht. Zudem solltest du, wenn du schon "nur" mit MySQL und vor allem PHP5 arbeitest mysqli verwenden. Dabei gibt es ein kleines Problem mit dem Contructor, hier mein Code: PHP-Code: class DB extends mysqli { // [...] /** * Unique Instance of Database * @var DB / private static $Instances = array(); /* * Database Contructor * Returns the unique Database instance * @return DB / public static function getInstance($id=0) { if (self::$Instances[$id] === NULL) { self::$singletonacc = true; self::$Instances[$id] = new self(); } return self::$Instances[$id]; } protected static $singletonacc = false; # needed due to __construct problem.. private final function __clone() {} /* * PROTECTED! / public function __construct() # due to some PHP "bug" I can't declare the method protected/private { if (!self::$singletonacc) { throw new DBException('Use ::getInstance() to get the DB object!'); } $cnfg = new DBConfig(); @parent::__construct($cnfg->host,$cnfg->user,$cnfg->passwd,$cnfg->dbname); if (mysqli_connect_error()) { throw new DBException('Connection failed ['.mysqli_connect_errno().']:'.mysqli_connect_error()); } self::$singletonacc = false; } // [....] } Wie du das mit der Config machst, ist natürlich dir überlassen. Dies ist nur eine Möglichkeit. Die "Multi-"Singleton habe ich auch erst seit kurzem, weil ich etwas testen wollte. Ich könnte ihn eigentlich wieder ausbauen, weil ich ihn nicht brauche, aber so stört es ja nicht. Geändert von Phoscur (01.12.2008 um 17:09 Uhr).*

01.12.2008, 17:06
Phoscur Erfahrener Benutzer Registriert seit: 01.12.2008 Beiträge: 450 PHP-Kenntnisse: Fortgeschritten	Nein, ich rede nur von einer Datenbankklasse auf mysqli Basis mit Singleton. Das "Multi-" habe ich dran gebaut, weil dieser Singleton mehr als eine einzige Instanz unterstützt. Der Aufbau oben ist für die meisten Verwendungszwecke unsinnig, weil er zwar mehrere Instanzen, aber nur eine Config erlaubt. Das müsste man ändern, wenn man doch mehrere Verbindungen verwaltet. Mir ging es aber eher darum zu zeigen, wie man mysqli mit singleton verwenden kann. Dennoch ist dies eindeutig keine Fabrik. __________________

01.12.2008, 18:38
Manko10 Supermoderator HD Registriert seit: 16.03.2008 Beiträge: 8.709 PHP-Kenntnisse: Fortgeschritten	@Doc: seit wann gibt es eigentlich ein Fabric-Muster? Unter Fabric verstehe ich ein Computernetzwerk, aber kein Design-Pattern. Meinst du nicht eher Factory oder Fabrik? __________________ Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”

01.12.2008, 20:21
dr.e. Moderator und Wett-König Registriert seit: 21.05.2008 Beiträge: 3.657 PHP-Kenntnisse: Fortgeschritten	@Manko: shame on me! Habs ausgebessert... __________________ Viele Grüße, Dr.E. ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1. Think about software design before you start to write code! 2. Discuss and review it together with experts! 3. Choose good tools (-> Adventure PHP Framework (APF))! 4. Write clean and reusable software only! ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~