Hai Chris,
»» - Eigenschaften der Klasse sollten nicht public sein
Es kommt darauf an. Man kann sie durchaus public setzen und dann mittels Overloading entsprechend intervenieren. Somit spart man sich die Getter- und Setter-Methoden fuer Properties, die keine weiteren Behandlungen benoetigen.
Okay dann werde ich die Datenbank Daten static aber nicht Publik lassen.
Irgendwie ist da ein wenig der Wurm drin ;)
Zum einen ergibt es recht wenig Sinn die Datenbankverbindungsdaten als static zu deklarieren. Denn wenn eine Connection einmal anhand dieser Daten erstellt wurde dann ist das nachtraegliche Aendern dieser Daten wirkungslos.
Du koenntest die benoetigten Werte als Parameter der connect()-Methode definieren und somit auf die db_set_vars-Methode verzichten.
»» also hier das Beschaffen der Zugangsdaten und den Erstaufbau der Datenbank-Verbindung.
Aber es soll doch eine Singleton Klasse sein.
Siehe oben.
»» - Der Destruktor sollte für ein geeordnetes Lebensende der Klasse sorgen, also hier das
»» Beenden der Datenbankverbindung.
Mit der Verwendung von Destruktoren unter PHP muss man recht achtsam sein, denn "The destructor method will be called as soon as all references to a particular object are removed or when the object is explicitly destroyed or in any order in shutdown sequence.". Und ein Aufrufer wird nur in den seltesten Faellen mueszig genug sein und den Destruktor explizit aufrufen.
MfG,
Sympatisant
"If the future isn't bright, at least it is colorful"