__destruct, zasada dzialania... Opcje

[goffyy]

Mam troche moze glupie pytanie ale nie dziala mi jedna rzecz i zastanawiam sie dlaczego.

Mam klase w ktorej jest zdefiniowana funkcja __destruct(). Niestety podczas konczenia wykonywania sie skryptow funkcja ta nie zostaje wywolana pomimo ze klasa o ktorej mowa zostala wykorzystana. Mam wiec pytanie czy dziala to na wszystkich wersjach php5 (o ile mi wiadomo to tak) i czy przy konczeniu programu wywoluje sie ta procedura automatyczna czy trzeba zrobic przykladowo unset($jakasklasa) - co tez u mnie nie dziala (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Z gory dzieki za podpowiedz.

[nasty]

tak dziala na wszystkich php5, podaj moze kod ?

[goffyy]

Kod jest banalny:

[PHP] pobierz, plaintext 
<?php
function __destruct() {
		if ($this->dirty) $this->save();
}
?>
[PHP] pobierz, plaintext 

Kod jest o tyle prosty ze ma wywolac funkcje save() ktora ma byc uzywana do zapisywania ustawien aplikacji - jest to prosty wzorzec Registry. Z gory mowie ze funkcja save() wywolywana oddzielnie dziala poprawnie. Niestety __destruct nie uruchamia sie automatycznie. Pozostale funkcje typu __construct(), __autoload() itp dzialaja poprawnie.

[nasty]

a napisz echo $this->dirty...
boc cos mi tu smierdzi falsem w $this->dirty (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[goffyy]

Probowalem ale nawet jak dasz jakies echo na poczatku to i tak sie nie wyswietla. Poprostu dziala tak jakby pomijal _destruct() i zastanawia mnie dlaczego (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[nasty]

nic nie wyswietla, bo nic niema w tej zmiennej, w destrukt, zrob $this->dirty = true a zobaczysz ze bedzie dzialac, ale to tylko zeby ci pokazac ze dziala destruktor, podaj mi caly kod to ci powiem co jest nie tak.

Ten post edytował nasty_psycho 24.08.2006, 16:50:14

[goffyy]

Caly kod klasy:

[PHP] pobierz, plaintext 
<?php
class kernel_ApplicationRegistry extends kernel_Registry {
	private static $instance;
	private $freezefile = "temp/ApplicationRegistry.txt";
	private $values = array();
	private $dirty = false;
 
	private function __construct() {
		$this->doReload($this);
	}
 
	static function instance() {
		if (!self::$instance) self::$instance = new self(); 
		return self::$instance;
	}
 
	static function reload() {
		self::instance()->doReload();
	}
 
	private function doReload() {
		if (!file_exists($this->freezefile)) return false;			
		$serialized = file_get_contents($this->freezefile, true);
		$array = unserialize($serialized);
		if (is_array($array)) {
			$array = array_merge($array, $this->values);
			$this->values = $array;
			return true;
		}
		return false;
	}
 
	private function save() {
		echo "save";
		$frozen = serialize($this->values);
		try {
			file_put_contents($this->freezefile, $frozen, FILE_USE_INCLUDE_PATH);
			} catch (KernelException $e) {
			$e->__toString();
			}
		$this->dirty = false;
	}
 
	function get($key) {
		return $this->values[$key];
	}
 
	function set($key, $val) {
		echo "ustawiam";
		$this->dirty = true;
		$this->values[$key] = $val;
	}
 
	function isEmpty() {
		return empty(self::instance()->values);
	}
 
	function __destruct() {
		echo "wychodze";
		if ($this->dirty) $this->save();
	}	
}
?>
[PHP] pobierz, plaintext 

Jak cos znajdziesz to daj znac (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[nasty]

no pewnie ze nic nie zrobi bo nigdze nie przydzielasz jej innej wartosci niz false na poczadku, daj np, w __construct $this->dirty = true, i wykonaj, ale wydac ze zapomniales ustalic w jakiejkowiek funkcji zeby zmieniala $this->dirty na true

[goffyy]

Jest ustalana na true tutaj:

[PHP] pobierz, plaintext 
<?php
function set($key, $val) {
		echo "ustawiam";
		$this->dirty = true;
		$this->values[$key] = $val;
	}
?>
[PHP] pobierz, plaintext 

Czyli w momencie jak ustawiasz jakas nowa zmienna lub jak zmianiasz istniejaca.

[nasty]

jesli ta funkcje wywolujesz, to powinno dzialac, jak nie to ja juz nieiwm (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[goffyy]

ja tez nie wiem wlasnie (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) musze do admina napisac bo wyglada ze cos nie tak maja z php (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[nasty]

admin tu ci nic nie pomoze, nie zawracaj mu glowy (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Ten post edytował nasty_psycho 24.08.2006, 17:14:50

[hwao]

a ja wiem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Dekstruktor jest generalnie zawsze wykonywany, ale jest z nim sporo problemow...

http://www.zyxist.com/pokaz.php/glupi_blad_php
O tu. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Zeby sprawdzic czy sie wlacza zrob po prostu

[PHP] pobierz, plaintext 
<?php
function __destruct() {
	  // echo nie bedzie widac, poniewaz dekstuktor
	  // zazwyczaj jest odpalany po skonczeniu przesylania danych do klienta http (czyli nie widzi juz tego)
		echo "wychodze";
	  // Zdebugujemy to tak
 
 
[b]file_put_contents( './test.txt', 'Desktruktor, uruchomiony'.date('d.m.Y h:i:s' ) );
[/b]
		if ($this->dirty) $this->save();
	}
?>
[PHP] pobierz, plaintext 

[goffyy]

Masz racje (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Tylko ze u mnie nie dizala na razie w zaden sposob (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[mariuszn3]

goffyy destruktor zawsze jest odpalany i nie jest to żadne widzimisie..
Przetestowałem Twój kod i zawsze wyświetla mi 'wychodzę' więc wszystko jest ok..
Sprawdź czy jakieś inne czynniki nie wpływają na to, że tego nie widzisz.
Jest tylko jedyna mi znana możliwość aby destruktor nie był odpalony.
Tak będzie jeśli przy wyjściu aplikacji wystąpił błąd 'fatal error' podczas wywołania destruktora jednego z obiektów, wtedy żadne inne destruktory jeszcze istniejących obiektów już nie zostaną odpalone.

[goffyy]

teoretycznie to i ja wiem ze tak powinno byc gorzej ze na tym serwerze co testuje nie jest (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[splatch]

Może to głupie pytanie.. ale masz jakąś instancje tej klasy [stworzony obiekt]?

[nasty]

a wez w construct napisz :

[PHP] pobierz, plaintext 
<?php
$this->__destruct();
?>
[PHP] pobierz, plaintext 

i zobacz czy wykona ..., bo jak nie to ....

[goffyy]

W ten sposob dziala wszystko poprawnie a wiec to wina php jakas (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[Cysiaczek]

Możesz pokazac klasę bazową?

[goffyy]

To jest prosta klasa abstrakcyjna:

[PHP] pobierz, plaintext 
<?php
abstract class kernel_Registry {
	private function _construct() {}
	abstract function get($key);
	abstract function set($key, $val);
}
?>
[PHP] pobierz, plaintext 

[Cysiaczek]

Hmm... jakbym nie kombinował i tak zawsze jest ok. Coś musiałes gdzieś namieszać w kodzie użytkującym klasę. Wygląda na to, że coś przestawia Ci zmienną $dirty na false. Upewnij się, że tak się nie dzieje.

Tak na marginesie - możesz smiało wywalić konstruktor z klasy bazowej - nie jest potrzebny (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam.

[goffyy]

Niestety to jednak php bo ja tylko ta klase do pliku wrzuclem i nie dziala (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) A co do konstruktora to masz racje.

[drizzt73]

ja mialem takie jaja z destruktorami kiedy wewnatrz destruktora jednej klasy odwolywalem sie do innej - ktora chyba niedoskonaly jeszcze w tym wzgledzie php pozwolil sobie juz zniszczyc. Pod www nie bylo tego w ogole widac, w shellu dopiero jak zaczalem sprawdzac to pojawil sie nawet segmentation fault. Dlatego popatrz jeszcze w miejsce gdzie ew. tak klasa moze sie niszczyc o ile nie jest to "naturalny" koniec.