Przekazywanie sesji do obiektu Opcje

[Mefiuu]

Witam. Mam problem. Otóż na jednej stronie użytkownik się loguje. Jeśli się zaloguje poprawnie to $_SESSION['logged'] = 1. Stworzyłem klasę do uploadu plików i chciałem w niej zawrzeć metodę do sprawdzania uprawnień, tzn. czy $_SESSION['logged'] == 1. To właśnie jako parametr przekazuję wartość $_SESSION. Metoda wygląda mniej więcej tak :

[PHP] pobierz, plaintext 
public function checkRights($level) {
		$this->level = $level;
		if ($this->level != 1) return false;
}
 
[PHP] pobierz, plaintext 

przekazuję tak :

[PHP] pobierz, plaintext 
 
if (!$upload->checkRights($_SESSION['logged'])) echo "nieudane logowanie";
 
[PHP] pobierz, plaintext 

I to właśnie nie działa. Wywala mi że nie udało się zalogować, ale wartość wynosi 1. Czemu ?

[Ilware]

robisz negacje z nulla nie wiem czy to coś nie knoci, spróbuj zwrócić true w funkcji

[skowron-line]

[PHP] pobierz, plaintext 
if ($upload->checkRights($_SESSION['logged']) == false) echo "nieudane logowanie";
[PHP] pobierz, plaintext 

[Mefiuu]

[PHP] pobierz, plaintext 
if ($upload->checkRights($_SESSION['logged']) == false) echo "nieudane logowanie";
[PHP] pobierz, plaintext 

niestety, nadal nieudane logowanie. Ale kiedy wyświetlę wartość w metodzie to pokazuje mi 1.

[mortus]

Jak zauważył Ilware, funkcja nie zwraca żadnej wartości w momencie, gdy uprawnienia się zgadzają. Stąd problem. Nie wiem, czy aż tak ciężko zinterpretować zdanie:

robisz negacje z nulla nie wiem czy to coś nie knoci, spróbuj zwrócić true w funkcji

Moja interpretacja:

[PHP] pobierz, plaintext 
public function checkRights($level) {
		$this->level = $level;
		if ($this->level != 1) return false;
                return true;
}
[PHP] pobierz, plaintext 

[Mefiuu]

Jak zauważył Ilware, funkcja nie zwraca żadnej wartości w momencie, gdy uprawnienia się zgadzają. Stąd problem. Nie wiem, czy aż tak ciężko zinterpretować zdanie:
Moja interpretacja:

[PHP] pobierz, plaintext 
public function checkRights($level) {
		$this->level = $level;
		if ($this->level != 1) return false;
                return true;
}
[PHP] pobierz, plaintext 

ech... rzeczywiście, mój błąd, po prostu nie doczytałem postu Ilware'a. Takie rzeczy też się zdarzają. Niemniej jednak kod działa, dziękuję za pomoc.

[mortus]

Gdyby nie fakt, że podczas porównywania za pomocą dwóch znaków równości dochodzi do rzutowania typów zmiennych i NULL zamieniane jest na false, to i kod użytkownika skowron-line by zadziałał. Natomiast można porównać dwie zmienne sprawdzając przy okazji zgodność ich typów (używamy do tego trzech znaków równości) i wówczas osiągamy oczekiwany rezultat.

[PHP] pobierz, plaintext 
    if ($upload->checkRights($_SESSION['logged']) === false) echo "nieudane logowanie";
[PHP] pobierz, plaintext 

Jednak skoro funkcja coś zwraca, to i powinna zwracać jakąś domyślną wartość (w tym przypadku true).

EDIT
Ale to tylko taka istotna ciekawostka.

Ten post edytował mortus 18.08.2011, 21:13:33