obsluga formularza Opcje

[skowron-line]

[PHP] pobierz, plaintext 
<?php
 
class obsluga_formularza {
 
	public $nazwa;
	public $adres;
	private $wzorzec = '/^[a-zA-Z0-9.-_]+@[a-zA-Z0-9-.]+.[a-zA-Z]{2,4}$/';
 
 
	function __construct($nazwa, $adres) {
 
		$this -> nazwa = $nazwa;
		$this -> adres = $adres;
 
	}
 
	function policz_znaki() {
 
		$ile_znakow = strlen($this -> nazwa);
		return $ile_znakow;
 
	}
	function sprawdz_adres() {
 
		if(!preg_match($this ->wzorzec, $this -> adres)){
			return "bledny adres e-mail";
		}else{
			return "adres e-mail jest poprawny";
		}
 
	}
	function pokaz() {
		return $this -> policz_znaki().
			$this -> sprawdz_adres();
	}
}
 
 
echo'<form action="" method="post">
		nazwa:<input type="text" name="nazwa">
		adres:<input type="text" name="adres">
	<input type="submit" name="submit">';
 
if(isset($_POST['submit'])){
 
		$parm = new obsluga_formularza($_POST['nazwa'], $_POST['adres']);
 
		echo $parm -> pokaz();
 
}
?>
[PHP] pobierz, plaintext 

od jakiegos czasu usiluje sie nauczyc programowac obiektowo i napisalem cos takiego.
nie wiem czy jest to napisane poprawnie ale dziala jak ja to sobie zalorzylem.
lecz moje pytanie brzmi czy tak powinno to wyglada czy moze sprawdzanie poprawnosci wpisanych danych powinno byc poza klasa?? moze to zly przyklad ale chodzi mi o to ze jezeli teraz chcialbym sprawdzic czy pola zostaly wypelnione to czy mam to napisac w klasie czy poza nia??

[Ace]

Jasne,

Najprostrzy przyklad, twoja klasa "obsluga_formularza" jest odpowiedzialna za wyswietlenie DANEGO formularza, oraz metody do walidacji ma w sobie. Czemu?

Pomysl lepiej nad obiektem formularza, ktory bedziesz mogl rozszerzac. Walidacja w formularzu? Zle zagranie. Walidacje powinienes trzymac w zupelnie innym miejscu - inna klasa? ktora bedzie zawierac zbior wszystkich metod walidacji czego zywnie sobie chcesz...

Ludzie chcacy pisac obiektowo, piszac proceduralnie po prostu ubieraja zbior funkcji w obiekt... (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pomysl inaczej, Formularz to jakis obiekt ma napewno jakies wlasnosci - lista pol, akcja ktora stanie sie po submit, wczesniej walidacja danych, kazde pole ma przypisane do siebie sposob walidacji, moze nie miec walidacji, moze miec 1 sposob walidacji, albo kilka walidacji...

Twojego przykladu nie da sie rozszerzac. W obiektowosci liczy sie to, ze dobrze napisane obiekty mozesz wykorzystac wiele razy, pozatym latwosc rozbudowy/przebudowy jest wielkim plusem OOP. Wszystko sklada sie z obiektow. Wiec obsluga_formularza powinna tez dzielic sie na wiele mniejszych obiektow. Chociazby lista pol, albo walidacje w klasie do obslugi formularzy.