[OOP] Klasa do filtrowania tekstu Opcje

[_olo_1984]

Witam

napisałem taką oto klasę, do filtrowania tekstu. W związku z tym, że dopiero zaczynam zabawę z OOP (w strukturalnym radzę sobie całkiem dobrze i czas podnieść poprzeczkę) mam do Was następujące pytania:
1. Czy klasa którą napisałem to nie pseudo oop? czy tak to powinno wyglądać (czy dobrze kombinuje)
2. Czy filtrowanie danych wchodzących do bazy (tryb 1) jest dobrym zabezpieczeniem przed atakami?
3. Już poza tym tematem, chodzi o smarty (z których tez korzystam), jak w innym obiekcie moge urzyć smarty, oraz jak w pliku inkludowanym urzyć smarty np. mam w głównym katalogu index.php (tu smarty działa ok), ale dołanczam do indexa plik z katalogu /includes/1.php i w tym pliku 1.php gdy prubuję uzyc szablonu, to nie działa (nie dolancza plikow tpl). smarty dolanczam w index. a po nim ten plik 1.php (mam nadzieje ze nie zamieszalem).

poniżej kod:

[PHP] pobierz, plaintext 
<?php
 
class text {
	private $zakazane = array("script", "SCRIPT",
					   "type", "TYPE",
					   "name", "NAME",
					   "union", "UNION",
					   "select", "SELECT",
					   "delete", "DELETE",
					   "modify", "MODIFY",
					   "show", "SHOW",
					   "NULL", "null", "NUL", "nul",
					   "<", ">", "'", "/", " ", "&", "%", ";", ":", "*", "&", "?");
	public $tekst;
	public $zmieniony;
	public function filter($tryb) {
		switch($tryb) {
			case '1':
			return $this->zmien_tekst_1();
			break;
 
			case '2':
			return $this->zmien_tekst_2();
			break;	
 
			case '3':
			return $this->zmien_tekst_3();
			break;		
		}
	}
	private function show_tekst() {
		echo $this->zmieniony;
	}
	private function zmien_tekst_1() {
		$this->zmieniony=str_replace($this->zakazane, "", strip_tags($this->tekst));
		return $this->show_tekst();
	}
	private function zmien_tekst_2() {
		$this->zmieniony=stripslashes($this->tekst);
		return $this->show_tekst();
	}
	private function zmien_tekst_3() {
		$this->zmieniony=htmlentities($this->tekst);
		return $this->show_tekst();
	}
}
$text=new text;
?>
[PHP] pobierz, plaintext 

pozdrawiam

[cinekz]

1. Skoro mówimy już o OOP to ta klasa nie ma z nim nic wspólnego. To powinno wyglądać inaczej. Powinieneś zrobić klasę bazową dla każdego filtra. Każdy filtr byłby osobną klasą wyprowadzoną z klasy głównej. Przykładowy kod:

[PHP] pobierz, plaintext 
<?php
abstract class Filter //klasa abstrakcyjna dla wszystkich filtrów
{
	private $value = '';
 
	public function __construct( $value = null )
	{
		$this->setValue( $value );
		$this->run();
	}
 
	public function setValue( $value )
	{
		if( !is_null( $value ) )
		{
			$this->value = $value;
		}
	}
 
	public function getValue()
	{
		return $this->value;
	}
 
	abstract protected function run();
}
 
class HtmlEntitiesFilter extends Filter // implementacja Filtra zamieniające tagi na encje
{
	protected function run()
	{
		$this->setValue( htmlentities( $this->getValue() ) );
	}
}
 
class AddSlashesFilter extends Filter // przykład Filtra dodającego slashe
{
	protected function run()
	{
		$this->setValue( addslashes( $this->getValue() ) );
	}
}
 
 
$filter = new HtmlEntitiesFilter( '<b>sasasas</b>' );
 
echo $filter->getValue(); // wynik:  &lt;b&gt;sasasas&lt;/b&gt;
?>
[PHP] pobierz, plaintext 

Myślę, że pomogłem. Pozdrawiam Marcin.

PS Dąż raczej do prywatności składowych klasy i udostępniaj akcesory.

Ten post edytował cinekz 26.02.2008, 16:26:34