[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 

[szczypior]

z jakiego powodu nie uzywasz PDO ?

Gdyż to tylko testowy kod, szybciej pisze się bez PDO. Nacodzień jednak korzystam z dobrodziejstw PDO.

podejscie z adduser jest dla mnie troche dziwne

Nie bardzo rozumiem jak mam oddzielić warstwę z userem od zapisywania? Rozumiem, że proponujesz, aby w inny sposób przekazać zvalidowane dane do zapytania SQL, tak? Masz jakąś konkretną propozycję?

Napisany przeze mnie kod miałby służyć do rejestracji użytkowników w serwisie poprzez formularz i miałby operować na otrzymanych danych, więc nie będzie to na masową skalę. Nie mniej jednak nie bardzo rozumiem idee "fabryki". Co ona powinna zawiera/robić?

Wyświetlanie komunikatów udało mi się rozwiązać po wskazówkach Nospor'a:) Czy ten sposób jest OK? Czy jest lepsze rozwiązanie?

Natomiast nadal nie bardzo wiem, jak udoskonalić resztę kodu. Bardzo proszę o porady, pomoc. Teraz wygląda on tak:

[PHP] pobierz, plaintext 
 
class new_user
{
    public $login;
    public $pass;
    public $re_pass;
    public $email;
    public $status;
 
    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
                $this->status = 'Wybrany login jest zajęty';
                return false;
            }
            else
            {
                 // login wolny
                return $this->login;
            }
        }
        else
        {
            $this->status = '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 
                {
                    $this->status = 'Podane hasło jest za krótkie (min. 6 znaków)';
                    return false;
                }
            }
            else
            {
                $this->status = 'Podane hasła są różne';
                return false;
            }
        }
        else
        {
            $this->status = '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
        {
            $this->status = '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())
            {
                $this->status = 'Użytkownik dodany poprawnie.';
                return true;
            }
            else
            {
                $this->status = 'Wystąpił błąd podczas dodawania użytkownika.';
                return false;
            }
        }
    }
 
}
 
$new = new new_user;
$new->login = 'sssssgs';
$new->pass = 'aaaaaaaaa';
$new->re_pass = 'aaaaaaaaa';
$new->email = 'tttttg@ggggg.pl';
$nowy = $new->add_user($new);
echo $new->status;
 
[PHP] pobierz, plaintext