Eerst login systeem

  • Hallo cp,


    Ik heb hier een eerste login (session) script.
    Ik ben alleen benieuwd of hij goed is gescript :S.


    dit is de session script.


    Ja De connectie met database enzo is goed.


    Als iemand een fout ziet of als er iets beter kan.
    willen jullie dat dan zeggen of een verbetering laten zien :D.


    Mvg,
    Marco Boekholt


  • Houd altijd de code overzichtelijk. Als een password MD5 wordt gehashed, wat ik zwaar af raad, hoef je er geen mysql_real_escape_strng overheen te doen. Header Locations altijd beeindigen met een exit(). En variabelen aanmaken van post's om zo overzicht te houden. Waarschijnlijk komt er een error met dat $_POST variable niet gevonden kan worden als deze niet wordt mee gestuurd. Dit is simpel op te lossen met dit stukje:


    PHP
    isset($_POST['username']) ? $_username = mysql_real_escape_string($_POST['username']) : $_username = '';
    if(empty($_username)) {
    die('Er is een ongeldige gebruikersnaam opgegeven.');
    }


    Daarnaast kun je nog bepaalde karakters toestaan in de username en password. Zoals alleen maar a-zA-Z en 0-9. Zet ook een maximum lengte voor de gebruikersnaam erop.


    Ik zelf zou het heel anders doen, maar dat is teveel om uit te leggen ;)

    Kast: HAF922 | CPU: I7-930 @ 4.0GHz | CPU Cooler: Noctua NH-D14 | HDD0: Crucial M4 128GB, HDD1: Kingston SSD 64GB, HDD2/3: WD Black & Green 1TB, HDD4: Seagate 1.5TB | Mem: Kingston HyperX 12GB @ 1600MHz | Graphics: Crossfire HD6970 | Res: 5760x1080

Participate now!

Heb je nog geen account? Registreer je nu en word deel van onze community!