[Erledigt] PDO singleton

[lottikarotti]

Hallöchen,

$queryNewUserInsert = Database::getInstance()->_db->prepare('INSERT INTO users (firstName, lastName) VALUES("Max", "Mustermann")');
$queryNewUserInsert->execute(); 

mal abgesehen davon, dass die Eigenschaft _db, im Ausgangspost, als private deklariert wurde und somit nicht von außen angesprochen werden kann, frage ich mich ernsthaft, welchen Mehrwert du dir von deiner Database-Klasse versprichst.

Viele Grüße,
lotti

[tr0y]

Du hantierst mit Silex, Doctrine und Twig rum und hast nahezu 0 OOP-Verständnis ?

Datebase::getInstance gibt ein Database-Objekt zurück, Prepare ist aber eine methode von PDO. PDO liegt im privaten property _db.

<?php

class Database
{

    private $pdo;

    private static $instance;

    private function __construct()
    {
        $this->pdo = new PDO(/** ... */);
    }

    public static function on()
    {
        if ( ! self::$instance instanceof self ) {
            self::$instance = new self;
        }

        return self::$instance->pdo;
    }

}

$foo = Database::on()->prepare(/** ... */);

tut was du willst.

<?php

class Database
{

    private static $container;

    public static function __callStatic($method, array $arguments)
    {
        if ( ! self::$container instanceof ArrayObject ) {
            throw new RuntimeException(
                'Static Facade is not connected to a container'
            );
        }

        if ( ! self::$container->offsetExists('database') ) {
            throw new RuntimeException(
                'Connected container has not Database Object'
            );
        }

        if ( ! self::$container->offsetGet('database') instanceof PDO ) {
            throw new RuntimeException(
                'Connected container serves a incompatible database object'
            );
        }

        return call_user_func_array(
            array(
                self::$container->offsetGet('database'),
                $method
            ),
            $arguments
        );
    }

    public static function connectContainer(ArrayObject $container)
    {
        self::$container = $container;
    }

}

$container = new ArrayObject();
$container['database'] = new PDO(/** ... */);

Database::connectContainer($container);

Database::Prepare(/** ... */);

Static Facade

[coolio]

Static Facade

Nice, also nur für Exception Handling?
Ausserdem sind in deiner Klasse oben noch mehr Fehler, als in der, die ich gepostet habe: z.B. ist db undefiniert.

[tr0y]

Jo, ich hab das im Foreneditor zusammen getippt, jetzt nicht mehr.

Das Exception Handling im Static Facade validiert nur bei jedem Aufruf den Container, Die Magische Methode führt direkt die Methode auf dem PDO Objekt im Container aus. Das ist der "Sinn" von solchen statischen Facades.

[coolio]

Also machst du nur am ArrayBezeichner aus, dass ein Objekt existieren darf? Nicht cool. Ausserdem muss man somit die Verbindungsdaten jedes Mal neu mitgeben. Und den Vorteil gegenüber Singleton habe ich immer noch nicht verstanden, ausser dass man

Database->getInstance()->db->prepare() 

durch

Database::prepare() 

ersetzen kann. Und das bringt mir? (ausser minimale Codereduktion)

[lstegelitz]

edit...

[lstegelitz]

edit..

verdammt... auf quote gekommen

[coolio]

@Isteglitz:Was machst du eigentlich?

[coolio]

@lottikarotti: Vorteile sind:
-Nur eine Instanz
-Verbindungsdaten zentral verwalten

[tkausl]

-Nur eine Instanz

Das ist aber nicht unbedingt ein Vorteil. Wenn ich jetzt auf 2 Datenbanken zugreifen will, was dann?

-Verbindungsdaten zentral verwalten

Jedes mir bekannte Framework nutzt Configs für sowas (also zentral) und gleichzeitig KEINE Singletons. Aber vieleicht ist bei denen auch einfach nur Magie im spiel

[coolio]

Wenn ich jetzt auf 2 Datenbanken zugreifen will, was dann?

Klasse um getter/setter erweitern?

Jedes mir bekannte Framework nutzt Configs für sowas

Nice, und dann wird in jeder Datei einzeln ein Datenbankaufbau gemacht. Die Art von Codewiederverwendung kenn ich noch gar nicht.

[tkausl]

Klasse um getter/setter erweitern?

Ähm... Zeig mal nen Beispiel, verstehe nicht wie das gemeint ist.

[coolio]

Ja ein doubleton machen, und dann attribwerte zuweisen.

[tkausl]

Ein Doubleton. Na klar, warum bin ich da nicht selbst drauf gekommen.

Nutze am liebsten allerdings Trippleton oder Hexaton, die sind mehrwertiger, wenn du verstehst was ich meine.

[coolio]

Was würdest du dann für den konnkreten Anwendungsfall vorschlagen. Würdest du auch eine static facade nutzen? Wenn ja, mache bitte deutlich, wie du die Probleme, die ich vorher bereits genannt habe, umgehen willst. Ausserdem verstehe ich deine unangebrachte Ironie nicht ganz.