[PHP]Podstawy obiektówki Opcje

[maxx000]

Witajcie serdecznie.

Jako, że ciągle próbuję walczyć z programowaniem obiektowym w PHP i chęcią rozgryzienia tego tematu, dziś napisałem jakiś prosty skrypt a'la rejestracja.

Ogółem działa, przy prosiłbym doświadczonych użytkowników o krytykę i ewentualne poprawki - no chyba, że całość jest nie w ten deseń, jaki być powinno.

Z góry dziękuję.

Poniżej kody z dwóch plików, które napisałem:
Plik z klasą:

[PHP] pobierz, plaintext 
<?php
$db=mysql_connect("localhost", "root", "krasnal");
$db_sel = mysql_select_db("projekt");
 
function oczysc($zmienna)
{
	$zmienna = strip_tags(trim($zmienna));
	return $zmienna;
}
################################ KLASY
class Rejestr
{
  public function form($login, $email)
 
	{
	 echo '
	<form action="" method="post">
	Login:<input type="text" name="login" value='.$login.'><br>
	Hasło:<input type="password" name="pass"><br>
	Potwierdź hasło:<input type="password" name="repass"><br>
	E-mail:<input type="password" name="email" value='.$email.'><br>
	<input type="submit" name="rejestr" value="Rejestruj">
	</form>
	';
	}
 
 
	public function sprawdz($login, $pass, $repass, $email)
	{
	$login=oczysc($login);// czy tego jakoś kurcze nie mozna 
	$pass=oczysc($pass);    // zautomatyzowac? :-) 
	$repass=oczysc($repass); // jakis 'myk' ze wszystkie przekazane dane
	$email=oczysc($email);   // filtrujemy przez oczysc
 
 
	$zap=mysql_query("Select id from uzytkownik where login='$login'");
	if((mysql_num_rows($zap)>0) || (strlen($login)<3))
	 {
		$blad .= "- Login zajęty, lub podano za krótki.<br>";
	 }
 
 
	$zap=mysql_query("Select id from uzytkownik where email='$email'");
	if((mysql_num_rows($zap)>0))
	 {
		$blad .= "- E-mail już zarejestrowany, lub podano niepoprawny.<br>";
	 }
 
	if(($pass!=$repass) || (strlen($pass))<4)
	 {
	 $blad .= "- Błąd przy podawaniu hasła.<br>";
	 }
 
	if(!empty($blad)) 
	{
	echo $blad;
	$this->form(oczysc($login), oczysc($email));
	}
	else
	echo 'Zarejestrowano poprawnie!';
	// mysql_query -> dodawanie uzytkownika
 
 
	}
 
}
?> 
[PHP] pobierz, plaintext 

Oraz plik główny:

[PHP] pobierz, plaintext 
<?php
  include_once('klasy.php');
	$rejestruj = new Rejestr();
 
 
 
	if($_SERVER['REQUEST_METHOD']=="POST")
	{
	 if(isset($_POST['rejestr']))
	  {
		 $rejestruj->sprawdz($_POST['login'], $_POST['pass'], $_POST['repass'], $_POST['email']);
 
		}
 
	}
	else
	{
 
 
 $rejestruj->form(oczysc($login), oczysc($email));	
	}
 
 
 
?> 
[PHP] pobierz, plaintext 

[Crozin]

Skoro to początki obiektówki to zapamiętaj sobie: podstawą OOP jest obiekt, nie klasa - podobnie jak podstawą budownictwa są budynki, nie ich plany/szkice. Dlatego na swój kod patrz przez pryzmat jego interfejsu, a nie implementacji.
Jedna z podstawowych zasad to tworzenie obiektów, które zajmują się jednym konkretnym zagadnieniem, czyli przykładowo: walidacja danych, obsługa błędów w formularzu, sam formularz, zapisanie użytkownika - to wszystko to osobne obiekty.

btw: jest 2010 rok, funkcje mysql_*() już naprawdę powinny odejść w niepamięć. PDO może i cudem nie jest, ale w porównaniu do pierwszego... deklasuje.