[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 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ę

[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 

[yevaud]

podejscie z adduser jest dla mnie troche dziwne. Moim zdaniem powinno byc jakies "save" ktore zapamietywaloby identyfikator uzytkownika, albo w ogole powinienes oddzielic warstwe z tym userem, od warstwy zapisujacej dane do bazy

z save wygladaloby to jakos tak

[PHP] pobierz, plaintext 
$user = new new_user();
$user->jakiesdane = $dane;
try {
  $user->save();
  // dodamy mu jeszzce jakies dane
  $user->innedane = $dane;
  $user->save();
} (..)
[PHP] pobierz, plaintext 

generalnie logiczniej bedzie jesli obiekt uzytkownika bedzie reprezentowal jednego konkretnego uzytkownika. Jesli chcesz masowo dodawac uzytkownikow to moim zdaniem powinienes zrobic inna klase - fabryke - ktora bedzie tworzyla obiekty klasy user

[PHP] pobierz, plaintext 
$fabryka = new fabryka();
$user = $fabryka->nowyuser();
$user->jakiesdane = $dane;
$user->save(); // insert
 
$user2 = $fabryka->nowyuser();
$user2->jakiesdane = $jakiesdane dla innego usera;
$user2->save(); //insert
 
$user3 = $fabryka->ladujusera($id);
$user3->jakiesdane = $jakiesdane dla innego usera;
$user3->save(); //update
 
[PHP] pobierz, plaintext 

moim zdaniem cos w ten desen, ale sie opisalem..

Ten post edytował yevaud 16.09.2010, 19:46:54