[PHP]Funkcja "wyloguj" nie działa. Opcje

[jerry1984]

Witam

Próbowałem zmodyfikować książkowy skrypt wylogowania się, lecz po wciśnięciu przycisku "Wyloguj" nadal jestem zalogowany.
Skrypt ogółem działa, zasada wykrywania czy jest zalogowany użytkownik czy odwiedził nas "Gość".
Oto skrypt z którym się borykam:

[PHP] pobierz, plaintext 
<?php
require_once './user/conn.inc.php';
 
function redirect($url) {
	if (!headers_sent()) {
	header('Location: http://'	. $SERVER['HTTP_HOST'] .
			dirname($_SERVER['PHP_SELF']) . '/' . $url);
	} else {
		die ('Nie mogę przekierować.');
	}
}
 
if (isset($_SESSION['user_logged'])) {
echo "<div>";
echo "Witaj: " . $_SESSION['user_logged'] ;
echo "&nbsp;";
echo "<a href=\"index.php?action=wylogyj\">Wyloguj</a>";
echo "</div>";
} else {
echo "Witaj: Gościu";
} 
 
if (isset($_REQUEST['action'])) {
   switch($_REQUEST['action']) {
	case 'wyloguj';
	session_start();
	session_unset();
	session_destroy();
 
	redirect('index.php?id=main');
	break;
	}
}
 
 
 
?>
[PHP] pobierz, plaintext 

Proszę o podpowiedz gdzie tkwi błąd. Przejrzałem sporo przykładów i nie wiedziałem aby ktoś używał do wylogowania funkcji switch.
Może nie powinienem w taki sposób pisać tego typu skryptów.

Z góry dziękuję
Pozdrawiam

[mat-bi]

No i panie masz książkowy błąd - literówka w linii 17

[kadlub]

linijki od 23 do 33 zastąp tym

[PHP] pobierz, plaintext 
if (isset($_GET['action'])) {
 
 
	session_start();
	session_unset();
	session_destroy();
 
	redirect('index.php?id=main');
 
}
 
[PHP] pobierz, plaintext 

[mat-bi]

Kadlub, a jak będzie miał ?action=login, to też ma mu niszczyć sesję?

Co do switch'a, można taką konstrukcje zastosować, zwykle jednak uzywa się jej zamiast rozbudowanych instrukcji if...elseif..else

[jerry1984]

miałeś rację ale jestem

Mam jeszcze jedno pytanko, w sprawie "redirect" po przekierowaniu wywala mi błąd dopisuje do ścieżki Url backslash: \ i wygląda to tak:

Kod

http://localhost\/index.php?id=main

a nie widzę żeby znów miał błąd literowy.

Pozdrawiam

Ten post edytował jerry1984 3.05.2011, 22:10:58

[Dipter]

Błędem jest to co podał mat-bi.
kadlub - Bardzo inteligentnie, teraz przy każdej wykonanej akcji będzie niszczyć sesję i przenosić do głównej strony. Chyba jasno widnieje instrukcja switch i stan wyrażenia "wyloguj".

Popraw linie 17 - ?action=wylogyj -> ?action=wyloguj

[kadlub]

Kadlub, a jak będzie miał ?action=login, to też ma mu niszczyć sesję?

Co do switch'a, można taką konstrukcje zastosować, zwykle jednak uzywa się jej zamiast rozbudowanych instrukcji if...elseif..else

ja nie widzę żeby tu przypisywał inną wartość a jak chce to sprawdzić to tak jak mówisz niech da jeszcze if

[mat-bi]

Błędem jest to co podał mat-bi.

Mogę wiedzieć, na jakiej podstawie tak sądzisz? W końcu podałeś to samo, co ja

Co do jerry'ego podaj wynik:

[PHP] pobierz, plaintext 
var_dump($_SERVER);
[PHP] pobierz, plaintext 

[Dipter]

mat-bi
Źle mnie zrozumiałeś ;P Przyznałem Ci rację.

dirname() jest niepotrzebny, poza tym przed zmienną url daj tylko slasha. Bez hostu i niczego ;P

[mat-bi]

Tak jak coś - zgaduję, że korzystasz z Wina

rzeczywiście, $_SERVER jest w tym jak piąte koło u wozu, wystarczy:

[PHP] pobierz, plaintext 
header("Location : $url");
[PHP] pobierz, plaintext 

[jerry1984]

Działa jak w szwajcarskim zegarku.

Dziękuje mat-bi pomogłeś mi bardzo.

Pozdrawiam