[PHP5][OOP] Problem ze skryptem Opcje

[qba10]

Witam

Na podstawie http://www.strefaphp.net/artykul36.html chciałem napisać ten skrypt w OOP, ale chyba coś nie wyszło....
Tu moja prośba by sprawdzić mój skrypt, czy ma identyczne działanie jak ten z linku...

Oto skrypt (połączenie z bazą danych jest wykonywane przed wykonaniem tego obiektu):

[PHP] pobierz, plaintext 
<?php
class logowanie
 {
  public $login;
  public $haslo;
  public $zapamietaj; 
  public  $login_use;
  public  $haslo_use;
  public $zalogowany;
  public $metod;
  public $dane;
  public $kxc = "a523d703f908dd69e02db64ec35d07315f4c49ab3cacad14a97304b02f2fbb09";   
 
 
	   
 private function sprawdzanie()
 {
 
if(isset($_COOKIE['dane'])) 
{
$this->dane = $_COOKIE["dane"]^$this->kxc;
$this->login_use = substr($dane, 0, 32);
$this->haslo_use = substr($dane, -32);
$this->metod = "cookies";
}
elseif(isset($_SESSION['login']) AND isset($_SESSION['haslo'])) 
{
$this->login_use = $_SESSION['login'];
$this->haslo_use = $_SESSION['haslo'];
$this->metod = "session";
}
elseif(isset($this->login) AND isset($this->haslo)) 
{
$this->login_use = md5(strtolower($this->login));
$this->haslo_use = md5($this->haslo);
$this->metod = "post";
} 
 
	   
 
 }	  
 public function autoryzacja($login, $haslo, $zapamietaj )
  {
 
 $this->login = $login ;
 $this->haslo = $haslo;
 $this->zapamietaj = $zapamietaj;
 
 
   $this->zalogowany = false ;
	   if(isset($this->metod)) 
	   {
	   $query = mysql_query("SELECT * FROM `users` WHERE `user`='".$this->login_use."' AND `pass`='".$this->haslo_use."';"); 
		   if(mysql_num_rows($query) == 1) 
		   {
 
		   $wiersz = mysql_fetch_array($query);
		   $_SESSION['login'] = $wiersz['user'];
		   $_SESSION['haslo'] = $wiersz['pass'];
		   $_SESSION['nick'] = $wiersz['nick']; 
		   $this->zalogowany = true;
				 if($this->metod == "post" AND isset( $this->zapamietaj) AND  $this->zapamietaj == "ok") 
				 {
				 $dane = ($login.$haslo)^$_CONFIG['kxc'];
				 setcookie ("dane", $dane,time()+3600*24*365);
				 }
 
	return $this->zalogowany ;	   
		   }
	   }
	   
  }
  }
?>
[PHP] pobierz, plaintext 

PS: Żadnego błędu mi nie wywala ale, logowanie nie chce działać...

Pozdrawiam
Qba10

Ten post edytował qba10 19.06.2008, 20:52:25

[nithajasz]

Gdzie przekazuje? Bo jeśli w klasie to albo jestem ślepy albo tego nie widzę (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Edit: Spojrzałem na ten tutek co podałeś. Tam masz zupełnie coś innego niż chcesz a praktycznie przepisujesz kod. Tam masz jedną funkcję w której masz to wszystko, co ty podzieliłeś na dwie metody. Jeśli chcesz by to działało to zwracaj coś z metody sprawdzanie, ten login, hasło i co tam masz i wywołuj metodę autoryzacja z tymi danymi.

Bo tak to trudno by zwróciła Ci coś innego jeśli nie podajesz jej żadnych danych do sprawdzenia, zawsze zwróci FALSE.

Przykład taki najprostszy:

[PHP] pobierz, plaintext 
<?php
class prosta_klasa{
 
public $zalogowany = 0;
 
public function __construct($login, $password)
{
	if($this -> sprawdz($login, $password)) $this -> loguj();
 
}
 
public function loguj()
{
	 // ustawiasz parametr, który świadczy o zalogowaniu uzytkownika
	$this -> zalogowany = 1;
	// lub 
	$_SESSION['logged'] = 1;
}
 
public function sprawdz($login, $password)
{
	// tu sprawdzasz co tam chcesz np. walidujesz zmienne, sprawdzasz czy uzytkownik i
stnieje w bazie
		// jesli dane sie zgadzaja to 
		return TRUE;
		// jesli nie to
		return FALSE;
}
 
}
?>
[PHP] pobierz, plaintext 

Wtedy dane z formularza przekazujesz jako parametr przy tworzeniu obiektu klasy

[PHP] pobierz, plaintext 
<?php
$user = new prosta_klasa($login, $password);
?>
[PHP] pobierz, plaintext 

Jakoś tak, pisane tak z palca więc nie gwarantuje, że dobrze. Chodzi o ogólną zasadę tego.

Jeszcze raz w funkcji którą przerabiasz te dane są przekazywane po przez utworzenie zmiennej. W metodach te dane musisz zwrócić i przekazać do innej metody. Tak jakbyś chciał użyć tej samej zmiennej w dwóch różnych funkcjach.

Ten post edytował nithajasz 19.06.2008, 22:58:35