Witam,

mam problem sprawdzam czy użytkownik jest zalogowany i kod wygląda w taki sposób że jeśli nie zwracam header z przekierowaniem w if-ie. Rozwiązanie działa ale czy dobrze to "wygląda"

Przykładowy kod, w praktyce tam jest kilka ifów, które sprawdzają poziom uprawnień i aktywacje maila jednak wszystkie maja header jako zasób w razie poprawności.

[PHP] pobierz, plaintext 
if(check_if_logged())
{
 //...
}
 
check_if_logged()
{
 if($this->session->logged())
{
 return TRUE;
}
else
{
 header("Location: http://example.com");
}
}
[PHP] pobierz, plaintext 

po header(); powinieneś dać jeszcze exit; w przeciwnym wypadku skrpyt będzie się wykonywał do końca (ew. return false, jeśli to sprawdzasz później)

Chyba lepiej by było napisać

[PHP] pobierz, plaintext 
function check_if_logged()
{
 if($this->session->logged())
    return TRUE;
else
    return FALSE;
}
 
if (check_if_logged)
   //...
else
   header("Location: <a href="http://example.com&quot%3b%29;" target="_blank">http://example.com");</a>
[PHP] pobierz, plaintext