Klasa input - jak wy to robicie Opcje

[tuner]

Możecie mi opisać jak wy rozwiązujecie problem klasy z danymi wejściowymi? Ja mam wątpliwości jak trzymać dane, jak ustalać do nich dostęp, czy dzielić je na get, post, session przy wyciągniu z klasy itp.

Ktoś może podzielić się swoim doświadczeniem?

[M4chu]

Ja w klasie Http mam metody get, post, cookie, file, ktore zwracaja dane wejsciowe (przy inicjalizacji obiektu robie filtrowanie itp). Dla sesji mam osobna klase (patrz art na php.pl ). A co do dostepu to tak samo jak tablice, ktore zawieraja ta dane - globalny.

[tuner]

A jak wygląda zapisywanie danych do sesji? Posługujesz się zwykłą tablicą $_SESSION[$var] czy masz jakąś metodę np. saveVar($var) ?

[M4chu]

Mozna i tak i tak, bo podmieniam session handlera, ale wole Session::set( 'name', value' ), bo gdy nie ma zmiennej sesyjnej to nie ma tez indeksu dla tablicy $_SESSION i wywala brzydki blad, a tak to ladnie sobie moge to obsluzyc w funkcji

[tuner]

[PHP] pobierz, plaintext 
<?php
$this->vars=array_merge($_GET?array():$_GET,$_POST?array():$_POST,$_SESSION?$_SESSION:array(),$_COOKIE?$_COOKIE:array());
?>
[PHP] pobierz, plaintext 

Do klasy input wcisnąłem jeszcze

[PHP] pobierz, plaintext 
<?php
		function saveVar($k,$v)
		{
				$_SESSION[$k]=$v;
				$this->vars[$k]=$v;
		}
?>
[PHP] pobierz, plaintext 

Co o tym myślisz?

Mógłbym to napisać od ręki, ale stwierdziłem, że przemyśle Session Handlera i klase Input by potem nie wprowadzać w całym skrypcie zmian.

Ten post edytował tuner 16.05.2005, 16:05:34

[M4chu]

Hehe, Twoje $this->vars, to nic innego jak $_REQUEST Zreszta moim zdaniem, Laczenie wszystkich danych wejsciowych w jedna tablice jest zle, i to bardzo: przeciez jakies $id z posta a z cooki to cos zupelnie innego. Wole trzymac odpowiednie dane w odpowiednich tablicach i robic do nich gettery i settery. A co do metody saveVar(), coz trudno tu wymyslic cos nowego To samo co przy innych - oddzielenie $session od $get, $post itp.
pozdro

[hawk]

Ja bym tak nieśmiało zaczął od tego, w jakim celu w ogóle jest to robione... nie żebym uważał, że żadna nadbudowa nad $_GET itd jest niepotrzebna. Wręcz przeciwnie. Ale najpierw trzeba określić, co się chce dzięki temu uzyskać, a potem to zaimplementować. Bo rozwiązań może być tyle, co developerów. A większość z nich, jak to zwykle bywa, będzie bez sensu, bo większość developerów nie zastanowi się nawet, czemu to ma służyć, tylko zrobi na hurra .

[chfast]

najpierw trzeba określić, co się chce dzięki temu uzyskać

No właśnie. To co się chce przez to uzyskać? I co się da przez to uzyskać?

[BugTomek]

najpierw trzeba określić, co się chce dzięki temu uzyskać

No właśnie. To co się chce przez to uzyskać? I co się da przez to uzyskać?

Np. możemy się pozbyć problemu z niezainicjowanymi zmiennymi, np.

[PHP] pobierz, plaintext 
<?php
$foo = $_SESSION['foo'];
?>
[PHP] pobierz, plaintext 

Okazuje się, że nie ma w sesji takiej zmiennej i pojawia się błąd:

Kod

Notice: Undefined index: foo in ... on line ...

Jeśli zrobimy to w ten sposób:

[PHP] pobierz, plaintext 
<?php
$foo = $session->get('foo');
?>
[PHP] pobierz, plaintext 

To możemy wszystko ładnie załatwić w metodzie get():

[PHP] pobierz, plaintext 
<?php
function get($var) {
if (!isset($_SESSION[$var]) 
	return NULL;
else
	return stripslashes($_SESSION[$var]);
}
?>
[PHP] pobierz, plaintext 

To tylko prosty przykład, pokazujący również kolejną zaletę tego rozwiązania, a mianowicie możliwość wprowadzenia łatwego filtrowania danych. To tylko początek: możliwości jest o wiele więcej.

[hawk]

@BugTomek: akurat twój przykład jest zły, bo zamienia jeden problem na drugi. Owszem, nie mam problemu z niezainicjowanymi zmiennymi, ale mam problem ze zmiennymi, które przyjęły wartość NULL, a ja o tym nie wiem. I tak trzeba sprawdzić wartość zmiennej w kodzie, żeby upewnić się, że ona w ogóle przyszła. Czyli: to co napisałeś nie daje żadnej korzyści.

Natomiast stripslashes to już dobre zastosowanie. Chociaż w złym miejscu. Nie powinno się znajdować w momencie, kiedy ktoś chce pobrać wartość z $_SESSION, ale powinno od razu wstawiać do $_SESSION poprawioną wartość. Inaczej nie tylko wywołujemy bez sensu stripslashes wielokrotnie, ale do tego ryzykujemy, że ktoś odwoła się bezpośrednio do $_SESSION i wszystko zmarnuje.

[BugTomek]

Co do pierwszego to zgadzam się w zupełności. Są co prawda przypadki, kiedy takie coś wystarcza, ale generalnie nie jest to idealne rozwiązanie - jak wobec tego zrobić to lepiej?
Co do drugiego, to faktycznie, do ideału temu rozwiązaniu wiele brakuje, ale chodziło jedynie o generalną zasadę co można z tym robić. Filtrowanie należałoby zrobić bardziej rozbudowane, przy wyciąganiu to w zasadzie bardziej logiczne byłoby addslashes (jeżeli dane z sesji umieszczamy w zapytaniach SQL), ale to również jedynie jako opcja. Możliwości jest tu mnóstwo, ale wiadomo, że chodzi o to, żeby te dane łatwo walidować.

[Ociu]

Ja mam zawsze byłem odmieńcem i poprzez input pobieram: ip, host, browser użytkownika, pobieranie danych z adresów, $_POST.

[PHP] pobierz, plaintext 
<?php
public function loadPost()
{
if(isset($_POST))
{
	 foreach($_POST AS $key => $value )
	 {
		  $this -> inputs[$key] = $value;
	 }
}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Ociu 18.05.2005, 12:56:46

[matid]

Ja osobiście wykorzystuje taką klasę nie tylko do filtrowania (strip_slashes, itp.), ale raczej do rzutowania typów, czyli implementuje sobie metody getInt, getString, itp.

[bacca]

A ja robię tak:

[PHP] pobierz, plaintext 
<?php
 
abstract class input {
 
	/* gdy potrzebuje liczby calkowitej */
	public static function getInt(&$source, $key) {
  if ((is_numeric($source[$key]))&&($source[$key] == (int)$source[$key])) {
  	return (int)$source[$key];
  } else {
  	return false;
  }
	}
 
	/* gdy potrzebuje liczby naturalnej (np. id rekordu w bazie) */
	public static function getNat(&$source, $key) {
  if ((is_numeric($source[$key]))&&($source[$key] > 0)&&($source[$key] == (int)$source[$key])) {
  	return (int)$source[$key];
  } else {
  	return false;
  }
	}
 
	public static function getFloat(&$source, $key) {
  if (is_numeric($source[$key])) {
  	return (float)$source[$key];
  } else {
  	return false;
  }
	}
 
	/* gdy potrzebuje pojedynczego slowa */
	public static function getWord(&$source, $key) {
  if (preg_match('/^([a-zA-Z0-9_-]+)$/', $source[$key])) {
  	return $source[$key];
  } else {
  	return false;
  }
	}
 
	/* gdy potrzebuje dowolnego tekstu ktory mam potem wyswietlic */
	public static function getText(&$source, $key) {
  global $engine;
  if (get_magic_quotes_gpc()) {
  	return htmlspecialchars($source[$key], ENT_NOQUOTES);
  } else {
  	return addslashes(htmlspecialchars($source[$key], ENT_NOQUOTES));
  }
	}
}
?>
[PHP] pobierz, plaintext 

A potem uzywam np. tak:

[PHP] pobierz, plaintext 
<?php
 
$login = input::getWord($_GET, 'login');
 
?>
[PHP] pobierz, plaintext 

Dodam jeszcze, że używam tego tylko do GET, POST i ciastek, bo niby jak user ma sie dobrać do sesji.

Ten post edytował NuLL 1.07.2005, 16:25:35