Forum PHP.pl

Witam,
postaram się jak najlepieej opisać problem.

Aplikacja opiera się o programowanie obiektowe.

Mam napisany sterownik bazy danych m.in. zawierający obliczanie ilości wykonanych zapytań do bazy.
Oprócz tej klasy istnieje klasa obsługująca użytkownika - zalogowanego, bądź nie.

Problem wydaje się banalny, ale nie dla mnie. Chciałbym aby sterownik obsługujący np. obliczanie ilości zapytań potrzebnych do wyświetlenia newsów na stronie (taki przykład) był również wykorzystywany w tej samej formie (ten sam identyfikator) do obliczania ilości zapytań podczas obsługi użytkownika, aby ilość zapytań była dodawana.

Tj. na początku skryptu znajduje się kod: $sql = new Mysql( [parametry] ); Aby użyć klasy bazy danych w klasie np. User musiałem tworzyć nowy identyfikator, albo zastąpić już istniejący. Aby temu zapobiec wykorzystywałem do tego konstrukcję global, ale już wiem, że to nie jest poprawne.

Jest na to jakieś rozwiązanie?
Pozdrawiam.

Najlepiej przekazać obiekt jako argument przy tworzeniu obiektu odpowiedzialnego za obsługę użytkownika.

[PHP] pobierz, plaintext 
<?php
$db = new DBClass('...');
$user_manager = new UserManager($db);
?>
[PHP] pobierz, plaintext 

Zachowujesz referencję do tego obiektu jako atrybut klasy i przy wywołaniu każdej metody masz zawsze dostępny z wewnątrz obiektu.

To nie lepiej Singleton (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

Singleton narzuca implementację sterownika. Chociaż nie ukrywam, że czasami jest wygodny.

Postanowiłem bliżej przyjrzeć się temu nieszczęsnemu zjawisku singletona.

Przeczytałem artyuł na stronie: http://webcity.pl/webcity/artykuly.php/t/62 i zastanawia mnie pewna rzecz.

Chciałbym wywoływać klasę w następujący sposób: $sql = new Mysql( [parametry] );,a później w innych klasach tworzyć singleton z referencją do tego egzemplarza.
Problem w tym, że kiedy chcem wywołać następujący singleton:

[PHP] pobierz, plaintext 
<?php
public function getInstance()
		 {
			 static $instance;
			 
			 if ( !isset( $instance ) )
			 {
				 $instance = new Mysql;
			 }
			 return $instance;
		 }
?>
[PHP] pobierz, plaintext 

.. w innej klasie to tworzony jest nowy egzemplarz klasy na miejscu tamtego i niestety nie rozwiązuje to problemu.

Zaciekawiło mnie ostatnie zdanie w podanym wyżej artykule: "Spróbujcie napisać funkcję, która będzie przyjmować nazwę klasy, a zwracać referencję do obiektu stworzonego za pomocą singletonu.". Czy to aby nie rozwiązało by tego problemu ? Jak to napisać?

Pozdrawiam

To nie tak działa. Po pierwsze instancję trzymamy w prywatnej statycznej zmiennej klasy, a nie funkcji. Po drugie, instancji nigdy się nie tworzy operatorem new, bo konstruktor ma być chroniony/prywatny. Zawsze należy użyć funkcji getInstance, która ma dostęp do konstruktora z wewnątrz obiektu. Konstruktor w przypadku klasy bazy danych jest niewygodny, bo obiekty są konstruowane na podstawie kilku argumentów.

Ok. Przedstawię co udało mi się wykombinować. Niby działa, ale...

Konstruktor sterownika bazy zajmuje się połączeniem i prywatny.

Singleton (martwi mnie to przekazywanie parametrów tutaj). [$oInstance jest private static]

[PHP] pobierz, plaintext 
<?php
public static function getInstance(  $sHost = false, $sUser = false, $sPass = false, $sName = false, $p = false )
{
	if ( self::$oInstance == null )
	{
		self::$oInstance = new Mysql( $sHost, $sUser, $sPass, $sName, $p );
	}
	return self::$oInstance;
}
?>
[PHP] pobierz, plaintext 

I teraz wygląda to tak, że w wywołaniu klasy (tu gdzie teraz byłoby wywołanie do konstruktora gdyby nie singleton, są przekazywane parametry - dlatego są opcjonalne) - czyli: $sql = Mysql::getInstance( [parametry] );
A w reszcie kodu: $sql = Mysql::getInstance();

Jednak wiem, że przekazywanie parametrów do singletona mija się z jego ideą. Powinienem więc zamiast konstruktora napisać funkcję do łączenia z bazą i wywoływać objekt klasy na kształt:

[PHP] pobierz, plaintext 
<?php
$sql = Mysql::getInstance();
	$sql = Mysql::connect();
?>
[PHP] pobierz, plaintext 

Albo na odwrót ? Wyczytałem to gdzieś i nie zabardzo wiem jak to działa w tym przypadku.

Prosiłbym o dogłębsze wytłumaczenie sprawy, może komuś się przyda, a ja chciałbym wiedzieć co i jak jest zalecane żeby pisać poprawnie.

Pozdrawiam.

Polecam wzorzec Property, z którego korzysta min. Zend_Framework w swoim rejestratorze obiektow (Zend::register())

Ten post edytował bigZbig 27.07.2006, 14:54:11

Heh... niezłe. Czyli mówisz, że Zend używa tego zamiast wzorcu singletona? To chyba rzeczywiście mniej zachodu...

Pofatygowalem sie i napisalem maly artykul na ten temat: Co zamiast global i GLOBALS - wzorzec Registry (Property).

nie widzę problemu w przekazywaniu zmiennych do getInstance(). Np. na początku skryptu robisz

[PHP] pobierz, plaintext 
<?php
$sql = Mysql::getInstance( $sHost , $sUser , $sPass , $sName , $p );
?>
[PHP] pobierz, plaintext 

a później już tylko getInstance bez parametrów.

A po co to komu - pozatym ze w dobrych edytorach nagle mozna sobie tagi PHPDoc o kant ..... potluc ? (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Singleton i Registry i nie maja ze soba nic wspolnego IMHM - bo nawet majac ten rejestr czy co tam moge sobie stworzyc instancje danej klasy.

Ten post edytował NuLL 29.07.2006, 20:46:20

Ale o co chodzi?


Jesli Rejestr jest obiektem a nie klasa z wlasciwosciami i metodami statycznymi to dobrze aby w aplikacji byl tylko jeden rejestr - i stad uzycie Singletona. A w rejestrze trzymasz co chcesz nie tylko obiekty klas typu singleton.