Mam oto taki kod, który niszczy sesję za pomocą liku <a href=''>. Jest on umieszczony na index2.php. Po wciśnięciu niszczy sesję i przekierowuje na index.php, gdzie jest formularz logowania.

[PHP] pobierz, plaintext 
session_start();
if(!isset($_SESSION['id'])) { header("Location: index.php"); return FALSE; }
if(!($_GET['action']='logout')) {
	unset($_SESSION['id']);
        session_destroy();
        header("Location: index.php");
}
 
[PHP] pobierz, plaintext 

Po wciśnięciu przycisku nie nie niszczy sesji, gdyż po przekierowaniu mogę bez logowania wejść na index2.php.

Spróbuj tak

[PHP] pobierz, plaintext 
session_start();
if(!isset($_SESSION['id'])) { header("Location: index.php"); return FALSE; }
if($_GET['action']=='logout') {
	unset($_SESSION['id']);
        session_destroy();
        header("Location: index.php");
}
 
[PHP] pobierz, plaintext 

Teraz nie mogę się zalogować nawet

Ekhm, na pewno return false? A nie miałeś na myśli exit?

Z tego co wiem obie funkcje są niemal takie same. Jeśli nie, dzie użyć exit()?

Nie są takie same.

Marsz do manuala.

No przecież napisałem że są niemal takie same.

Nie, return służy do zwracania wyniku działania czy to funkcji czy metody w klasie. exit zabija dalsze wykonywanie skryptu. Dalej niemal takie same?

dobrze nie są takie same. Rozwiązałem problem:

[PHP] pobierz, plaintext 
if($_GET['action']) {
if($_GET['action']='logout') {
	unset($_SESSION['id']);
        session_destroy();
        header("Location: index.php");
        exit();
} }
[PHP] pobierz, plaintext 

A czy takie coś nie było by lepsze?

[PHP] pobierz, plaintext 
switch($_GET['action']){
case"logout":
	unset($_SESSION['id']);
    session_destroy();
    header("Location: index.php");
    exit();
}
[PHP] pobierz, plaintext