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

[Dipter]

Hmm...

Kod

Fatal error: Non-static method PDO::__construct() cannot be called statically in C:\wamp\www\class\db.class.php on line 33

nie wiem co poszło nie tak...

Błąd jest chyba logicznie prosty (Niestety mój (IMG:style_emoticons/default/tongue.gif) ) - Metoda connect() nie może być w tym przypadku statyczna ->

[PHP] pobierz, plaintext 
 
 
DB::getInstance()->connect([...]);
DB::getInstance()->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
[PHP] pobierz, plaintext 

Klasa CheckUser i AddUser jest na razie bezsensu napisana, dlaczego? Przypatrz się uważnie - Skoro robisz jedną metodę w klasie to oznacza to, że klasa to tylko opakowanie na funkcję (IMG:style_emoticons/default/tongue.gif)

Natomiast co do klasy CheckMode, ja nie mam żadnych zastrzeżeń (przynajmniej jakichś większych).

BTW:
Dlaczego w klasie DB nie przypiszesz takich stałych jak prefix?

pozdrawiam.

[rav1989]

Błąd jest chyba logicznie prosty (Niestety mój (IMG:style_emoticons/default/tongue.gif) ) - Metoda connect() nie może być w tym przypadku statyczna ->

[PHP] pobierz, plaintext 
 
 
DB::getInstance()->connect([...]);
DB::getInstance()->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
[PHP] pobierz, plaintext 

Błąd jest nadal czepia się tej linii:

[PHP] pobierz, plaintext 
parent::__construct($config['dsn'], $config['username'], $config['password'], $config['driver_options']);
[PHP] pobierz, plaintext 

prawdopodobnie chodzi mu o ten zapis parent::__construct(...); bo to wskazuje, że konstruktor rodzica jest statyczny...

Klasa CheckUser i AddUser jest na razie bezsensu napisana, dlaczego? Przypatrz się uważnie - Skoro robisz jedną metodę w klasie to oznacza to, że klasa to tylko opakowanie na funkcję (IMG:style_emoticons/default/tongue.gif)

Hmm faktycznie, a teraz:

[PHP] pobierz, plaintext 
class ValidateUser
{
	static public function isExist(httpRequest $req, DB $pdo)
	{
		$stmt = $pdo -> prepare('SELECT id, email, name, password, description, avatar, mode FROM '.MySql_TPrefix.'users WHERE email=:email AND password=:password');
		$stmt -> bindValue(':email', $req->getPost('email'), PDO::PARAM_STR);
		$stmt -> bindValue(':password', sha1(md5('@TuJestBardzo_').md5($req->getPost('haslo')).md5('_TrudnaSol@')), PDO::PARAM_STR);
		$stmt -> execute();
		$stmt -> setFetchMode(PDO::FETCH_CLASS, 'User', array(0 => $pdo, 1=>false));
		if($user = $stmt -> fetch())
		{
			$stmt -> closeCursor();
			return $user;
		}
		else
		{
			$stmt -> closeCursor();
			return null;
		}
	}
	static public function add(httpRequest $req, DB $pdo)
	{
		$result = self::isExist($req,$pdo);
		if($result  instanceof user){ return false; }
		$stmt = $pdo -> prepare('INSERT INTO '.MySql_TPrefix.'users (email, password, mode, name) VALUES (:email, :password, :mode, :name)');
		$stmt -> bindValue(':email', $req->getPost('email'), PDO::PARAM_STR);
		$stmt -> bindValue(':password', sha1(md5('@TuJestBardzo_').md5($req->getPost('haslo')).md5('_TrudnaSol@')), PDO::PARAM_STR);
		$stmt -> bindValue(':mode', MODE_USER, PDO::PARAM_INT);
		$stmt -> bindValue(':name', $req->getPost('nazwa'), PDO::PARAM_STR);
		$ile = $stmt -> execute();
		if($ile > 0)
		{
			return true;
		}
		else
		{
			return false;
		}
	}
}
[PHP] pobierz, plaintext 

BTW:
Dlaczego w klasie DB nie przypiszesz takich stałych jak prefix?

No wiesz, jak będę przenosił stronę na inny serwer to prefix się może zmienić i wtedy będę musiał szukać go po klasie a tak mam w jednym miejscu na widoku razem z innymi danymi (IMG:style_emoticons/default/biggrin.gif) (Jest mniejsze prawdopodobieństwo, że o nim zapomnę).

Pozdrawiam
Rav