Dziedziczenie magicznych metod, __get($zmienna) i __set($zmienna, $wartosc) Opcje

[Asmox]

Chciałem zrobić uniwersalną implementację dla metody magicznej __get($var). W tym celu, w klasie abstractObject napisałem coś takiego:

[PHP] pobierz, plaintext 
 
public function __get($zmienna) {
if (!property_exists($this, $zmienna)) {
	$exc = new ExceptionConceptor('Proba odwolania do nieistniejacego pola');
	$exc->addConcept('pole', $zmienna);
	throw $exc;
}
if (!isset($this->$zmienna)) {
	if (method_exists($this, 'pobierzPole'.ucfirst($zmienna)))
		call_user_func(array($this, 'pobierzPole'.ucfirst($zmienna)));
}
//return $this->$zmienna;
}
[PHP] pobierz, plaintext 

Natomiast w klasie dziedziczącej coś takiego:

[PHP] pobierz, plaintext 
public function __get($zmienna) {
	parent::__get($zmienna);
	return $this->$zmienna;
}
[PHP] pobierz, plaintext 

Powiem szczerze, że jestem zszokowany koniecznością napisania czegoś takiego. Myślałem, że dziedzicząc __geta, klasa dziedzicząca pobierze normalnie swoje pole. Natomiast gdy mój kod wyglądał w ten sposób, że w klasie dziedziczącej nie było nic dotyczącego tej metody, a w abstractObject nie było komentarza przy returnie, wywalało mi komunikat:

Notice: Undefined property: tag::$(tu nazwa zmiennej do której się odwoływałem) in...

Natomiast jeśli chodzi o dziedziczenie magicznej metody __set(), jest jeszcze gorzej. Kod wygląda tak: (klasa abstractObject)

[PHP] pobierz, plaintext 
public function __set($zmienna, $wartosc) {
if ($this->blokada) {
	if (isset ($this->$zmienna)) {
		$exc = new abstractObjectException('Proba dostepu do zablokowanego pola');
		throw $exc;
	}
	else
		$this->$zmienna = $wartosc;
}
$this->$zmienna = $wartosc;
echo $this->$zmienna;
	}
[PHP] pobierz, plaintext 

Jedno pytanie: dlaczego tak się dzieje? :-(

Ten post edytował Asmox 11.11.2011, 15:18:28

[mortus]

Musisz pamiętać, że metody __set() i __get() służą do ustawiania i pobierania wartości zmiennych prywatnych i chronionych. Jeśli w klasie nie masz np. prywatnej zmiennej $a, a użyjesz kodu

[PHP] pobierz, plaintext 
$object->a = 'jakaś wartość tekstowa';
[PHP] pobierz, plaintext 

to tak naprawdę nie zadziałają Twoje metody __set() i __get(), a zadziałają te metody domyślne. W obiekcie danej klasy od tej chwili będziesz miał zmienną $a, ale będzie to zmienna publiczna, którą możemy ustawić/pobrać nie przeciążając metod __set()/__get(). Dlatego używanie przeciążonych metod __set() i __get() w ten sposób, w jaki Ty to robisz mija się z celem, chyba, że każdą zmienną jaką później będziesz dodawał masz zadeklarowaną jako prywatną lub chronioną. Lepszy podejściem byłoby coś takiego:

[PHP] pobierz, plaintext 
class A {
 private $properties = array();
 public function __set($key, $value) {
  if(!isset($this->properties[$key])) {
   $this->properties[$key] = $value;
  }
 }
 public function __get($key) {
  if(isset($this->properties[$key])) {
   return $this->properties[$key];
  }
 }
}
[PHP] pobierz, plaintext 

Jeżeli zatem koniecznością u Ciebie jest przeciążanie metod __set() i __get(), to nie masz innego wyjścia, jak przeciążać te metody dla każdej klasy, czy to potomnej, czy rodzicielskiej.
Natomiast zastanów się, czy rzeczywiście jest Ci to potrzebne. Zamiast się zastanawiać możesz również sprawdzić, jaki będzie zasięg (widoczność) tych Twoich zmiennych, robiąc var_dump jakiegoś przykładowego obiektu.

Kiedyś już coś na ten temat pisałem. Poczytaj.