[php] klasa dodawająca użytkowników + wysyłanie maila, pobranie ID Opcje

[Lejto]

Mam problem z klasą

Tak wykonuje rejestracje:

[PHP] pobierz, plaintext 
            $u->username = htmlspecialchars($_POST['username']);
            $u->password = htmlspecialchars($_POST['password1']);
            $u->emailAddr = htmlspecialchars($_POST['mail']);
	    $u->imie = htmlspecialchars($_POST['imie']);
   	    $u->nazwisko = htmlspecialchars($_POST['nazwisko']);
            $token = $u->setInactive();
            echo  '<p><strong>Dziękujemy za ' .
                'zarejestrowanie się.</strong></p><p>Należy pamiętać o zweryfikowaniu konta i kliknąć w łącze załączone w mailu który sotał wysłany';
[PHP] pobierz, plaintext 

Dodaje pomyślnie dane do bazy.
Gorzej jest z pobraniem id zarejestrowanego użytkownika. (jak wysyłam maila to potrzebuje te dane)
pobieram tak:

[PHP] pobierz, plaintext 
$user->userId;
[PHP] pobierz, plaintext 

nic nie zwraca

Klasa:

[PHP] pobierz, plaintext 
<?
class User extends functions
{ 
	public $uid;
	public $fields = array();	
 	protected $sql;
	public $userId;
 
	public function __construct($sql)
	{
    	$this->sql = $sql;
		$this->uid = null;
		$this->fields = array('username' => '',
							  'password' => '',
							  'emailAddr' => '',
							  'gg' => '',
							  'imie' => '',
							  'nazwisko' => '',
							  'isActive' => false);
 	}
	public function __get($field)
	{
		if($field == 'userId')
		{
			return $this->uid;
		}
		else
		{
			return $this->fields[$field];
		}
	}
	public function __set($field, $value)
	{
		if(array_key_exists($field, $this->fields))
		{
			$this->fields[$field] = $value;
		}
	}
	public  function validateUserName($username)
	{
  		if(strlen($username)<3)
		{
			echo 'Nazwa użytkownika musi zawierać co najmniej 3 znaki.';
		}
		else
		{
			return true;
		}
		if(strlen($username)>20)
		{
			echo 'Za długa nazwa użytkownika max 20 znaków.';
		}
		else
		{
			return true;
 		}
 	}
	public   function validateEmailAddr($email)
	{
		return filter_var($email, FILTER_VALIDATE_EMAIL);
		if(false)
		{
			echo 'Niepoprawny adres e-mail';
		}
	}
	public function getById($uid)
	{ 	
 		$db = $this->sql;
 
		$u = new User($db);
 		$sql =  $db->query('select * from users where user_id = "'.$uid.'"');
 
		if($sql->num_rows)
		{			 
			$row = $sql->fetch_array();
			$u->username = $row['username'];
			$u->password = $row['password'];
			$u->emailAddr = $row['email_addr'];
			$u->isActive = $row['is_active'];
			$u->imie = $row['imie'];
			$u->nazwisko = $row['nazwisko'];
			$u->uid = $uid;
		}
		return $u;
	}
 
	public function getByUsername($username)
	{ 
		$db = $this->sql;
   		$u = new User($db);
 		$sql = $db->query('select * from users where username = "'.$username.'" and is_active = 1');
 
		if($sql->num_rows)
		{
			$row = $sql->fetch_array();
  			$u->password = $row['password'];
			$u->username = $row['username'];
			$u->emailAddr = $row['email_addr'];
			$u->isActive = $row['is_active'];
			$u->imie = $row['imie'];
			$u->nazwisko = $row['nazwisko'];
			$u->uid = $row['user_id'];
		}
		return $u;
 	}
 
	public function save()
	{
 
 
 		$db = $this->sql;
		$data = date('d.m.y H:i:s');
 
 		if($this->uid)
		{
			$sql = $db->query('update USERS set username = "'.$this->username.'", password =  "'.$this->password.'",
							  email_addr = "'.$this->emailAddr.'", is_active = "'.$this->isActive.'" where user_id = "'.$this->userId.'"');
 
 
 		}
		else
		{
			 $sql = $db->query('insert into users (username, password, email_addr, is_active, gg,imie,nazwisko,data)
												  values ("'.$this->username.'", "'.sha1($this->password).'", "'.$this->emailAddr.'", "0",
														  "'.htmlspecialchars($_POST['gg']).'","'.$this->imie.'","'.$this->nazwisko.'","'.$data.'")');
			 if(!$sql)
			 {
				 var_dump($this->password);
			 }
 		}
	}
 
	public function setInactive()
	{
		$db = $this->sql;
		$this->isActive = false;
		$this->save();
 
		$token = $this->random_text(5);
		$sql = $db->query('insert into users_pending  (user_id, token) values ("'.$this->uid.'", "'.$token.'")');
 
		return $token;
	} 
 
	public function setActive($token)
	{
		$db = $this->sql;
 
		$sql = $db->query('select token from users_pending where user_id = "'.$this->uid.'" and token = "'.$token.'"');
 
		if(!$sql->num_rows)
		{
			return false;
		}
		else
		{
			$sql = $db->query('delete from users_pending where user_id = "'.$this->uid.'" and token = "'.$this->token.'"');
			$this->isAdcive = true;
			$this->save();
			return true;
		}
	}
}
[PHP] pobierz, plaintext 

[Crozin]

Co robi obiekt klasy User?

1) Reprezentuje jeden konkretny rekord z bazy danych
2) Sprawdza poprawność danych wprowadzonych w formularzu
3) Wyświetla sformatowaną odpowiedź
4) Pobiera inne obiekty klasy User z bazy danych
5) Generuje losowe tokeny
6) Przetwarza formularz
7) Wyświetla formularz

A zgodnie z ideą programowania obiektowego: jeden obiekt - jedno zadanie.

Poza tym:
1) W User::__get() masz tzw. dead-code - czyli fragment kodu, który się nigdy nie wykona (chodzi o ten drugi IF z showUserId)
2) Brak jakiegokolwiek powiadamiania czy coś co zostało wykonane (nie) wykonało się poprawnie
3) Wielokrotnie powtarzasz dokładnie ten sam kod różniący się przykładowo jednym waruniem. Google: DRY reguła, Google: KISS reguła
4) htmlspecialchars używa się przed wyświetleniem danych w dokumentach XML/HTML i im podobnych

Tak więc:
1) Poczytaj co nieco teorii na temat programowania obiektowego - bo brniesz w coś czego nie da się sklasyfikować jako OOP - co więcej... to jedynie utrudnia Ci życie
2) Podziel sobie różne zadania na różne obiekty
3) Zaplanuj jak owe obiekty będą ze sobą współpracować
4) Dopiero wtedy bierz się za pisanie