[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 

[slimy]

A spróbuj wyswietlic całą zawartosc obiektu

[PHP] pobierz, plaintext 
print_r($user)
[PHP] pobierz, plaintext 

i zobacz czy są potrzebne dane.

[PHP] pobierz, plaintext 
public function getByUsername($username)
	{ 
		$db = $this->sql;
   		$u = new User($db);
 		$sql = $db->query('select * from users where username = "'.$username.'" and is_active = 0');
 
		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->uid = $row['user_id'];
		}
		return $u;
 	}
[PHP] pobierz, plaintext 

Czemu pracując na obiekcie w jego metodzie tworzysz nowy obiekt tego samego typu? Czy nie lepiej zainicjowac zmienne w bieżącym obiekcie? Mozliwe, ze to powoduje blad. W sensie czy wywloujesz:

[PHP] pobierz, plaintext 
$user->getByUsername('user');
print $user->uid;
[PHP] pobierz, plaintext 

a funkcja nie inicjalizuje zmiennych w tym obiekcie tylko tworzy nowy. (IMG:style_emoticons/default/winksmiley.jpg) Stosujac Twoj kod powinno byc:

[PHP] pobierz, plaintext 
$user = $user->getByUsername('user');
print $user->uid;
[PHP] pobierz, plaintext 

ale ja bym zmienil ta funkcje by nie tworzyla nowego obiektu - chyba, ze jest to akurat celowe (IMG:style_emoticons/default/winksmiley.jpg)

Ten post edytował slimy 18.04.2010, 22:53:08