mysqli: erweitern oder aggregieren?

Ein close kann dann sinnvoll sein, wenn keine DB Verbindung mehr benötigt wird und die restliche Scriptlaufzeit nach dem Zugriff relativ lang ist und man mit einer sehr hohen Aktivität sprich Zugriffsszahl sprich Userzahl rechnet.

In den meisten Fällen relativ uninteressant aber in speziellen Fällen durchaus sinnvoll.

[tim-gt]

Danke euch allen für die essentiellen Infos.

[h3ll]

Du meinst z.B. zu einer zweiten Datenbank?

Oder zu einem zweiten Server.

[tim-gt]

Was ist, wenn du irgendwann eine zweite Verbindung brauchst? Dann stehst du mit einem Singleton in einer Sackgasse.

Du meinst z.B. zu einer zweiten Datenbank?

[h3ll]

Genau das ist es, was mir nicht einleuchten will. Wieso denn keine Singletons? Denn wenn ich das Datenbankobjekt jeweils an die anderen Klassen weitergeben muss, bedeutet das doch zusätzliche Schreibarbeit? Und es wird ja auch hier immer auf das gleiche Objekt, also die gleiche Verbidung zugegriffen?!

Was ist, wenn du irgendwann eine zweite Verbindung brauchst? Dann stehst du mit einem Singleton in einer Sackgasse.

Ich muss vielleicht noch sagen, dass sich meine Projekte in kleinem Rahmen bewegen und es kaum zu erwarten ist, dass irgendwann Performance-Probleme auftreten werden, da wohl nie mehr als höchstens 2-3 Leute gleichzeitig auf einer Website sein werden. Oder ist das jetzt ein Scheinargument? :-)

Eine hohe Anzahl an Datenbankverbindungen können jede Pimpifax-Webseite stark verlangsamen.

[tim-gt]

close() brauchst du nicht. PHP schließt die Verbindung automatisch, wenn das Script beendet ist.

Auf Singletons solltest du verzichten. Übergib das Datenbankobjekt an die anderen Klassen oder Funktionen weiter.

Genau das ist es, was mir nicht einleuchten will. Wieso denn keine Singletons? Denn wenn ich das Datenbankobjekt jeweils an die anderen Klassen weitergeben muss, bedeutet das doch zusätzliche Schreibarbeit? Und es wird ja auch hier immer auf das gleiche Objekt, also die gleiche Verbidung zugegriffen?!

Ich muss vielleicht noch sagen, dass sich meine Projekte in kleinem Rahmen bewegen und es kaum zu erwarten ist, dass irgendwann Performance-Probleme auftreten werden, da wohl nie mehr als höchstens 2-3 Leute gleichzeitig auf einer Website sein werden. Oder ist das jetzt ein Scheinargument? :-)

[h3ll]

Ok, scheint wirklich die passendere Lösung zu sein..

Mir stellt sich in diesem Zusammenhang einfach noch die Frage, ob ich quasi mit dem Singletonmuster eine einzige Connection aufrechterhalten soll und diese ganz erst am Ende des Skripts via Destruktor und $this->connection->close() trennen soll (dann hätte ich ja für jede Abfrage die gleiche Thread ID, oder?) oder bei jeder Abfrage ein neues DB-Objekt instantiieren und dieses dann jeweils wieder zerstören (und damit die Verbindung schliessen - auch im Desktruktor) soll?!

Dass ich $this->result immer wieder mit free() bereinigen sollte, ist mir klar.

close() brauchst du nicht. PHP schließt die Verbindung automatisch, wenn das Script beendet ist.

Auf Singletons solltest du verzichten. Übergib das Datenbankobjekt an die anderen Klassen oder Funktionen weiter. Du solltest nicht mehrmals die gleiche Datenbankverbindung neu aufmachen. Das bremst ziemlich das Script aus.

[tim-gt]

Ok, scheint wirklich die passendere Lösung zu sein..

Mir stellt sich in diesem Zusammenhang einfach noch die Frage, ob ich quasi mit dem Singletonmuster eine einzige Connection aufrechterhalten soll und diese ganz erst am Ende des Skripts via Destruktor und $this->connection->close() trennen soll (dann hätte ich ja für jede Abfrage die gleiche Thread ID, oder?) oder bei jeder Abfrage ein neues DB-Objekt instantiieren und dieses dann jeweils wieder zerstören (und damit die Verbindung schliessen - auch im Desktruktor) soll?!

Dass ich $this->result immer wieder mit free() bereinigen sollte, ist mir klar.

Da ich nun schon drei Versionen einer solchen Klasse geschrieben habe, die zudem erfolgreich im Einsatz sind, lautet meine Empfehlung ebenfalls Methode 2.

[tim-gt]

Ist eigentlich schon so, ich möchte "mir selber" im eigentlichen Skript dann nur ein paar wenige, zugeschnittene Methoden erlauben, dafür aber immer sicher gehen, dass alle Eingaben sauber überprüft und nötigenfalls modifiziert werden und dass bei allen Fehlern eine sauber definierte Fehlerroutine aufgerufen wird. Ich frage mich einfach, ob ich so auch problemlos von Prepared Statements und all dem Zeug profitieren kann..

[mermshaus]

Kommt wohl drauf an, was du mit dem Objekt planst.

Beim Erweitern fügst du eben neue Methoden hinzu, die ursprünglichen Methoden von MySQLi bleiben jedoch vollständig ansprechbar.

Beim "Aggregieren" musst du jede MySQLi-Methode, die du nach außen hin sichtbar machen willst, in einer neuen Methode wrappen, kannst dafür aber genau festlegen, welche das sein sollen.

Ich denke mal, es gibt wenig Grund, MySQLi erweitern zu wollen. Wenn du planst, konkret zugeschnittene Model-Methoden wie getUserById() zu implementieren -- also den DB-Aufruf zu abstrahieren --, würde ich die zweite Lösung wählen.

[tim-gt]

Ich werde demnächst eine DB-Klasse (ausschliesslich MySQL), die auf der mysqli-Extension basiert, für meine Website coden und möchte euch diesbezüglich fragen, welchen Ansatz ihr besser findet und welche Vor- und Nachteile es gibt. Ich bin auf 2 Ansätze gestossen.

"Erweitern":

class DB extends MySQLi{
     // Konstruktor, etc... 

     /* Statement erzeugen */
    public function prepare($query){
         $stmt = new DBStmt($query);
         // Fehlerbehandlung etc... 
         return $stmt;

    /* Result erzeugen */ 
   public function query($sql){
         $this->real_query($sql);
         // Fehlerbehandlung etc...
         $result = new DBResult($this); 
         return $result; 
}  

class DBResult extends MySQLi_Result {//...}
class DBStmt extends MySQLi_Stmt {//...} 

"Aggregieren":

class DB {
    private $connection; 
    private $result; 
    public function __construct($host, $user, $pw, $db){
        $this->connection = new MySQLi($host, $user,$pw,$db);
    }
    
    public function query($sql){
        $this->result = $this->connection->query($sql);
        // Fehlerbehandlung etc.. 
    
    // z.B... 
    public function getResultAsObject{
        while($data = $this->result->fetch_object()){
               //....
        }
} 

Der erste Ansatz scheint mir auf den ersten Blick etwas komplizierter oder abstrakter, der zweite benötigt wahrscheinlich ein bisschen mehr Schreibarbeit - aber ich bin echt auf dem Holzweg, was besser für mich sein könnte.