[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

[Cysiaczek]

Mi sie nie podoba kilka rzeczy, w tym jedna jest błędem.

1. Stosuj angielskie nazwy zmiennych, metod itd. Polskie wyglądają po prostu komicznie, a u Ciebie to wogóle jest esperanto
2. W show_text() widze jakieś echo. Błąd! nigdy echo w takiej metodzie. Zawsze return
3. Nazwy metod typu zmien_tekst_1() nc nie mówią o tym, co dana operacja robi. Jedną z zasad OOP jest nazywanie metod zgodnie tym, co robią. Zwłaszcza, jeśli dotyczy to interfejsu publicznego (wiem, masz prywatny, ale czytaj dalej)
4. To, że zastosuję filter($tryb) też mi nic nie mówi

[PHP] pobierz, plaintext 
<?php
$text=new text();
$text->tekst="<b>lol</b>";
$text->filter(2); // No i skąd mam widzieć, co robi filter z parametrem 2?
?>
[PHP] pobierz, plaintext 

5. Ograniczasz funkcję strip_tags() - ona ma jeszcze więcej parametrów, których teraz nie można użyć.

Podsumowując. Kod jest raczej bezużyteczny, bo powoduje więcej problemów niż rozwiązuje i do tego jest nieintuicyjny.
Nie martw się jednak, bo akurat tego typu klasy zazwyczaj są mało przydatne. Zazwyczaj taki kod leci automatycznie gdzieś we framworku, albo wogóle w modelu. To nie jest najlepszy pomysł na naukę oop - lepiej postaraj się wymodelować jakieś zachowania programu np. klasę News, albo księgę gości etc.

Pozdrawiam