__get $_GET, $_POST, pobieranie danych od usera Opcje

[dr_bonzo]

Zamiast spac wpadlem na ciekawy pomysl i napisalem cos takiego:

[PHP] pobierz, plaintext 
<?php
class UserData
{
	private $arrValues = array();
 
	public UserData( $strSource )
	{
		$this->strSource = strval( $strSource );
	}
 
	public __get( $strValName )
	{
		if ( isset( $this->arrValues[ $strValName ] ) )
		{
			if ( is_null( $this->arrValues[ $strValName ] ) )
			{
				throw new Exception( 'Value not present' );
			}
			else
			{
				return $this->arrValues[ $strValName ];
			}
		}
		else
		{
			$arrSuperGlobalArray = NULL;
 
			switch ( $this->arrValues )
			{
				case 'get':
				{
					$arrSuperGlobalArray &= $_GET;
					break;
				}
 
				case 'post':
				{
					$arrSuperGlobalArray &= $_POST;
					break;
				}
 
				case 'cookie':
				{
					$arrSuperGlobalArray &= $_COOKIE;
					break;
				}
 
				default :
				{
					throw new Exception( 'Invalid source type '' .  $this->strSource . ''' );
				}
			}
 
			if ( isset( $arrSuperGlobalArray[ $strValName ] ) )
			{
				return $arrSuperGlobalArray[ $strValName ];
			}
			else
			{
				$this->arrValues[ $strValName ] = NULL;
			}
		}
	}
}
 
$objGetValues = new UserData( 'get' );
print( $objGetValues->action );
 
$objPostValues = new UserData( 'post' );
print( $objPostValues->id );
 
?>
[PHP] pobierz, plaintext 

Nowy sposob pobierania danych wykorzystujacy php5. Nie testowany -- nie mam serwera pod reka. Co o tym sadzicie?

[Imperior]

1. Brak deklaracji $this->strSource
2. Niedopracowanie/nieskończone: nie ma możliwości aby:

[PHP] pobierz, plaintext 
<?php
isset( $this->arrValues[ $strValName ] )
?>
[PHP] pobierz, plaintext 

było prawdziwe, a zarazem:

[PHP] pobierz, plaintext 
<?php
is_null( $this->arrValues[ $strValName ] )
?>
[PHP] pobierz, plaintext 

nie prawdziwe.
3. Korzystasz z $this->arrValues w switchu zamiast $this->strSource
4. Ta klasa nic nie robi, czy to jest podstawa, którą można jakoś rozszerzać? Jeśli tak, to pola jako protected powinny być.
5. Jeśli to ma być rozszerzane, to może jakiś dekorator by się przydał?