Połączenie z BD ( Singleton ) - Problem Opcje

[ziomek32]

Witam wszystkich bardzo serdecznie:)

Mam o to taki kod:

[PHP] pobierz, plaintext 
final class DB
{
 
    private static $pdo = false;
 
 
    public static function pdo()
    {
        if( self::$pdo == false )
        {
            self::$pdo = new PDO('mysql:host=localhost;dbname=baza', 'user', 'haslo');
        }
        return self::$pdo;
    }
 
    private function __construct() {
 
    }
}
 
$db = DB::pdo();
 
$a = $db->query("select pole from tabela");
 
while($b = $a->fetch(PDO::FETCH_OBJ)){
 
echo $b->pole.'<br />';
 
}
 
class test {
 
public function t(){
 
$a = $db->query("select pole from tabela");
 
while($b = $a->fetch(PDO::FETCH_OBJ)){
 
echo $b->pole;
 
}
 
}
 
}
 
$c = new test();
$c->t();
[PHP] pobierz, plaintext 

Chciałbym zrobić tak aby wywołanie połączenie z baza danych działało przez $db w klasie test, jak narazie działa mi to tylko poza klasą czy mógłby mi ktoś pomóc rozwiązać mój problem?

Pozdrawiam z góry dzięki za odp:)

[wookieb]

Podstawy...

[PHP] pobierz, plaintext 
$db = DB::pdo();
$a = $db->query("select pole from tabela");
 
while($b = $a->fetch(PDO::FETCH_OBJ)){
 
echo $b->pole;
 
 
[PHP] pobierz, plaintext 

[ziomek32]

Tzn? gdzie zrobiłem błąd? Przepraszam że zadaje takie pytania ale dopiero zaczynam z OOP, byłbym wdzięczny gdyby mi ktoś dokładniej wytłumaczył co jest nie tak (IMG:style_emoticons/default/smile.gif)

[wookieb]

Brakowało CI

[PHP] pobierz, plaintext 
$db = DB::pdo();
[PHP] pobierz, plaintext 

Na początku twojej metody test()
A problem wynika z zasięgu zmiennych. Metoda nie widzi zmiennych zdefiniowanych poza nią bądź nie będących właściwościami klasy w jakiej się znajduje.

[ziomek32]

Aha, to ja do każdej metody muszę teraz dodawać $db = DB::pdo(); aby mi nawiązało połączenie z DB, nie da się tego jakoś "obejść" bo to chyba nie jest najbardziej optymalne rozwiązanie?

[wookieb]

Singleton w twoim wypadku nie nawiązuje za każdym razem połączenie tylko zwraca ci obiekt, które to połączenie nawiązało. Tak więc bez względu jak wiele wywołań DB::pdo dasz to i tak będzie wykorzystane i nawiązane jedno połączenie.

[ziomek32]

Tak, wiem dlatego zdecydowałem się zastosować singleton, czyli to rozwiązanie jest dobre?

[wookieb]

Nie singleton nie jest dobrym rozwiązaniem aczkolwiek bardzo wygodnym. Naj naj lepszym rozwiązaniem jest zastosowanie wzorca Context natomiast wygodniejszym i najczęściej stosowanym jest Registry albo Factory. Musisz o nich poczytać (materiałów jest sporo aczkolwiek nie mam pewności co do Contextu)

[Crozin]

aczkolwiek bardzo wygodnym

Jedynie pozornie, ponieważ korzystanie tego w taki sposób jak pokazany tutaj w postach sprawia, że można się pochlastać później (debuggowanie, modelowanie relacji pomiędzy obiektami).

Naj naj lepszym rozwiązaniem jest zastosowanie wzorca Context

To zabrzmiało jak jakaś prawda absolutna. (IMG:style_emoticons/default/winksmiley.jpg) Kontekst rzeczywiście byłby w tej sytuacji lepszym rozwiązaniem - ale czy najlepszym? Niekoniecznie - zależy od sytuacji.

Tak się "czepiam" tylko (IMG:style_emoticons/default/winksmiley.jpg)

Co do Singletona samego w sobie - chciałem zaznaczyć, że ideą tego wzorca jest zapewnienie tylko jednej (dwie lub więcej mogłyby zaszkodzić aplikacji - połączenie z bazą danych nie jest taką klasą) instancji danej klasy, a nie globalny dostęp do takowej. Co więcej tutaj Singleton jest źle zaimplemetowany - nadal można klonować obiekt.

Ten post edytował Crozin 30.06.2010, 14:38:00