Prośba o sprawdzenie klasy Opcje

[Jarod]

W książce natrafiłem na przykład, który wydaje mi się błędny. Wiem, że książki helionu mają literówki i różnego rodzaju błędy (na stronie helionu w erracie nie ma poprawki do tego), dlatego proszę o sprawdzenie poniższego kodu.

[PHP] pobierz, plaintext 
<?php
interface Validator{
	abstract function validate();
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
require_once('interface.Validator.php');
 
abstract class PropertyObject implements Validator {
 
	protected $propertyTable = array();	 //Przechowuje pary nazwa-wartość przypisujące
											//własności do pól bazy danych
	protected $changedProperties = array(); //Lista własności które zostały zmodyfikowane
	protected $data;						//Dane z bazy
	protected $errors = array();
 
	public function __construct($arData) {
		$this->data = $arData;
	}
 
	function __get($propertyName) {
		if (!array_key_exists($propertyName, $this->propertyTable))
			throw new Exception('Błędna własność '.$propertyName.'!');
 
		if (method_exists($this, 'get' , $propertyName)) {
			return call_user_func(array($this, 'get' , $propertyName));
		}
		else { 
		return $this->data[$this->propertyTable[$propertyName]];
	  }
	}
 
 
	function __set($propertyName, $value) {
		if (!array_key_exists($propertyName, $this->propertyTable))
			throw new Exception('Błędna własność '. $propertyName.' !');
 
		if (method_exists($this, 'set' , $propertyName)) {
			return call_user_func(array($this, 'set' , $propertyName), $value);
		}
		else { 
			if ($this->propertyTable[$propertyName] != $value &&
			   !in_array($propertyName, $this->changedProperties)) {
			   $this->changedProperties[] = $propertyName;
			}
 
		$this_>data[$this->propertyTable[$propertyName]] = $value;
		}
	}
 
	function validate() {
 
	}
}
?>
[PHP] pobierz, plaintext 

I teraz ten kod gdzie podejrzewam, że jest błąd..

[PHP] pobierz, plaintext 
<?php
reguire_once ('class.PropertyObject.php');
 
class Address extends PropertyObject {
 
	function __construct($addressid) {
		$arData = DataManager::getAddressData($addressid);
 
		parent::__construct($arData);
 
		$this->propertyTable['addressid'] = 'adres_id';
		$this->propertyTable['id'] = 'adres_id';
		$this->propertyTable['entityid'] = 'jednostka_id';
		$this->propertyTable['address1'] = 'sadres1';
		$this->propertyTable['address2'] = 'sadres2';
		$this->propertyTable['city'] = 'smiasto';
		$this->propertyTable['zipcode'] = 'skod';
		$this->propertyTable['type'] = 'styp';
	}
 
 
   function validate () {
   
	   if (strlen($this->zipcode) != 6) {
	   $this->errors['zipcode'] = 'Należy podać poprawny kod pocztowy.';
	   } 
 
	   if ($this->address1) {
	   $this->errors['address1'] = 'Adres to pole wymagane';
	   } 
 
	   if ($this->city) {
	   $this->errors['city'] = 'Miasto to pole wymagane';
	   } 
	   
	   if (sizeof($this->errors)) {
		 return false;
	   } else {
		 return true;
	   }
 
   }
}
?>
[PHP] pobierz, plaintext 

Wydaje mi się, że błąd jest w metodzie validate(). Autor odwołuje suię w ten sposób

[PHP] pobierz, plaintext 
<?php
if ($this->address1) {
	   $this->errors['address1'] = 'Adres to pole wymagane';
	   }
?>
[PHP] pobierz, plaintext 

a powinno być chyba

[PHP] pobierz, plaintext 
<?php
if ($this->propertyTable['address1']) {
	   $this->errors['address1'] = 'Adres to pole wymagane';
	   }
?>
[PHP] pobierz, plaintext 

jeśli nie to dlaczego właśnie tak?

[Ludvik]

Nie podoba mi się ten kod. Po pierwsze właściwość nie jest walidatorem, co najwyżej może być walidowana (czyli raczej Validable). Poza tym nie rozumiem, po co deklarować metodę validate, w klasie PropertyObject... Jest to typowy przykład zastosowania metody abstrakcyjnej, a nie bezsensownego wklepywania pustej...

Tak poza tym, to dlaczego my mamy to sprawdzać? Więcej się nauczysz, jak sam będziesz chciał to sprawdzić... Odpal skrypt, zdebuguj go xdebugiem czy czymś innym...

[Jarod]

Nie podoba mi się ten kod. Po pierwsze właściwość nie jest walidatorem, co najwyżej może być walidowana (czyli raczej Validable). Poza tym nie rozumiem, po co deklarować metodę validate, w klasie PropertyObject... Jest to typowy przykład zastosowania metody abstrakcyjnej, a nie bezsensownego wklepywania pustej...

Coraz częściej dochodze do wniosku, że ta książka jest bardzo zamotana. Chyba tylko po to, żeby więcej stron było.

Zauważ, że masz funkcje __get() oraz __set() - pierwsza wykorzystywana jest wtedy kiedy robisz

Ok. Powoli łapie. Ale nie mogę zrozumieć tego kawałka kodu (z metody __get() ):

[PHP] pobierz, plaintext 
<?php
if (method_exists($this, 'get' , $propertyName)) {
			return call_user_func(array($this, 'get' , $propertyName));
		}
		else { 
		return $this->data[$this->propertyTable[$propertyName]];
?>
[PHP] pobierz, plaintext 

Po co sprawdza się czy metoda istnieje. Nie łapie też argumentów tej metody oraz call_user_func. Czytałem o tych funkjach ale nie mogę załapać. Możecie mi to wytłumaczyć?