[PHP, MySQL] Codedesign

[Luckie]

Hallo,
ich habe mir eine private Adressdatenbank geschrieben, die auf meinem Webspace liegt. Jetzt habe ich gerade daran gebastelt und dabei ist mir aufgefallen, dass alle Abfragen verstreut in den PHP Dateien herumliegen. Jetzt hab eich mir gedacht, ob es nicht sinnvoll wäre alle Abfragen in einer PHP Datei zu sammeln. Dann wäre das etwas einfacher, wenn man was ändert und man hat dann auch keinen doppelten Code.

Was meint ihr dazu? Oder wie macht man so was normalerweise?

[mkinzler]

Am Besten als parametrisierte Abfragen

[Luckie]

Jetzt haben wir eine Überschneidung mit meinem andren Thema. Ich suche gerade mit Google. Hast du da gerade einen Link parat, wo ich das mal nachlesen kann? Oder kannst du das hier kurz erklären, wie das funktioniert?

[mkinzler]

Siehe anderen Beitrag.

[Luckie]

:mrgreen: Siehe anderen Beitrag. ;)

[s.h.a.r.k]

Wenn du OO programmierst, dann kannst du ja alles in entsprechende Klassen kapseln und brauchst die Abfragen nicht irgendwo sammeln -- oder handhabst du das in Delphi auch so? Zudem würde ich mir zugleich dann noch das MVC-Pattern anschauen, da ich nur noch damit programmieren würde. Schau dir vielleicht mal aktuelle Frameworks an, die das so umsetzen. Ich nutze zurzeit Yii, denn davon kann man sich recht viel abgucken, wie man es in PHP in etwa machen sollte und was alles möglich ist.

Und hier noch der Link zum anderen Thema ;)

[Luckie]

Na ja, der Code ist entstanden, da habe ich von PHP und Klassen noch keine Ahnung gehabt. Deswegen dachte ich eine kleiner Verbesserung wäre die Abfragen erst mal zentral zusammeln.

[s.h.a.r.k]

Hm, wenn du nicht unbedingt auf OOP umsteigen willst, dann würde ich höchstens "Getter"-Methoden zentral sammeln, die dann als Parameter eben die Params für die Query verlangen. Das kann den anderen Code schon übersichtlicher machen.

Aber ich würde mir da meist die Arbeit machen und auf ein FW umsteigen. Hat den Grund, dass man dabei viel lernen kann und zum anderen auch schon sehr viel bzgl. ORM etc. gelöst hat.

[Luckie]

Getter-Methoden? Was meinst du damit? Die gibt es doch ohne OOP gar nicht. Und bevor ich irgendwelche Frameworks einsetze, wollte ich eigentlich lernen was da im Hintergrund passiert.

[s.h.a.r.k]

Das alles ist ohne Gewähr, dass es funktioniert ;) Ist einfach mal schnell runtergeschrieben! Hier mal wie ich mir die "Getter"-Methoden vorstellen könnte -- das Getter war extra in Anführungszeichen geschrieben, da ich damit eher das "Daten-Holen" meinte.

Code:

<?php
// 
define('TABLE_USER', 'my_users');

//
$pdoConnection = null;

//
function getPdoConnection()
{
    global $pdoConnection;
    $dsn = 'mysql:dbname=testdb;host=127.0.0.1';
    $user = 'dbuser';
    $password = 'dbpass';
    try {
        $pdoConnection = new PDO($dsn, $user, $password);
    } catch (PDOException $e) {
        die('Connection failed: ' . $e->getMessage());
    }
}

//
function getAllUsers()
{
    $sql = 'SELECT * FROM :table';
    $connection = getPdoConnection();
    $query = $connection->prepare($sql);
    $query->bindParam(':table', TABLE_USER, PDO::PARAM_STR);
    $query->execute();
    if ($query === false) {
        return false;
    } else {
        return $query;
    }
}

function gerUser($username = '')
{
    $sql = 'SELECT * FROM :table WHERE username = :username';
    $connection = getPdoConnection();
    $query = $connection->prepare($sql);
    $query->bindParam(':table', TABLE_USER, PDO::PARAM_STR);
    $query->bindParam(':username', $username, PDO::PARAM_STR);
    $query->execute();
    if ($query === false) {
        return false;
    } else {
        return $query;
    }
}

So, nun schaut alles aufgeräumter aus:

Code:

$users = getAllUsers();
if ($users === false) {
    echo 'There are no users registered.';
} else {
    while ($user = $users->fetch(PDO::FETCH_OBJ)) {
        echo $user->username;
    }
}