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

Może opisz co dokładnie się dzieje...

A wysyłasz jakieś dane w ogóle ?

No się loguję za pomocą formularza no i metoda autoryzacja() zwraca mi false mimo że takie dane mam w mysql i z tym skryptem napisanym strukturalnie all było git, ale teraz uczę się OOP i chciałem go napisać obiektowo.

PS: Znacie jakiś dobry kurs OOP ale w PHP5, bo w google żadnego porządnego nie mogą znaleść a ten na wortalu jakoś mi nie podchodzi....

Ja się tak zapytam, masz dwie metody, sprawdzanie i autoryzacja. Gdzie one są ze sobą powiązane? W skrypcie gdzieś dalej? Po drugie jeśli najpierw sprawdzasz dane, metodą sprawdzanie to ona powinna coś zwracać bo inaczej nie przekażesz żadnych danych do autoryzacji.

Przynajmniej moim zdaniem nie znam kursu może coś innego tam wymodzili.

Ale przecież metoda sprawdzenie() przekazuje do właściwości klasy $metod odpowiednią wartość. I wtedy metoda autoryzacja() wczytuje wartość $metod. Czy takie coś jest błędem?

Gdzie przekazuje? Bo jeśli w klasie to albo jestem ślepy albo tego nie widzę

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.

No to wymodziłem coś takiego:

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

No i teraz passer wywala mi to:


Co napisałem źle

A jak wywołujesz tą metodę autoryzacja?

[PHP] pobierz, plaintext 
<?php
$logowanie = new logowanie ;
$logowanie->autoryzacja($_POST['user'],$_POST['haslo'], $_POST['zapamietaj']);
?>
[PHP] pobierz, plaintext 

A zrób przed tym var_dump($_POST) i zobacz, czy na pewno są jakieś dane w $_POST.

Parser dokładnie Ci powiedział co jest nie tak. Brakuje Ci danych przekazywanych do tej autoryzacji, więc pewnie wywołujesz ją w momencie gdy tablice $_POST masz pustą, np. przed wysłaniem formularza..

Zrób sobie coś na podobieństwo tego:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST[&apos;form_check&apos;]){
//utowrzenie obiektu klasy i wywołanie autoryzacji
} else {
//wyswietlenie formularza i dodaj w nim pole :
// 
}
?>
[PHP] pobierz, plaintext 

To powinno w jakiś sposób pomóc jak zrozumiesz o co chodzi.

Po drugie nie bardzo rozumiesz zasadę obiektowego programowania. Poszukaj na google.pl haseł "programowanie obiektowe php5" parę pozycji jest, z którymi warto się zapoznać na początek.

Dobrym źródłem jest też PHPEdia.. prosto opisane co i jak. Może nie wyszystko ale podstawy tam są.
http://www.phpedia.pl/wiki/Klasa

Lub wikibooks : http://pl.wikibooks.org/wiki/PHP/Klasy_i_obiekty