[PHP, MySQL] Codedesign

[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;
    }
}

[blackfin]

Ich würde das auch so ähnlich wie shark beschrieben hat machen.
Alle Abfragen irgendwo sammeln ist nicht der richtige Weg.
Wenn du aber alles, was man irgendwie aus den Seiten extrahieren kann, in diese Getter / Setter-Funktionen auslagerst und sie in logische Gruppen / php-Dateien einordnest, die du evtl. auch noch dynamisch bei Gebrauch einbinden kannst, tust du dich im zweiten Schritt (irgendwann mal) auch leichter, das ganze in Klassen zu kapseln, wie es die grossen Frameworks wie Zend oder Symfony machen.

Generell kann man sich so ein MVC-ähnliches System auch ohne OOP zusammenbauen und im zweiten Schritt dann in OOP transferieren.

[Luckie]

Na ja, ich lasse das jetzt erst mal ruhen. Ich habe das was ich will und es ist ja nur für mich. Den Code wird niemand anders zu sehen bekommen.

[Valle]

Hallo Luckie,

willst du deinen Code schöner machen, vor allem auch hinsichtlich SQL, dann ist Zentralisierung mit Sicherheit der falsche Weg. Allerdings gilt auch hier das gleiche, wie ich schon in diesem Thread schrieb.

Natürlich ist das anfangs (!) etwas mehr Aufwand. Aber es lohnt sich und macht Spaß. Also mein Vorschlag: Lass Code Code sein und versuche es im nächsten Projekt mal mit einem der genannten Frameworks. In der DP sind genügend Leute die dir damit helfen können.

Liebe Grüße,
Valentin