Was ist der beste Ansatz für die Behandlung von db-Abfragen

Question

Ich bin mir sicher, was ich derzeit nicht optimal bin, aber ich würde gerne einige Meinungen darüber hören, wie Sie alle db-Abfragen in einer Webanwendung einschließlich behandeln die Verbindungszeichenfolge

Im Moment habe ich ein classes Verzeichnis, das eine Datei für jede Klasse hat, und ich habe ein anderes Verzeichnis namens db, die ein conn.php enthält, die die Verbindungszeichenfolge und die anderen Dateien auf das db Verzeichnis classes Verzeichnis hat, sind wie in dem , eine für jede Klasse, aber um die mysql-Abfragen zu behandeln.

Also im Grunde habe ich alle db Abfragen pro Klasse in einer Datei und wenn ich etwas aus der Klasse Datei abfragen müssen, rufe ich die Funktion auf der db-Datei

ich jede db-Datei in der entsprechenden db-Datei enthalten Zum Beispiel in der user.class.php Datei finden Sie include('db/user.db.php').

Außerdem schließe ich die Datei conn.php in jede db-Datei ein.

user.class.php:

include('db/user.db.php'); 
class User { 
    public $fname; 
    public $userid; 

    function __construct($userid) { 
     $this->user_id = $userid; 
     $this->fname = DB_GetFirstName($userid); 
     } 
} 

user.db.php:

include('conn.php'); 
function DB_GetFirstName($userid) { 
    $result = mysql_fetch_array(mysql_query("SELECT USR_FName FROM users WHERE USR_ID = '$userid'")); 
    return $result[0]; 
} 

conn.php:

$conn = mysql_connect("localhost", "user", "pass"); 
mysql_select_db("dbname", $conn); 

Wie geht es Ihnen?

Answer 1

Sie tun bereits das Richtige, indem Sie Ihre Domänenebene von der Datenbankzugriffsebene trennen. Aber die Art, wie Sie es zusammensetzen, könnte verbessert werden. Sehen Sie sich das Muster Table Data Gateway (TDG) an.

Im Table Data Gateway-Muster kapselt eine Klasse den gesamten Zugriff auf eine bestimmte Tabelle. Das ist eine Art wie Ihre User.Db.php mit dem Unterschied, dass das TDG eine echte Klasse ist. Statt einer Reihe von Funktionen gruppieren Sie alle zugehörigen Datenbankzugriffe in diese bestimmte Klasse. Dies hat den unmittelbaren Vorteil, dass Sie die Instanz an jede Klasse übergeben können, die sie benötigt, anstatt Funktionsaufrufe in sie zu codieren.

Wenn Sie mit einer bestimmten Tabelle arbeiten müssen, verwenden Sie das TDG, um Zeilen daraus abzurufen/zu ändern. Sie können dann entweder mit den zurückgegebenen Recordsets arbeiten. Oder verwenden Sie eine DataMapper, um die Daten aus dem Recordset auf Ihre Domain-Klassen, z. Ihre Benutzerklasse. Für einfache DataMapper ist die eigene Erstellung in Ordnung. Sobald es komplexer wird, verwenden Sie besser ein vorhandenes ORM. Eine Alternative zum TDG wäre die Row Data Gateway pattern.

Es ist eine gute Einführung in TDGs (mit einem Beispiel Zend Framework) bei

• http://css.dzone.com/books/practical-php-patterns-table

und für Row Data Gateway

• http://css.dzone.com/books/practical-php-patterns/practical-php-patterns-row

Answer 2

Verwenden Sie eine Database Abstraction Layer. Was Sie getan haben, klingt nach einem besseren Ansatz als rohe SQL-Zeilen, solange Sie Eingaben validieren und Injektionsangriffe verhindern.

Beliebte PHP-Frameworks wie Doctrine2 have built-in database abstraction layers, die unter die Lupe genommen wurden und viele Dinge abdecken, die Sie wahrscheinlich am Ende selbst tun werden.

Ich empfehle, eine Open-Source-Schicht wie oben erwähnt zu verwenden, bevor Sie Ihre eigene erstellen, da es bereits sehr solide Codebasen für Sie gibt. Erfinde das Rad nicht neu. Wenn Sie einen Mangel feststellen, denken Sie darüber nach, das Projekt zu verbessern.

Answer 3

Für kleine Projekte verwende ich sehr einfache PHP-Klasse http://code.google.com/p/edb-php-class/

$result = $db->q("select * from `users`limit 3"); 

foreach($result as $a){ 
     echo $a['name'].' '.$a['surname'].' '.$a['email'].' '.$a['country'].'</br>'; 
} 

für größere eine Datenbank-Abstraktionsschicht verwenden.