witam

czegos chyba nie do konca rozumiem i mam prosbę o wyjaśnienie

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



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??

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

bystre oko - juz poprawilem, ale nie tu lezy problem - niestety


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

he he, sorry nie odpowiadałem ale jakość tak inne posty (szczególnie gości ) na tym forum mnie zaciekawiły


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 

Masz racje Cezar708. dziekuje bardzo.

Czyli tak lopatologicznie - ja probowalem przypisac obiekt do tej zmiennej

zamist odpalic konstruktor

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

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

pozdr.

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

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

pozdr.

nie bede sie kłócił, ponieważ brak mi argumentów , ale czy mógłbyś rozwinąc tę myśl?

czy jesli obiekt zwroci samego siebie to juz nie jest singletonem?? przeciez dzieki temu mam zawsze jeden obiekt tego typu?


pozdrawiam

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

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.

wow - widze drobne niuanse, a jednak roznica duza.

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

Pozdrawiam