DB-Verbingung als construct oder als Objekt?

[Leyyin]

Vielleicht muss ich von Anfang an anfangen, damit ich meine Code operieren kann.
Denn ich habe gemerkt, dass eine falsche Denkweise andere falsche Denkweisen bringt.

Gut, dann fange ich von meinem DB-Verbindung an:

Zuerst zu den Situation:

Ich versuche gerade für einen Verein eine Formular zu erstellen,
wo die Daten einer Person, die zu einer Aktivität von Verein eingeladen wird,
von den Mitarbeitern eingegeben werden sollen.

So, diese Formular beinhaltet die Eingabefelder wie :

• Titel
• Vorname
• Nachname
• Adresse
• Telefonnummer
• E-Mail adresse
• etc...

Neue Persondaten sollen in DB gespeichert werden, wenn die eingegebene E-Mail-Adresse
noch nicht in DB existiert. Andernfalls ein Update soll durchgeführt werden.

So, dafür habe ich 4 php-Skripten:

1. person-neu.php
2. Person.php //Eine Klasse
3. Formular.php //Eine Klasse (Formular und Validierung steht hier)
4. SqlQuery.php // Eine Klasse (Diese beinhaltet alle Querys)

Meine Frage ist: "Wo muss ich die Datenbankverbindung erstellen?"
Eine DB-Verbindung brauche ich ja für jeden Skript ein mal.
Ich muss wahrscheinlich nicht in jedem Skript folgende Code schreiben müssen, oder?

PHP-Code:

$db = @new mysqli('localhost', 'username', 'password', 'database');
if (mysqli_connect_errno()) {
    die ('Konnte keine Verbindung zur Datenbank aufbauen: '.mysqli_connect_error().'('.mysqli_connect_errno().')');
} 


ich dachte, es wäre vielleicht besser einen Konstruktor für die DB verbindung zu schreiben.
Immer wenn eine neue Objekt erstellt wird, muss dann die DB-Verbindung automatisch hergestellt werden.

Nur wo muss ich diese dann erstellen?
In Person.php oder in SqlQuery.php?
weil ich ein Objekt $db dann in beiden und auch in person-neu.php brauchen werde.

Wie macht ihr das?

[mgr]

Hallo Leyyin

die meiner Meinung nach beste und sauberste Möglichkeit, ist das Singleton Pattern anzuwenden.

Beispiel:

PHP-Code:

class db {
    private static $dbObj;
    
    private function __construct(){}
    
    public final function __clone() {
        throw new BadMethodCallException("Clone is not allowed");
    }
    
    public static function getInstance() {
        if(!(self::$dbObj instanceof mysqli)) {
            self::$dbObj = new mysqli(DB_HOST, DB_USER, DB_PASSWORD, DB_NAME);
        }
        
        return self::$dbObj;
    }
} 


In den Klassen, in denen du eine DB-Verbindung brauchst, kannst du eine private Member-Variable $_db halten,
und ihr im Konstruktor der Klasse eine Instanz von "db" zuordnen. Z.B. :

PHP-Code:

public function __construct() {
    $this->_db = db::getInstance();
}

public function query($sql) {
    $result = $this->_db->query($sql)
    [...]
} 


So wird nicht für jede Query eine neue Verbindung aufgebaut, du musst keine globalen Variablen benutzen und kannst in jeder beliebigen
Klasse eine DB-Verbindung herstellen.

Wenn du einen Schritt weiter gehen willst, kannst du dich auch zum Thema "MVC" schlau machen. Sobald du das im Griff hast, wird es dir
eine grosse Hilfe beim Design deiner Applikation sein. Geht bei Zeit- oder Lust-Mangel für kleinere 1-Mann Projekte aber auch ohne.

bei Google findest du haufenweise weitere Beispiele, wenn du nach den Begriffen"php mysql singleton" suchst.



Gruss
mgr

[Leyyin]

Zuerst danke für deine Antwort mgr,

Zitat von mgr

In den Klassen, in denen du eine DB-Verbindung brauchst, kannst du eine private Member-Variable $_db halten,
und ihr im Konstruktor der Klasse eine Instanz von "db" zuordnen...

Hmm... Dann muss ich wahrscheinlich in dieser anderen Klasse die Klasse "db"
per include oder require verbinden, oder?

Nur, wo kann ich diese Code am besten schreiben?

PHP-Code:

//So?:
class Beispiel{

   require 'class/db.php';

}

//oder so?:

require 'class/SqlQuery.php';
class Beispiel{
    //...
} 


Zitat von mgr

Wenn du einen Schritt weiter gehen willst, kannst du dich auch zum Thema "MVC" schlau machen.

Das ist meine erste Projekt und bin leider immer noch ziemlich am Anfang.
Es wäre vielleicht besser, wenn ich zuerst meine erste Programm lauffähig hinkriege
und mich erst dann mit Erweiterungen bzw. Optimierungen beschäftige.
Wenn also diese "MVC" keine Lebenswichtige Thema ist, geht es dass ich es
bisschen in die Zukunft verschiebe? :)

[mgr]

Ahoihoi

Zitat von Leyyin

Hmm... Dann muss ich wahrscheinlich in dieser anderen Klasse die Klasse "db"
per include oder require verbinden, oder?

Nur, wo kann ich diese Code am besten schreiben?

PHP-Code:

//So?:
class Beispiel{

   require 'class/db.php';

}

//oder so?:

require 'class/SqlQuery.php';
class Beispiel{
    //...
} 


die 1. Variante ist sicher falsch, wenn du die Datei innerhalb der Klasse einfügen willst, dann im konstruktor oder in einer funktion. Ich nehme immer die 2. Variante, also alle includes und requires ganz oben (mit wenigen Ausnahmen).
Ob eine der beiden Variante jetzt Vor- oder Nachteile haben kann ich dir nicht sagen.

Alternative: autoloader verwenden

Zitat von Leyyin

Das ist meine erste Projekt und bin leider immer noch ziemlich am Anfang.
Es wäre vielleicht besser, wenn ich zuerst meine erste Programm lauffähig hinkriege
und mich erst dann mit Erweiterungen bzw. Optimierungen beschäftige.
Wenn also diese "MVC" keine Lebenswichtige Thema ist, geht es dass ich es
bisschen in die Zukunft verschiebe? :)

Bei kleinen Projekten mit einer handvoll Scripts kann man da sicher ein Auge zudrücken. Aber sobald etwas grösser wird kann ein gutes Verständnis von OO-Design und Patterns Programmierer-Leben retten.
Es ist eigentlich wie wenn du 10 Shots auf Ex trinkst: man hat eine Menge Spass, aber bereuts vielleicht im nachhinein. Oder so :)

Gruss
mgr


/add:
Ist mir grad so eingefallen: deine nächste Frage könnte sein "aber wo füge ich das __autoload() ein?"

Vorschlag: mach dir ein Script "library.php" oder so, dort pappst du die funktion und alle andere Helfer-Funktionen rein.
Dann erstellst du ein Script "base.inc.php" wo du all die Dateien einbindest, die du ständig brauchst (wie hier "library.php").
Die "base.inc.php" bindest du dann einfach in jedem Script, das direkt aufgerufen wird, mit "require_once" ein.
Das macht vermutlich jeder Entwickler ein bisschen anders.
Wenn du Zeit und Lust hast, kannst du mal im Internet nach "Best practice php" suchen. So findest du vielleicht den einen
oder anderen Tip der dir das Programmieren etwas erleichtert.

[Leyyin]

Zitat von mgr

In den Klassen, in denen du eine DB-Verbindung brauchst, kannst du eine private Member-Variable $_db halten,
und ihr im Konstruktor der Klasse eine Instanz von "db" zuordnen. Z.B. :

PHP-Code:

public function __construct() {
    $this->_db = db::getInstance();
}

public function query($sql) {
    $result = $this->_db->query($sql)
    [...]
} 


Ein neues Problem:
Fatal error: Using $this when not in object context in C:\meinverzeichnis\xampp\htdocs\aktivitaet_oop\class\SqlQuery.php on line 94

Diese Fehlermeldung bekomme ich für die folgende Code.

PHP-Code:

public function __construct() {
    $this->_db = db::getInstance();
}

public static function issetMail($_mail){
        $sql = "SELECT t_email FROM ak_teilnehmer WHERE t_email= '$_mail'";
        $result = $this->_db->query($sql) or die('Etwas stimmte mit dem Query $sql (issetMail) nicht: '.$this->_db->error);
        [...]
} 


(Die Zeile 94 bezieht sich auf $result = $this->_db...)
Hat das Problem irgendwas damit zu tun, dass meine Funktion issetMail() eine statische Funktion ist?
Oder hat es einen anderen Grund?

Danke schon mal

Leyyin

[DokuLeseHemmung]

Ja hat es!
In Statische Methoden gibts kein $this!
Das steht aber auch im Handbuch.

Tipp:
Statische Methoden und Eigenschaften sollte man nach Möglichkeit vermeiden.

[Leyyin]

Alles klar, danke.

Nur... Mich bringt es zum Überlegen;
Wann brauche ich dann diese Statische Eigenschaften bzw. Methoden am meisten,
wenn sie möglichst vermeiden werden sollen?

[cortex]

Zitat von Leyyin

Wann brauche ich dann diese Statische Eigenschaften bzw. Methoden am meisten

ich persönlich verwende bspw. statische methoden in meinem (selbstgeschriebenen) debugger. statische properties nutze ich für die vorhaltung und ausgabe zentraler fehlermeldungen und für den globalen zugriff auf die validierten (request-)variablen aus post, get, cookie und server.

ansonsten mal hier lesen: OOP mit PHP5 - Statische Methoden und Eigenschaften.

cx

[Leyyin]

Hmm... ok, Danke.