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:)

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 

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

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.

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?

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.

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

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)

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).To zabrzmiało jak jakaś prawda absolutna. Kontekst rzeczywiście byłby w tej sytuacji lepszym rozwiązaniem - ale czy najlepszym? Niekoniecznie - zależy od sytuacji.

Tak się "czepiam" tylko

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.