[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 

[Ziem]

Raczej nie chodzi o to.
Powinieneś zakupić jakąś książkę omawiającą OOP - na pewno dużo Ci da i odpowie na wiele pytań. Zobaczysz jak to wszystko wygląda (IMG:style_emoticons/default/smile.gif) .

Ten post edytował Ziem 20.09.2010, 09:13:56