Optymalizacja klasy do obsługi użytkownika Opcje

[rav1989]

Witam

Napisałem klasę która ma obsługiwać użytkownika wydaje mi się, że jest ona nieoptymalna... wstawiam ją aby ktoś z większym doświadczeniem ją obejrzał i mnie nakierował co można zrobić lepiej i co mam poprawić (IMG:style_emoticons/default/biggrin.gif)

Klasa:

[PHP] pobierz, plaintext 
session_start();
 
define("MySql_Host", "localhost");
define("MySql_Login", "root");
define("MySql_Password", "");
define("MySql_DbName", "test");
define("MySql_TPrefix", "");
 
define("MODE_USER",0);
define("MODE_MOD",1);
define("MODE_ADMIN",2);
 
function removeSlashes(&$value)
		{
             if(is_array($value))
            {
                return array_map('removeSlashes', $value);
            }
            else
            {
                 return stripslashes($value);
            }
		}
 
class User{
private $id;
private $email;
private $haslo;
 
public $nazwa;
public $opis;
public $avatar;
public $mode;
public $urls;
private $sesja;
 
private $pdo;
 
public function __construct($email_ = null, $haslo_ = null){
$this->pdo = new PDO('mysql:host='.MySql_Host.';dbname='.MySql_DbName, MySql_Login , MySql_Password);
$this->pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
if(isset($email_) and isset($haslo_)){
$this->email = $email_;
$this->haslo = md5(sha1($haslo_).sha1('TuJestSol'));
}elseif(isset($_SESSION['token'])){
$this->sesja = $_SESSION['token'];
$this->Load();
}else die('Ups! Odwiedź rejestrację!');
}
 
private function mysql_insert($table, $inserts) {
    $values = array_map("removeSlashes", array_values($inserts));
    $keys = array_keys($inserts);
 
	$query = 'INSERT INTO `'.MySql_TPrefix.$table.'` (`'.implode('`,`', $keys).'`) VALUES (\''.implode('\',\'', $values).'\')';
	try{
    return $this->pdo -> exec($query);
	}
	catch(PDOException $e)
        {
                echo 'Wystąpił błąd biblioteki PDO: ' . $e->getMessage();
				return false;
        }
}
 
public function Rejestruj($nazwa_){
$this->nazwa = $nazwa_;
$this->mode = MODE_USER;
      $stmt = $this->pdo -> query('SELECT * FROM `users` WHERE `haslo`="'.$this->haslo.'" and `email`="'.$this->email.'"');
	  if($stmt->fetch()) return false;
$tmp = $this->mysql_insert('users',array('nazwa'=>$this->nazwa, 'haslo'=>$this->haslo, 'email'=>$this->email, 'mode'=>$this->mode));
 
	if($tmp)
		return true;
		return false;
}
 
public function Loguj(){
session_start();
   try
   {
		$_SESSION['token'] = md5(rand());
		$this->sesja = $_SESSION['token'];
      $this->pdo -> query('UPDATE `users` SET `sesja`="'.$_SESSION['token'].'" WHERE `haslo`="'.$this->haslo.'" and `email`="'.$this->email.'"');
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
	  return null;
   }
}
public function Wyloguj(){
   try
   {
      $this->pdo -> query('UPDATE `users` SET `sesja`="NULL" WHERE `sesja`="'.$this->sesja.'"');
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
	  return null;
   }
   session_destroy();
}
 
public function __set($name,$value){
$this->$name($value);
}
private function Load(){
   try
   {
      $stmt = $this->pdo -> query('SELECT * FROM `users` WHERE `sesja`="'.$this->sesja.'"');
	  $row = $stmt->fetch();
      $stmt -> closeCursor(); 
	  $urls = $this->pdo -> query('SELECT url FROM `user_sites` WHERE `users_id`="'.$row['id'].'"'); 
	  foreach($urls as $url){
	  $this->urls[] = base64_decode($url['url']);
	  }
	  $urls -> closeCursor();
	  $this->opis = $row['opis'];
	  $this->avatar = $row['avatar'];
	  $this->nazwa = $row['nazwa'];
	  $this->mode = $row['mode'];
	  $this->id = $row['id'];
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
	  return null;
   }
}
 
public function AddUrl($url){
$url = base64_encode($url);
$stmt = $this->pdo -> query('SELECT * FROM `user_sites` WHERE `url`="'.$url.'" and `users_id`="'.$this->id.'"'); 
if($stmt->fetch()) return false;
$tmp = $this->mysql_insert('user_sites',array('url'=>$url, 'users_id'=>$this->id));
$this->urls[] = base64_decode($url);
 
	if($tmp)
		return true;
		return false;
}
 
public function Opis($value){
$this->opis = $value;
   try
   {
      $this->pdo -> query('UPDATE `users` SET `opis`="'.$this->opis.'" WHERE `sesja`="'.$this->sesja.'"');
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
	  return null;
   }
}
 
public function Avatar($value){
$this->avatar = $value;
   try
   {
      $this->pdo -> query('UPDATE `users` SET `avatar`="'.$this->avatar.'" WHERE `sesja`="'.$this->sesja.'"');
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
	  return null;
   }
}
 
public function Mode($value){
$this->mode = $value;
   try
   {
      $this->pdo -> query('UPDATE `users` SET `mode`="'.$this->mode.'" WHERE `sesja`="'.$this->sesja.'"');
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
	  return null;
   }
}
 
 
 
}
[PHP] pobierz, plaintext 

Logowanie:

[PHP] pobierz, plaintext 
require_once('class/user.class.php');
$user = new User('test','test');
$user -> Loguj();
[PHP] pobierz, plaintext 

Update/Odczyt danych:

[PHP] pobierz, plaintext 
require_once('class/user.class.php');
$user = new User();
$user->Opis = 'adad';
$user->AddUrl ='http://wp.pl';
echo $user -> opis;
print_r($user -> urls);
[PHP] pobierz, plaintext 

Wylogowanie:

[PHP] pobierz, plaintext 
require_once('class/user.class.php');
$user = new User();
$user->Wyloguj();
[PHP] pobierz, plaintext 

Struktura SQL:

[SQL] pobierz, plaintext 
CREATE  TABLE IF NOT EXISTS `users` (
  `id` INT NOT NULL AUTO_INCREMENT ,
  `nazwa` VARCHAR(45) NULL ,
  `email` VARCHAR(45) NULL ,
  `haslo` VARCHAR(45) NULL ,
  `opis` TEXT NULL ,
  `avatar` VARCHAR(128) NULL ,
  `mode` INT NULL ,
  `sesja` VARCHAR(45) NULL ,
  PRIMARY KEY (`id`) )
ENGINE = MyISAM
DEFAULT CHARACTER SET = utf8
COLLATE = utf8_general_ci
PACK_KEYS = DEFAULT;
[SQL] pobierz, plaintext 

Pozdrawiam
Rav