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]

Powinno być

[PHP] pobierz, plaintext 
<?php
method_exists($this, 'get' . $propertyName)
?>
[PHP] pobierz, plaintext 

Pierwszy argument to instancja klasy. Drugi argument ('get' . $propertyName) to nazwa metody. Nie widzę nic dziwnego w tym wywołaniu...

Dalej...

[PHP] pobierz, plaintext 
<?php
call_user_func(array($this, 'get' . $propertyName));
?>
[PHP] pobierz, plaintext 

Tak samo... Pierwszy argument ma być typu callback. Jest to nazwa funkcji albo w przypadku wywoływania metod tablica. Pierwszy elementem tablicy jest referencja do obiektu (albo nazwa klasy, jeżeli wywołujemy statyczną metodę). Drugi element to nazwa metody.

Oni to zrobili po to, aby przeciążyć "magiczne" metody, ale nie tracić zachowania metod programisty getXXX... Jeżeli istnieje metoda getWlasciwosc, a ty poprosisz o dane tak:

[PHP] pobierz, plaintext 
<?php
echo $obj->wlasciwosc;
?>
[PHP] pobierz, plaintext 

To i tak efektem będzie wywołanie metody getWlasciwosc...

[Jarod]

Powinno być

[PHP] pobierz, plaintext 
<?php
method_exists($this, 'get' . $propertyName)
?>
[PHP] pobierz, plaintext 

Pierwszy argument to instancja klasy. Drugi argument ('get' . $propertyName) to nazwa metody. Nie widzę nic dziwnego w tym wywołaniu...

$this nie jest instancją klasy(?) getNazwaMetody samo w sobie nie jest dziwne. Dziwne jest to, że w przykładzie, który podałem na samym początku nie ma metody. Jest tylko magiczna (czyli jak dobrze przeczytałem wywoływana automatycznie gdy ktoś chce wywołać nieistniejącą metode), nie ma żadnej innej metody.

Dalej...

[PHP] pobierz, plaintext 
<?php
call_user_func(array($this, 'get' . $propertyName));
?>
[PHP] pobierz, plaintext 

Tak samo... Pierwszy argument ma być typu callback. Jest to nazwa funkcji albo w przypadku wywoływania metod tablica. Pierwszy elementem tablicy jest referencja do obiektu (albo nazwa klasy, jeżeli wywołujemy statyczną metodę). Drugi element to nazwa metody.

Oni to zrobili po to, aby przeciążyć "magiczne" metody, ale nie tracić zachowania metod programisty getXXX... Jeżeli istnieje metoda getWlasciwosc, a ty poprosisz o dane tak:

[PHP] pobierz, plaintext 
<?php
echo $obj->wlasciwosc;
?>
[PHP] pobierz, plaintext 

To i tak efektem będzie wywołanie metody getWlasciwosc...

Thx.