[OOP] problemy początkującego - rejestracja Opcje

[szczypior]

Witam

Postanowiłem spróbować swoich sił w OOP, na przykładzie skryptu rejestracji użytkowników. Napisałem coś takiego i o dziwo działa (IMG:style_emoticons/default/winksmiley.jpg) Gdy dane są poprawne dodaje użytkownika, gdy nie to nie dodaje, ale za to również nie wyświetla komunikatu. Moja prośba do Was, doświadczonych w temacie osób, to zobaczenie czy ten kod jest napisany poprawnie (zgodnie ze sztuką), co można w nim ulepszyć i jak najlepiej rozwiązać wyświetlanie komunikatów, które w tej formie nie działają.

Za wszelkie opinie, porady serdecznie dziękuję(IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
class new_user
{
	public $login;
	public $pass;
	public $re_pass;
	public $email;
 
	function validate($dane)
	{
		$dane = htmlspecialchars($dane, ENT_QUOTES);
		$dane = mysql_real_escape_string($dane);
    		if (!get_magic_quotes_gpc())
    		{
			$dane = addslashes ($dane);
    		}
        	return $dane;
	}
 
	function check_login ()
	{
		if ($this->login==$this->validate($this->login))
		{
			$query = mysql_query('SELECT id FROM test WHERE login = "'.$this->login.'"') or die (mysql_error()); 
 			if (mysql_num_rows($query) != 0)
			{
 				// login zajęty
				$komunikat = 'Wybrany login jest zajęty';
				return false;
			}
			else
			{
 				// login wolny
				return $this->login;
			}
		}
		else
		{
			$komunikat = 'Login zawiera niedozwolone znaki';
			return false;
		}
	}
 
	function check_pass ()
	{
		if ($this->pass==$this->validate($this->pass))
		{
			if ($this->pass==$this->re_pass)
			{
 				if (strlen($this->pass)>5)
				{
					return md5($this->pass);
				}
				else 
				{
					$komunikat = 'Podane hasło jest za krótkie (min. 6 znaków)';
					return false;
				}
			}
			else
			{
				$komunikat = 'Podane hasła są różne';
				return false;
			}
		}
		else
		{
			$komunikat = 'Hasło zawiera niedozwolone znaki';
			return false;
		}
 
	}
 
	function check_email ()
	{
		if (ereg("^[a-z0-9_\\.-]+@([a-z0-9_-]+\\.)+[a-z]{2,}$", $this->email))
		{
			return $this->email;
		}
		else
		{
			$komunikat = 'Podaj poprawny adres e-mail';
			return false;
		}
	}
 
	function add_user ()
	{
		if($this->check_login() && $this->check_pass() && $this->check_email())
		{
			mysql_query("INSERT INTO test(login,pass,email) VALUES('".$this->check_login()."','".$this->check_pass()."','".$this->check_email()."')");
			if (mysql_affected_rows())
			{
				return true;
			}
			else
			{
				return false;
			}
		}
	}
 
}
 
$new = new new_user;
$new->login = 'ssssss';
$new->pass = 'aaaaaaaaa';
$new->re_pass = 'aaaaaaaaa';
$new->email = 'ttttt@gggggg.pl';
$nowy = $new->add_user($new);
echo $komunikat;
[PHP] pobierz, plaintext 

[Maciekbjw]

Kolego według mnie nie potrzebnie pakujesz całą funkcjonalność do jednej klasy.
Model MVC się kłania.

Zrób osobno akcje, osobną klasę do walidacji danych, osobną do zapisu usera do bazy (tam metody zapisujace, a takze sprawdzajace dostepnosc jakiegos loginu). W ten sposób będzie to miało ręce i nogi (IMG:style_emoticons/default/smile.gif) bo to co przedstawiles to faktycznie klasy ale nijak ma sie to do sensu programowania obiektowego (IMG:style_emoticons/default/smile.gif)

[szczypior]

Kolego według mnie nie potrzebnie pakujesz całą funkcjonalność do jednej klasy.
Model MVC się kłania.

Zrób osobno akcje, osobną klasę do walidacji danych, osobną do zapisu usera do bazy (tam metody zapisujace, a takze sprawdzajace dostepnosc jakiegos loginu). W ten sposób będzie to miało ręce i nogi (IMG:style_emoticons/default/smile.gif) bo to co przedstawiles to faktycznie klasy ale nijak ma sie to do sensu programowania obiektowego (IMG:style_emoticons/default/smile.gif)

Tzn mówisz, żeby zrobić osobno klasę do walidacji i zapisu, następnie wywoływać je po kolei?

[PHP] pobierz, plaintext 
$check = new validate;
$check ->login = 'ssss#%@#&;"sg/s0aa';
$check ->pass = 'aaaaaaaaa';
$check ->re_pass = 'aaaaaaaaa';
$check ->email = 'tttttg@ggggg.pl';
 
$new = new user($pdo);
$new->login = $check ->login;
$new->pass = $check ->pass;
$new->email = $check ->email;
$new->save();
[PHP] pobierz, plaintext 

Tak miałoby wyglądać ich wywołanie?