[skrypt] uniwersalny skrypcik logowania Opcje

[epud]

jako, że zaczynam dopiero zabawe z obiektowym php prosze o ocene skrypciku na ponize, jest to "uniwersalny" skrypcik logowania uzytkownika do systemu... zalozenie bylo takie abym mogl to prosto przystosowac do innych projetkow bez za duzego grzebania w kodzie, dla tego podajemy nazwe tabeli z uzytkownikami, i nazwe pol z loginem i haslem

co myslicie nadaje sie?

a pod wzgledem obiektowym?

[PHP] pobierz, plaintext 
<?php 
class auth{
	var $db = NULL;
	var $table_name;
	var $login_name;
	var $pass_name;
	var $md5;
	var $login;
	var $pass;
	var $dbpass;
	var $errors = array();
	public function __construct($db, $table_name, $login_name, $pass_name, $md5 = false){
		session_start();
		//$_SESSION['isLoggedIn'] = false;
		$this -> db = $db;
		$this -> table_name = $table_name;
		$this -> login_name = $login_name;
		$this -> pass_name = $pass_name;
		$this -> md5 = $md5;
	}
	public function authorize($login, $pass){
		if(empty($login) || empty($pass)){
			array_push($this -> errors, 'login i hasło nie mogą być puste');
			return false;
		}
		$this -> login = $login;
		if($this -> md5){
			$this -> pass = md5($pass);
		}
		else{
			$this -> pass = $pass;
		}
		if($this -> getPass()){
			if($this -> pass == $this -> dbpass){
				$_SESSION['isLoggedIn'] = true;
				$_SESSION['userData'] = $this -> getUserData();
				header("Location: index.php");
			}
			else{
				array_push($this -> errors, 'błędny login lub hasło');
			}
		}
		else{
			array_push($this -> errors, 'błędny login lub hasło');
		}
	}
	private function getPass(){
		$this -> dbpass = $this -> db -> GetOne("SELECT ". $this -> pass_name ." FROM ". $this -> table_name ." WHERE ". $this -> login_name." = '".$this -> login."'");
		if($this -> dbpass){
			return true;
		}
		return false;
	}
	public function isLoggedIn(){
		return $_SESSION['isLoggedIn'];
	}
	private function getUserData(){
		return trim_array($this -> db -> GetRow("SELECT * FROM ".$this -> table_name." WHERE ".$this -> login_name." = '".$this -> login."'"));
	}
}
?>
[PHP] pobierz, plaintext 

sprawdzanie czy zalogowany czy nie..

[PHP] pobierz, plaintext 
<?php 
$auth = new auth($db, PREFIX."users", 'user_name', 'user_pass', true);
/* END AUTH */
 
if($auth -> isLoggedIn()){//jesli zalogowany
	$smarty -> display('index.tpl');
}
else{//jesli nie zalogowany
	$smarty -> display('loginForm.tpl');
}//if($auth -> isLoggedIn())
?>
[PHP] pobierz, plaintext 

wersyfikacjia usera

[PHP] pobierz, plaintext 
<?php
$auth = new auth($db, PREFIX."users", 'user_name', 'user_pass', true);
$auth -> authorize($_POST['login']['user_name'], $_POST['login']['user_pass']);
?>
[PHP] pobierz, plaintext 

z gory dzieki za wszelkie wypowiedzi (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował epud 5.06.2006, 18:03:37

[TomASS]

Bardzo mądrze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) A jak wygląda ta funkcja trim_array?

[epud]

Bardzo mądrze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) A jak wygląda ta funkcja trim_array?

tak o:

[PHP] pobierz, plaintext 
<?php 
function trim_array($array){
	$new = array();
	foreach($array as $a => $row){
		if(is_array($row)){
			$new[$a] =  trim_array($row);
		}
		else{
			$new[$a] = trim($row);
			$new[$a] = str_replace("SELECT", " ", $row);
			$new[$a] = str_replace("INSERT", " ", $row);
			$new[$a] = str_replace("DELETE", " ", $row);
			$new[$a] = str_replace("UPDATE", " ", $row);
		}
	}
	return $new;
}
?>
[PHP] pobierz, plaintext 

hm.. tylko wlasnie chyba stwierdzilem ze nie zadziala jak ktos poscic w teks SeLeCt a baza i tak to zinterpetuje chyba (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

ma ktos jakis pomysł jak to porpawic

a powracajac do klasy dodalem TIMEOUTA po ktorym wylogowuje po czasie bezczynnosc:

[PHP] pobierz, plaintext 
<?php 
/**
 * Unwersalna autrozacja uzytkownia
 * @author Acid
 * @version 1.0
 * 
 */
class auth{
	var $db = NULL;
	var $table_name;
	var $login_name;
	var $pass_name;
	var $md5;
	var $login;
	var $pass;
	var $dbpass;
	var $sessionTime;
	var $errors = array();
	/**
	 * Konstruktor klasy
	 *
	 * @param object $db
	 * @param string $table_name
	 * @param string $login_name
	 * @param string $pass_name
	 * @param bool $md5
	 */
	public function __construct($db, $table_name, $login_name, $pass_name, $sessionTime = '1200',$md5 = false){
		session_start();
		$this -> db = $db;
		$this -> table_name = $table_name;
		$this -> login_name = $login_name;
		$this -> pass_name = $pass_name;
		$this -> sessionTime =$sessionTime;
		$this -> md5 = $md5;
	}
	/**
	 * Autoryzacja
	 *
	 * @param string $login
	 * @param string $pass
	 * @return bool
	 */
	public function authorize($login, $pass){
		if(empty($login) || empty($pass)){
			array_push($this -> errors, 'login i hasło nie mogą być puste');
			return false;
		}
		$this -> login = $login;
		if($this -> md5){
			$this -> pass = md5($pass);
		}
		else{
			$this -> pass = $pass;
		}
		if($this -> getPass()){
			if($this -> pass == $this -> dbpass){
				$_SESSION['isLoggedIn'] = true;
				$_SESSION['beginTime'] = time();
				$_SESSION['userData'] = $this -> getUserData();
				header("Location: index.php");
			}
			else{
				array_push($this -> errors, 'błędny login lub hasło');
			}
		}
		else{
			array_push($this -> errors, 'błędny login lub hasło');
		}
	}
	/**
	 * pobieranie hasła z bazy
	 *
	 * @return bool
	 */
	private function getPass(){
		$this -> dbpass = $this -> db -> GetOne("SELECT ". $this -> pass_name ." FROM ". $this -> table_name ." WHERE ". $this -> login_name." = '".$this -> login."'");
		if($this -> dbpass){
			return true;
		}
		return false;
	}
	/**
	 * Sprawdzanie czy zalogowany
	 *
	 * @return bool
	 */
	public function isLoggedIn(){
		if(isset($_SESSION['isLoggedIn']) && $this -> sessionTimeOut()){
			return true;
		}
		return false;
	}
	/**
	 * Pobieranie danych uzytkownia z bazy
	 *
	 * @return array mixed
	 */
	private function getUserData(){
		return trim_array($this -> db -> GetRow("SELECT * FROM ".$this -> table_name." WHERE ".$this -> login_name." = '".$this -> login."'"));
	}
	/**
	 * Wylogowywanie
	 *
	 */
	public function logOut(){
		session_destroy();
		unset($_SESSION);
		header("Location: index.php");
	}
	private function sessionTimeOut(){
		if($_SESSION['beginTime'] + $this -> sessionTime < time()){
			session_destroy();
			unset($_SESSION);
			array_push($this ->errors, 'przekroczyłeś czas bezczynnosc');
			return false;
		}
		return true;
	}
}
?>
[PHP] pobierz, plaintext