[Erledigt] PDO singleton

[tr0y]

Du hällst deine Konfiguration in deinem Singleton nicht zentral, du hardcodest sie, was mit verlaub das dümmste ist was man mit Konfigurationen die Environment-abhängig sind anstellen kann.

Statische Facades sind ein Beispiel wie man "global" auf ein und die selbe Instanz zugreifen kann ohne Singletons zu benutzen. Statische Facades sind entweder an Container-Einträge oder ganze Container geknüpft oder im Zweifelsfall nur an eine Instanz eines bestimmten Objekts ( ohne einen Container ).

[coolio]

Und wieso hälts du dann die Instanziierung auch nicht zentral?

[tr0y]

Tue ich doch, der Container existiert nur einmal in der ganzen Anwendung. Die Instanz also auch. Der Container wird im Bootstrap der Anwendung befüllt. Nach dem selben Muster funktioniert übrigens auch Silex ( Pimple ).

[coolio]

Ja, aber du rufst doch

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

jedes Mal neu auf. Das meinte ich.

[ApoY2k]

Das hat er doch gesagt, das wird genau einmal aufgerufen: Im Bootstrap. Warum sollte er das jedes Mal neu ausführen?

[coolio]

http://stackoverflow.com/questions/6...container-good
Troy, was meinst du?

[lottikarotti]

Hallöchen,

-Nur eine Instanz

Und für einen Vorteil, den du vermeintlich geschickt gelöst haben willst, kleisterst du dir deinen Code mit statischen Abhängigkeiten voll? Pack die Instanz in einen DIC (Dependency Injection Container) und injiziere selbige in alle Klassen, welche sie benötigen. So kannst du von außen jederzeit beeinflussen, welche Objekt-Instanz verwendet werden soll, ohne Änderungen an der Klasse selbst vornehmen zu müssen. Das nenne ich einen Vorteil.

Nicht so optimal:

class UserStorage{
    public function findUserById($id){
        Database::getInstance()->prepare(..);
        // ..
    }
} 

Way to go:

// Konfiguration
$dic['pdo'] = new PDO(..);

..

// Schnittstellen-Definition
interface IUserStorage{
    function findById($id);
}

..

// Implementierung
class PDOUserStorage implements IUserStorage{
    protected $pdo = null;

    public function __construct(PDO $pdo){
        $this->pdo = $pdo;
    }

    public function findById($id){
        $this->pdo->prepare(..);
        // ..
    }
} 

--

http://stackoverflow.com/questions/6...container-good
Troy, was meinst du?

Ich empfehle dir mal folgenden Artikel von Martin Fowler:

- Inversion of Control Containers and the Dependency Injection pattern

Viele Grüße,
lotti

[tr0y]

http://stackoverflow.com/questions/6...container-good
Troy, was meinst du?

Ich meine ich habe ein Beispiel gegeben für dessen Implementierung du eigenverantwortlich bist. Ich habe keinen Service Resolver implementiert, sondern nur umrissen wie Static Facades funktionieren.

Wenn du da mehr rein interpretieren willst, bitte schön. Es heißt übrigens tr0y, nicht troy. Dein o ist eigentlich eine 0.

[coolio]

Ok, vielen Dank. Habs schlussendlich so gelöst, dass ich den Code von troy in modifizierter Weise ausgelagert habe und nun in meiner Anwendung einbinde und z.B. mittels
[PHP]Database::Prepare()[PHP]
aufrufe.

[tr0y]

Aus eigenem Interesse zeig uns lieber den Code, damit wir dir sagen können ob das halbwegs Hand und Fuß hat was du daraus gemacht hast.

[coolio]

Datei container.php

<?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('mysql:host=localhost;dbname=test', 'root', '');

Database::connectContainer($container);

Und in gaestebuch.php z.B.

<?php

include('container.php');

//code, code, code

$val = Database::prepare('SELECT * FROM beitrage WHERE id = ?);
$val->bindParam(/**/);
//code, code, code

[monolith]

Und das [static facade] bringt mir? (ausser minimale Codereduktion)

Ich finde das viel schöner! Bei Symfony ging mir das immer auf die Nerven, mir stets die Instanzen von allem erst mal holen zu müssen. DRY ist das nicht!

[tr0y]

Okay, dann noch was damit du dich nicht vertippst beim array key:

<?php
class Database{

    const CONTAINER_FIELD = '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(self::CONTAINER_FIELD)){
            throw new RuntimeException('Connected container has not Database Object');
        }

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

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

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

}

$container = new ArrayObject();
$container[Database::CONTAINER_FIELD] = new PDO('mysql:host=localhost;dbname=test', 'root', '');

Database::connectContainer($container);

[coolio]

Danke dir, mein Freund. Bald dürfte deine Wunschliste auf Amazon kleiner werden

[tr0y]

Danke dir, mein Freund. Bald dürfte deine Wunschliste auf Amazon kleiner werden

Da freut sich tr0y.