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]

5. Znów wszystko jest namieszane jak leci. Musisz przemyśleć dokładnie jak to ma wyglądać.

Nie rozumiem... dlaczego jest namieszane?

Staraj się by jedno zajmowało się jednym, a drugie drugim.

1. HttpRequest - Dodałeś GET i POST, ale co z SERVER, REQUEST, SESSION, COOKIE etc. Poza tym nie lepiej by była to klasa statyczna, bądź Singleton by nie przekazywać non stop obiektu?
2. Klasa DB - Dobry pierwszy krok, ale Singleton z tego marny, właściwie w ogóle go brak (IMG:style_emoticons/default/tongue.gif) getInstance() powinien zwracać Ci obiekt klasy DB, oraz przy okazji klasy PDO.

Przykład:

[PHP] pobierz, plaintext 
 
class DB extends PDO {
 
	static private $instance = null;
 
	public function __construct() {}
 
	static public function getInstance() {
		if(self::$instance === null) {
			self::$instance = new self();
		}
		return self::$instance;
	}
 
	static public function connect($config = array()) {
		parent::__construct($config['dsn'], $config['username'], $config['password'], [...]);
	}
 
}
 
[PHP] pobierz, plaintext 

I wykorzystanie na początku aplikacji?

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

3. Co do klasy User - To tutaj głównie mam zarzuty co do mieszania. Masz tutaj rejestracje, sprawdzanie, pobieranie informacji o użytkowniku oraz zwracanie czy to Admin, czy Moderator. Jeśli mowa tutaj o optymalizacji trzeba też wspomnieć o jakimś ładzie - Osobna klasa do rejestracji użytkownika, sprawdzania czy istnieje i innych. Inna znowu do pobierania danych użytkownika o id / nazwie x, oraz ew. rozszerzenie do klasy pobierającej dane, która ustali czy to Admin/Mod/Użytkownik czy ktokolwiek.

Życzę miłej pracy (IMG:style_emoticons/default/tongue.gif)

[rav1989]

1. HttpRequest - Dodałeś GET i POST, ale co z SERVER, REQUEST, SESSION, COOKIE etc. Poza tym nie lepiej by była to klasa statyczna, bądź Singleton by nie przekazywać non stop obiektu?

Nie mam pojęcia jak to zrobić aby było to Singleton... natomiast dodałem SERVER i COOKIE (jakoś reszty nie potrzebuję) analogicznie jak zrobiłem to z GET i POST

2. Klasa DB - Dobry pierwszy krok, ale Singleton z tego marny, właściwie w ogóle go brak (IMG:style_emoticons/default/tongue.gif) getInstance() powinien zwracać Ci obiekt klasy DB, oraz przy okazji klasy PDO.

Przykład:

[PHP] pobierz, plaintext 
 
class DB extends PDO {
 
	static private $instance = null;
 
	public function __construct() {}
 
	static public function getInstance() {
		if(self::$instance === null) {
			self::$instance = new self();
		}
		return self::$instance;
	}
 
	static public function connect($config = array()) {
		parent::__construct($config['dsn'], $config['username'], $config['password'], [...]);
	}
 
}
 
[PHP] pobierz, plaintext 

I wykorzystanie na początku aplikacji?

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

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...

3. Co do klasy User - To tutaj głównie mam zarzuty co do mieszania. Masz tutaj rejestracje, sprawdzanie, pobieranie informacji o użytkowniku oraz zwracanie czy to Admin, czy Moderator. Jeśli mowa tutaj o optymalizacji trzeba też wspomnieć o jakimś ładzie - Osobna klasa do rejestracji użytkownika, sprawdzania czy istnieje i innych. Inna znowu do pobierania danych użytkownika o id / nazwie x, oraz ew. rozszerzenie do klasy pobierającej dane, która ustali czy to Admin/Mod/Użytkownik czy ktokolwiek.

Zrobiłem osobne klasy do sprawdzenia poziomu (Anonim, User, Admin)

[PHP] pobierz, plaintext 
class CheckMode
{
	static public function isAnonymous(User $user)
	{
		return ($user -> getMode() == MODE_GUEST ? true : false);
	}
 
	static public function isAdmin(User $user)
	{
		return ($user -> getMode() == MODE_ADMIN ? true : false);
	}
 
	static public function isUser(User $user)
	{
		return ($user -> getMode() == MODE_USER ? true : false);
	}
}
[PHP] pobierz, plaintext 

Zrobiłem też osobne klasy do rejestracji oraz sprawdzania Usera (IMG:style_emoticons/default/biggrin.gif)

[PHP] pobierz, plaintext 
class CheckUser
{
	static public function run(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;
		}
	}
}
 
class AddUser
{
	static public function run(httpRequest $req, DB $pdo)
	{
		$result = User::checkUser($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 

Pozdrawiam
Rav