Publiczny modyfikator dostępu..., ...a może jednak korzystać? Opcje

[starach]

Cześć,

Tak się zastanawiałem ostatnio nad zmianą przyzwyczajeń jeśli chodzi o tą sprawę. Deklarowanie wszystkich składowych jako prywatnych lub chronionych jest ni mniej ni więcej upierdliwe jak pisanie setterów i getterów za pomocą __get() __set(), ale estety trochę mam wrażenie na tym cierpi. Wyskrobałem sobie przed chwilą następujący kod:

[PHP] pobierz, plaintext 
class PropertiesInjector
{
	/**
	 * array('id' => array('type' => 'integer'),
	 *       'name' => array('type' => 'string'),
	 *			 'ip' => array('length' => <int>))
	 * @var array
	 */
	protected $aPatterns;
	/**
	 * Stores properties and their values
	 * @var array
	 */
	protected $aProperties;
	/**
	 * Returns class property or throws exception if it does not exists
	 * @param string $sName
	 * @return mixed
	 */
	public function  __get($sName)
	{
		$this->_propertyExists($sName);
		return isset($this->aProperties[$sName]) ? $this->aProperties[$sName] : null;
	}
	/**
	 * Sets class property value or throws an error if value
	 * is other type than declared or property does not exists
	 * @param string $sName
	 * @param mixed $mValue
	 */
	public function __set($sName, $mValue)
	{
		$this->_propertyExists($sName);
 
		if(isset($this->aPatterns[$sName]['type']))
		{
			$aTypes = $this->aPatterns[$sName]['type'];
			$aTypes = \is_array($aTypes) ? $aTypes : array($aTypes);
			if(false === \array_search(gettype($mValue), $aTypes))
			{
				throw new Exception\PropertyValue($sName, $mValue);
			}
		}
		if(isset($this->aPatterns[$sName]['length']))
		{
			if(\strlen($mValue) > $this->aPatterns[$sName]['length'])
			{
				throw new Exception\PropertyToLong($sName, $mValue, $this->aPatterns[$sName]['length']);
			}
		}
		$this->aProperties[$sName] = $mValue;
	}
	/**
	 * Throws exception if property name is incorrect
	 * @param string $sName
	 */
	private function _propertyExists($sName)
	{
		if(!isset($this->aPatterns[$sName])) {
			throw new Exception\PropertyName($sName);
		}
	}
[PHP] pobierz, plaintext 

Tak więc się zastanawiam jak to teraz powinno być z tymi modyfikatorami dostępu. Co o tym sądzicie?

[starach]

__isset owszem z tym że chciałbym żeby w przypadku nie istniejącej właściwości skrypt płuł mi wyjątkiem czego wydaje mi się nie należy robić przez __isset żeby dać możliwość programiście sprawdzenia czy dana właściwość istnieje.

[PHP] pobierz, plaintext 
/**
	 * Checks whether class property exists or not
	 * @param string $sName
	 * @return boolean
	 */
	public function __isset($sName)
	{
		return isset($this->aPatterns[$sName]);
	}
	/**
	 * Throws exception if property name is incorrect
	 * @param string $sName
	 */
	private function _propertyExists($sName)
	{
		if(!isset($this->$sName)) {
			throw new Exception\PropertyName($sName);
		}
	}
[PHP] pobierz, plaintext 

Czyli nie ma żadnych przeciwskazań. Super.