Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [ OOP ] Singleton z wikipedii nie działa
Forum PHP.pl > Forum > PHP > Object-oriented programming
Babcia@Stefa
Witam, użyłem klasy Singleton z wikipedii, ale niestety nie działa.

  1. <?php
  2.  
  3. class Singleton
  4. {
  5.   private static $instance;
  6.   private function __construct() {} // Blokujemy domyślny konstruktor publiczny
  7.   private function __clone(){} //Uniemozliwia utworzenie kopii obiektu
  8.  
  9.   function getInstance()
  10.   {
  11.     return ($i = self::$instance) ? $i : $i = new self;
  12.   }
  13. }
  14. ?>


Czy ona nie powinna wyglądać tak?

  1. <?php
  2.  
  3. class Singleton
  4. {
  5.   private static $instance;
  6.   private function __construct() {} // Blokujemy domyślny konstruktor publiczny
  7.   private function __clone(){} //Uniemozliwia utworzenie kopii obiektu
  8.  
  9.    function getInstance()
  10.    {
  11.    return ($i = self::$instance) ? $i : $i = self::$instance = new self;
  12.    }
  13. }
  14. ?>


Przy użyciu:
  1. <?php
  2. class myClass extends Singleton
  3. {
  4. (...)
  5. }
  6.  
  7. $Object = myClass::getInstance();
  8. ?>


Albo źle zrozumiałem pojęcie "Singletonów" albo jest błąd na wikipedii tongue.gif
Jeśli się mylę, proszę o wskazanie mi błędu smile.gif

Pozdrawiam, WebNuLL
dr_bonzo
Jeszcze brakuje $i+++++++b z C.

zapisz tego ifa w sposob czytelniejszy ibedziesz mial dobra wersje:

jesli self::$instance jest null { to utworz nowy obiekt }

a potem zwroc go
Babcia@Stefa
Ten if to oryginał skopiowany z wikipedii tongue.gif (nie mój)
A tak po za tym to słyszałem że skrócone ify są szybsze od tych normalnych, ale nie jestem pewien.

Ale ten nowy obiekt wtedy wrzucam do self::$instance aby go później zwrócić raczej tongue.gif

@edit
Cytat(orglee)
Kto dodał taki beznadziejny kod na wiki. :|


Na pewno nie ja winksmiley.jpg

Pozdrawiam, WebNuLL
dr_bonzo
Wiem ze kod z wikipedii - zajrzalem tam przed odpowiedzeniem.

skrocone ify szybsze, yhym, przez 10 min jeszcze nie zalapalem jak ten kod mialby sie wykonac, co oznacza ze ten zapis jest mega nieczytelny.

(oczywiscie $x = ($newX ? $newX : "default" ) jest dla mnie jak najbardziej dopuszczalny, w innych przypadkach nie uzywam skroconego ifa)

edit: tak, w ifie wrzucasz obiekt do self::$instance
starach
  1. <?php
  2. class Singleton_Extended extends Singleton
  3. {
  4.    public function setVar($value)
  5.    {
  6.        parent::setVar($value + 1);
  7.    }
  8. }
  9.  
  10. class Singleton
  11. {
  12.    private $var;
  13.    private function __construct() {}
  14.    private function __clone() {}
  15.    function getInstance()
  16.    {
  17.        static $instance;
  18.        return empty($instance) ? $instance = new self : $instance;
  19.    }
  20.    public function setVar($value)
  21.    {
  22.        $this->var = $value;
  23.    }
  24.    public function getVar()
  25.    {
  26.        return $this->var;
  27.    }
  28. }
  29.  
  30. Singleton::getInstance()->setVar(1);
  31. echo Singleton::getInstance()->getVar();
  32. // Zwróci: "1"
  33.  
  34. Singleton_Extended::getInstance()->setVar(1);
  35. echo Singleton_Extended::getInstance()->getVar();
  36. // Zwróci: "1" bo tak czy srak trzeba nadpisac implementacje singletona
  37. ?>


Kto dodał taki beznadziejny kod na wiki. :|
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.