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

[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