If sætning

min else virker ikke (tror det er fordi den er inde i while). hvordan kan det laves? og er der en smartere måde at gøre loginsystemet sikkert på?

Umiddelbart kan jeg ikke se, at der er noget galt i din if-sætning, men det kan gøres nemmere ved f.eks. at gøre således:

$query = mysql_query("select brugernavn from admins where brugernavn='".$_SESSION['brugernavn']."'") or die(mysql_error());
if(mysql_num_rows($query) > 0) {
  echo "Logget ind";
} else {
  echo "!!!!";
}

eller:
$query = mysql_query("select count(*) from admins where brugernavn='".$_SESSION['brugernavn']."'") or die(mysql_error());
if(mysql_result($query, 0) > 0) {
  echo "Logget ind";
} else {
  echo "!!!!";
}

Den sidste er vist mindst ressourcekrævende...

Hvad hvis jeg også vil skrive nogle felter ud?

F.eks.:

$query = mysql_query("select brugernavn from admins where brugernavn='".$_SESSION['brugernavn']."'") or die(mysql_error());
if(mysql_num_rows($query) > 0) {
  while($row = mysql_fetch_array($query) {
    echo Dit brugernavn er: ".$row['brugernavn']."<br />";
  }
} else {
  echo "!!!!";
}

Er den så sikker?

Hvad mener du om den er sikker? Den er i hvert fald ligeså sikker som den if-sætning du startede med...

Ja, det ved jeg godt.

Men er der yderligere ting der bør laves for at gøre den mere sikker?

Det skulle jo være at tage et password-check med i din forespørgsel, et brugernavn i sig selv kan nok gættes.

Jeg har jo selvfølgelig lavet et check af password i min loginform.

Burde det også være dér?

Hvordan tjekker du det i login-formen da? Du bør jo krydstjekke det med hvad der ligger i tabellen.

Jeg får jo kun lavet min session hvis brugernavn og kodeord er korrekt skrevet ind??

Nå, ja det er klart nok. Så synes jeg sådan set det er udemærket.

Smid et svar.

Hvad bruges session_id egentlig til?

Session_id kan f.eks. bruges hvis sessions skal sendes mellem sider med GET tror jeg.

Hvis du vil læse mere, så prøv www.php.net/session_id

Det er for at adskille forskellige sessioner fra hinanden.. Du kan f.eks. adskille besøgende på din hjemmeside vha sessionsid'et, da idét er unikt.

Sessionsid'et bliver ofte brugt til at genskabe sessionsdata fra f.eks. en database.

Men er session_id nødvendigt?

Overhovedet ikke..

- ikke for dig i hvert fald.