__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ł?

[dr_bonzo]

To nie byla gotowa klasa (nie mialem jak przetestowac, pisana od reki w pospiechu) -- pomylki poprawie pozniej (zaraz wychodze).

[hawk]

Refactoring:

Jeżeli już, to w parametrze nie przekazujemy 'cookie', tylko $_COOKIE. Oszczędność kodu, czasu i po prostu czytelniejsze. \

Idźmy dalej. Potrzebna jest funkcjonalność pozwalająca na wyciągnięcie wszystkich parametrów np. POST. Możemy więc dodać metodę np. getAllParameters(). Ale znacznie lepiej użyć wbudowanych iteratorów - funkcjonalność będzie znacznie większa, a wysiłek mały.

Kontynuujmy. Ponieważ to jest wrapper na tablicę, zamiast __get lepiej przeciążyć operator []. Czyli implementujemy kolejny interfejs php (chyba ArrayAccess?) i możemy używać naszej klasy tak jak tablicy. Będzie to bardziej naturalne w użyciu, więc chyba lepsze.

Skutek? Napisaliśmy kawałek kodu, który zachowuje się tak samo jak oryginalna tablica, wygląda jak tablica, nie robi praktycznie nic innego, za to zajmuje miejsce. Coś jest nie tak...

Tak naprawdę, jest jedna różnica: skrypt może zmienić zawartość $_POST, a nie może zmienić zawartości naszej pseudo-tablicy. Co może być zaletą. Dla mnie jest. Ale pomijając to, taki skrypt może być początkiem jakiejś klasy HttpRequest, ale jako cel sam w sobie jest mało praktyczny.

[NuLL]

@hawk - a mi się wydaje, że każdy powinien robić tak jak ma ochotę.
To co jest pod spodem to mój lib, który stosuję do prawie każdego projektu - niestety pod php 4, ale wiadomo jakie mamy realia. Mnie osobiście jeśli koduje coś saemu takie rozwiązanie w zupełności wystarcza. Jakoże naprawdę nie wiele reczy przechodzi przez $_GET nie mam potrzeby rozgraniczania $_GET i $_POST. Reszta tablic to inna mańka (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[PHP] pobierz, plaintext 
<?
class inputter{
	var $aData=array();
 
	function clean_value($val){
		global $config_vars;
 
		if ($val == &#092;"\") return \"\";
 
		$val = str_replace( &#092;" \", \" \", $val );
 
		if ( $onfig_vars['_strip_space_chr'] ){ $val = str_replace( chr(0xCA), &#092;"\", $val );}
 
		$val = str_replace( &#092;"&\", \"&amp;\", $val );
		$val = str_replace( &#092;"<!--\", \"<!--\"  , $val );
		$val = str_replace( &#092;"-->\", \"-->\", $val );
		$val = preg_replace( &#092;"/<script/i\"  , \"<script\"   , $val );
		$val = str_replace( &#092;">\", \"&gt;\", $val );
		$val = str_replace( &#092;"<\", \"&lt;\", $val );
		$val = str_replace( &#092;"\"\", \"&quot;\", $val );
		$val = preg_replace( &#092;"/n/\", \"<br />\", $val ); //nl2br()
		$val = preg_replace( &#092;"/$/\", \"$\", $val );
		$val = preg_replace( &#092;"/r/\", \"\", $val ); // /r
		$val = str_replace( &#092;"!\", \"!\", $val );
		$val = str_replace( &#092;"'\", \"'\", $val ); //sql sec
		//unicode
		if ( $config_vars['_allow_unicode']){ $val = preg_replace(&#092;"/&amp;#([0-9]+);/s\", \"
\", $val );}
		//wywalamy slashe
		if ( $config_vars['_stripslashes'] ){$val = stripslashes($val);}
		//wywalamy backslash'e
		$val = preg_replace( &#092;"/(?!&amp;#|?#)/\", \"\", $val );
		return $val;
	}
 
	function clean_key($key){
		if ($key == &#092;"\"){return \"\";}
		$key = htmlspecialchars(urldecode($key));
		$key = preg_replace( &#092;"/../\"		   , \"\"  , $key );
		$key = preg_replace( &#092;"/__(.+?)__/\"  , \"\"  , $key );
		$key = preg_replace( &#092;"/^([w.-_]+)$/\", \"$1\", $key );
		return $key;
	}
 
	function parse_incoming(){
		$return = array();
		if( is_array($_GET)){
			while( list($k, $v) = each($_GET) ){
				if ( is_array($_GET[$k]) ){
					while( list($k2, $v2) = each($_GET[$k]) ){
						$return[ inputter::clean_key($k)][ inputter::clean_key($k2)]=inputter::clean_value($v2);
					}
				}else{
					$return[ inputter::clean_key($k) ] = inputter::clean_value($v);
				}
			}
		}
 
		if( is_array($_POST) )
		{
			while( list($k, $v) = each($_POST) )
			{
				if ( is_array($_POST[$k]) )
				{
					while( list($k2, $v2) = each($_POST[$k]) )
					{
						$return[inputter::clean_key($k)][inputter::clean_key($k2)]=inputter::clean_value($v2);
					}
				}
				else
				{
					$return[ inputter::clean_key($k) ] = inputter::clean_value($v);
				}
			}
		}
		$return['request_method'] = strtolower($_SERVER['REQUEST_METHOD']);
		return $return;
	}
}
 
?>
[PHP] pobierz, plaintext 

Ta klasa zawraca mi tablicę wszystkich danych ktore przyszły.