__get und __set

Zitat von Paxi

Danke für das Feeback mat.
Ich habe mir auch bereits gedacht, dass es ableitenden Klasse weiters möglich sein sollte __set und __get zu ändern. Das ist auch im aktuellen Code bereits implementiert.
Bezüglich der Sichtbarkeit stimme ich dir auch zu, aber liegt das "Problem" da nicht allgemein an der Verwendung von __get und __set?
__set() is run when writing data to inaccessible properties.
__get() is utilized for reading data from inaccessible properties.

Oder meinst du weil ich ein Array verwende und die Properties nicht wirklich existieren? Ich habe auch schon überlegt, die einzelnen Properties wirklich in jeder Klasse zu speichern und dann zb mit $this->$i im getter zurückzugeben, wäre das besser womöglich?

Hab hier nochmal einen kleinen Zusatz zum vorherigen Beitrag gemacht mit der Basisklasse.

hier

Code: PHP

<?php
abstract class BaseObject {
	
protected $validationfunctions;
	
/**
 * constructor
 */
public function __construct() {
	$this->reset();
	$this->initFunctions();
}

/**
 * magic method __get
 * calls onGet() in deriving class
 *
 * @param String $i
 *
 * @return mixed
 */
public function __get($i) {
	// only returns non private fields
	if(!array_key_exists($i, get_class_vars($this->className()))) {
		return $this->onGet($i);
	}

	echo '<br />'.$this->className().' property $'.$i. ' is private or does not exist';
	return null;
}

/**
 * magic method __set
 * calls onSet() in deriving class
 * 
 * @param String $i
 * @param mixed $val
 *
 * @return void
 */
public function __set($i, $val) {
	if(!array_key_exists($i, get_class_vars($this->className()))) {
		echo '<br />'.$this->className().' property $'.$i. ' is private or does not exist';
		return;
	}

	if(call_user_func($this->validationfunctions[$i], $val)) {
		$this->onSet($i, $val);
	} else {
		echo '<br />'.$this->className().' error setting property $'.$i;
	}
}


/**
 * initialize all properties
 * invoked in constructor
 * 
 * @return void
 */
public abstract function reset();

/**
 * initialize all validation functions
 * invoked in constructor
 * 
 * @return void
 */
protected abstract function initFunctions();

/**
 * @see BaseObject::set()
 * @param String $i
 * @param mixed $val
 * 
 * @return void
 */
protected abstract function onSet($i, $val);

/**
 * @see BaseObject::get()
 * @param String $i
 * 
 * @return mixed
 */
protected abstract function onGet($i);

/**
 * @return String classname
 */
protected abstract function className();
}

class Object extends BaseObject {
	
	// private data stays private from outside and deriving classes but can be used within this class
	private $id;
	
	// protected data can be accessed from outside and deriving classes but is validated before setting
	protected $username;

/**
 * initialize member var(s)
 *
 * @return void
 */
public function reset() {
	$validationfunctions = array();
	
	$this->id = 10;
	$this->username = 0;
}

protected function onSet($i, $val) {
	$this->$i = $val;
}
	
protected function onGet($i) {
	return $this->$i;
}

protected function className() {
	return 'object';
}

/**
 * initialize functions
 * 
 * @return void
 */
protected function initFunctions() {
	
	$this->validationfunctions['username'] = function($val) {
		return ( is_numeric($val) ? false : true);
	};
}

}
?>