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?

[Crozin]

Jaki jest cel tej walidacji? Nie wygląda mi to na sprawdzanie danych pochodzących od użytkownika (formularze itp.), a sprawdzanie samego programisty jest raczej średnim pomysłem. By to miało sens musiałbyś tworzyć wiele skomplikowanych reguł (w końcu ograniczenie się do samego typu to słaba walidacja). W dodatku w PHP jako języku, który nie przykłada większej uwagi do typu danych mogłoby to powodować niepotrzebne "wymuszenia" typu: doSth((int) $id) (bo ID było stringiem).

btw: Źle używasz wyjątków. To co wyrzucasz (instancję jakiej klasy) powinno być uwarunkowane tym co powoduje dany błąd, a nie skąd on pochodzi. W tym przypadku akurat powinieneś wykorzystać istniejącą już klasę InvalidArgumentException i ewentualnie DomainException.