[SQL / PDO...] Powtarzający się kod Opcje

[terabit]

Witam (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

mam np taki oto kod:

[PHP] pobierz, plaintext 
<?php
public function getOnePerson($id) {
        
        try {
            
            $pdo = new PDO('mysql:host=localhost;dbname=contactmanager', 'root', 'root');
            $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $row = $pdo -> query("SELECT * FROM persons WHERE id = $id");
 
            return $row;
            
        } catch (PDOException $e) {
            echo 'Error:' . $e->getMessage();    
        }
        
        $row -> closeCursor();
        
    }
?>
[PHP] pobierz, plaintext 

i nasuwa się pytanie, co zrobić żeby tego tyle razy nie powtarzać ?
mam jakąś tam klase do zarządzania Osobami i praktycznie w każdej metodzie musze powtarzać ten sam fragment kodu...

jak wy to rozwiązujecie?
czy poprostu zrobić dodatkową metodę do której będę przekazywać coś takiego:

[PHP] pobierz, plaintext 
<?php
$row = $pdo -> query("SELECT * FROM persons WHERE id = $id");
?>
[PHP] pobierz, plaintext 

choć to pewnie nia zawsze się sprawdzi...

pisząc obiektowo raczej nie powinno się powtarzać kilka razy (DRY;))

jakieś rozwiązanie ?

z góry dzięki za pomoc (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[Methestel]

[PHP] pobierz, plaintext 
<?php
class PdoFactory {
    private static $instance = NULL;
    private function __construct() {}
    public static function get() {
        if (self::$instance === NULL) {
            $dsn = 'mysql:dbname=***;host=***;port=***;'; //
            $user = 'nobody';                              //zmien na swoje
            $password = 'nobody';                         //
            try {
                $pdo = new PDO($dsn, $user, $password);
                $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            } catch (PDOException $e) {
                throw new Exception ($e->getMessage());
            }
            self::$instance = $pdo;
        }
        return self::$instance;
    }
}
?>
[PHP] pobierz, plaintext 

Nazwa klasy wskazuje że jest to fabryka bo bardzo łatwo przerobić tą klasę tak żeby fabryką była (fabryka czyli klasa która serwuje różne obiekty w zależności od parametrów) i jeśli w projekcie korzystasz z więcej niż jednej bazy danych należałoby przekazać do metody PdoFactory::get() jeszcze jeden parametr dzięki któremu będziesz mógł uzyskać połączenie z określoną bazą danych

Zaprezentowany wyżej kod jest implementacją wzorca projektowego o nazwie Singleton i sprawdza się bardzo do kodu który ma być dostępny w dowolnym miejscu twojego skryptu. Jest to tak naprawdę opakowana w klasę zmienna globalna dlatego radze nie nadużywać tego wzorca.

Mając tak zdefiniowaną klasę obiekt klasy PDO pobierasz przez:

[PHP] pobierz, plaintext 
<?php
$pdo = PdoFactory::get();
?>
[PHP] pobierz, plaintext 

Metoda z twojego pierwszego postu może wyglądać teraz np tak:

[PHP] pobierz, plaintext 
<?php
public function getOnePerson($id) {
        
        $pdo = PdoFactory::get();
           //Jeżeli do zapytania przekazujesz jakikolwiek parametr radze zamiast z PDO::query() skorzystać z PDO::prepare()
            $row = $pdo -> query("SELECT * FROM persons WHERE id = $id"); 
 
            return $row;
        
        $row -> closeCursor(); //to jest bez sensu bo nigdy nie będzie wykonane przez to że umieściłeś to za "return"
        
    }
?>
[PHP] pobierz, plaintext 

albo obiekt klasy PDO pobrać już w samym konstruktorze i przypisać go do prywatnej składowej tej klasy.

Pozwoliłem dodać sobie dwa komentarze do kodu na jego temat.

Ten post edytował Methestel 8.04.2009, 17:43:29