instancje obiektow - jak to rozumiec, łamigłowka Opcje

[mrok]

witam

czegos chyba nie do konca rozumiem i mam prosbę o wyjaśnienie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Mam kod jak ponizej:

[PHP] pobierz, plaintext 
<?php
class DataBase{
	private static $AdoDbLiteInstance = null;
 
	private function __construct ( )
	{
		echo 'konstruktor<br>';	 //1
		self::$AdoDbLiteInstance = NewADOConnection("mysqlt");
		self::$AdoDbLiteInstance->Connect($dbHost, $dbUser, $dbPass, $dbName);
 
		self::$AdoDbLiteInstance->Execute("SELECT * FROM tabla"); //2	
		echo self::$AdoDbLiteInstance.'<br>'; //3
	}
 
	public static function getInstance ( )
	{
		echo 'getinstance<br>'; //4
		if (is_null(self::$AdoDbLiteInstance))
			self::$AdoDbLiteInstance = new self();
		echo self::$AdoDbLiteInstance.'<br>'; //5
		self::$AdoDbLiteInstance->Execute("SELECT * FROM tabla"); //6	
		return self::$AdoDbLiteInstance;
	}
}
?>
[PHP] pobierz, plaintext 

wynikiem dzialania
$db = DataBase::getInstance(); jest

getinstance //4
konstruktor //1
Object id #4 //3
Object id #3 //5

Fatal error: Call to undefined method DataBase::Execute() in class/DataBase.class.php on line 32 //6

zmienna $AdoDbLiteInstance jest zmienna statyczna i stale odwoluje sie do niej przez self:: - nie wiem dlaczego wiec raz jest to object4 a raz object3. Ma ktos jakis pomysl?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

Ten post edytował mrok 4.03.2008, 22:57:58

[Cezar708]

w linii 3 (private static = null;) masz parsera, brakuje Ci nazwy zmiennej

poza tym nie wiadomo gdzie masz określone zmienne $dbHost, $dbUser, $dbPass, $dbName, jeśli to wszystko sobie poustawiałem to mi zadziałało (wydaje mi się że wg założenia)

Pozdrawiam

[mrok]

w linii 3 (private static = null;) masz parsera, brakuje Ci nazwy zmiennej

bystre oko (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) - juz poprawilem, ale nie tu lezy problem - niestety

poza tym nie wiadomo gdzie masz określone zmienne $dbHost, $dbUser, $dbPass, $dbName

wyrzucilem te linie aby nie wrzucac 60linii kodu (wtedy pewnie nikt w niego nie spojrzy)
Zauwaz ze w wynikach nie ma nic na temat niezdefiniowanych zmiennych

[Cezar708]

he he, sorry nie odpowiadałem ale jakość tak inne posty (szczególnie gości (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ) na tym forum mnie zaciekawiły (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)


otóż masz błąd:


linia 19: self::$AdoDbLiteInstance = new self();

czyli wywołujesz konstruktor... a w konstruktorze odwołujesz się do tej samej zmiennej statycznej:

linia 8: self::$AdoDbLiteInstance = NewADOConnection("mysqlt");

czyli program na pewno ma błąd logiczny. Nie wiem co chcesz osiągnąć ale na 99% pewności (opartej o nazwę tej zmiennej statycznej) trzeba zmienić linię 19 z:

[PHP] pobierz, plaintext 
<?php
self::$AdoDbLiteInstance = new self();
?>
[PHP] pobierz, plaintext 

na :

[PHP] pobierz, plaintext 
<?php
new self();
?>
[PHP] pobierz, plaintext 

[mrok]

Masz racje Cezar708. dziekuje bardzo.

Czyli tak lopatologicznie - ja probowalem przypisac obiekt do tej zmiennej

self::$AdoDbLiteInstance = new self();

zamist odpalic konstruktor

new self();

ktory wykona na niej odpowiednie operacje??
czy tak??

[Kocurro]

Chcesz tworzyc martwy obiekt zużywając na niego pamięć ?

pozdr.

[mrok]

Chcesz tworzyc martwy obiekt zużywając na niego pamięć ?

Dlaczego martwy - to typowy singleton tylko na poczatku troche go zepsulem??

dzieki temu potem w kodzie w jakimkolwiek obiekcie moge wywolac cos takiego

[PHP] pobierz, plaintext 
<?php
$db = DataBase::getInstance();
?>
[PHP] pobierz, plaintext 

i juz mam obiekt $db klasy AdoDb z jednym aktywnym polaczeniem do bazy. Nie musze duzej ilosci zmiennych przekazywac w parametrach, a i obiekt mam tylko jeden wiec ograniczam zuzycie pamieci.


pozdrawiam

[Kocurro]

No właśnie ... to NIE JEST singleton ...

pozdr.

[mrok]

No właśnie ... to NIE JEST singleton ...

nie bede sie kłócił, ponieważ brak mi argumentów (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) , ale czy mógłbyś rozwinąc tę myśl?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

czy jesli obiekt zwroci samego siebie to juz nie jest singletonem?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ? przeciez dzieki temu mam zawsze jeden obiekt tego typu?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)


pozdrawiam

[Kocurro]

Twój kod:

[PHP] pobierz, plaintext 
<?php
class DataBase{
	private static $AdoDbLiteInstance = null;
 
	private function __construct ( )
	{
		echo 'konstruktor<br>';	 //1
		self::$AdoDbLiteInstance = NewADOConnection("mysqlt");
		self::$AdoDbLiteInstance->Connect($dbHost, $dbUser, $dbPass, $dbName);
 
		self::$AdoDbLiteInstance->Execute("SELECT * FROM tabla"); //2	
		echo self::$AdoDbLiteInstance.'<br>'; //3
	}
 
	public static function getInstance ( )
	{
		echo 'getinstance<br>'; //4
		if (is_null(self::$AdoDbLiteInstance))
		 new self();
		echo self::$AdoDbLiteInstance.'<br>'; //5
		self::$AdoDbLiteInstance->Execute("SELECT * FROM tabla"); //6	
		return self::$AdoDbLiteInstance;
	}
}
?>
[PHP] pobierz, plaintext 

Natomiast poprawny kod to byłby np.

[PHP] pobierz, plaintext 
<?
 
class DataBase extends
{
 private static $_instance;
 private $_ado;
 
 private function __construct()
 {
		$this->_ado = NewADOConnection("mysqlt");
		$this->_ado->Connect($dbHost, $dbUser, $dbPass, $dbName);
 
		$this->_ado->Execute("SELECT * FROM tabla"); //2	
 }
 
public static function getInstance()
{
 
		if (is_null(self::$AdoDbLiteInstance))
		 self::$AdoDbLiteInstance = new self();
		return self::$AdoDbLiteInstance;
}
 
public static function getADO()
{
		if (is_null(self::$AdoDbLiteInstance))
		 self::$AdoDbLiteInstance = new self();
		return self::$AdoDbLiteInstance->_ado;
}
}
?>
[PHP] pobierz, plaintext 

Pisane na szybko z palca by uzmysłowić o co chodzi (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ogólnie chodzi o to by nie mieszać części statycznej od części dynamicznej. Część dynamiczna nie powinna dotykać części statycznej.

W Twoim przypadku o wiele lepiej byś zrobił jakbyś napisał zamiast swojego taki kod:

[PHP] pobierz, plaintext 
<?php
class DataBase{
	private static $AdoDbLiteInstance = null;
 
	public static function getInstance ( )
	{
		echo 'getinstance<br>'; //4
		if (is_null(self::$AdoDbLiteInstance))
		{
		  self::$AdoDbLiteInstance = NewADOConnection("mysqlt");
		  self::$AdoDbLiteInstance->Connect($dbHost, $dbUser, $dbPass, $dbName);
 
		  self::$AdoDbLiteInstance->Execute("SELECT * FROM tabla"); //2   
		}
		echo self::$AdoDbLiteInstance.'<br>'; //5
		self::$AdoDbLiteInstance->Execute("SELECT * FROM tabla"); //6	
		return self::$AdoDbLiteInstance;
	}
}
?>
[PHP] pobierz, plaintext 

Dzięki temu nie tworzysz obiektu do którego nie masz nigdzie uchwytu.

pozdr.

Ten post edytował Kocurro 7.03.2008, 12:17:50

[mrok]

wow - widze drobne niuanse, a jednak roznica duza.

Dzieki za wytlumaczenia. Twoj pomysł, jak to zrobić lepiej jest swietny. Dziekuję

Pozdrawiam