(DEV) Loginsysteem

Hoi,

Ik ben aant werk met een leuk loginsysteempje, geen idee of er nog extra mogelijkheden komen, maargoed, check uit wat klaar is. Als je nog tips hebt hoop ik ze graag te lezen!
Het word gemaakt in PHP en MYSQLi en deels OOP. ;$

Al gedaan:

Inloggen 100%
Uitloggen 100%
Registratie 100%

Credits aan:

Zakenman 50% (scripten van alle shitz)
Cssdeck.com 50% (de thema)

Snippet/screens:

$mail = $con->real_escape_string($_POST[email]);
$password2 = $con->real_escape_string(md5($_POST[password2]));

//empty field check
if(empty($_POST[username]) || empty($_POST[password]) || empty($_POST[email]))
{
    header(/aanmelden?error=1);
    }else{
        //username check
        $checkIfExists = $con->query("SELECT * FROM leerling WHERE username=$username");
         if($checkIfExists->num_rows > 0)
        {
            header(location: /aanmelden?error=2);
        }else{

UPDATE:

* Dankzij Aar heb ik nu i.p.v empty trim gebrukt. Dankjewel <3

http://www.mediafire.com/?ladv616fp6re26a
//
http://yupa.nl

Have fun!

Groetjes,
Ripz
http://yupa.nl

je bent op regel 7 location vergeten

Gebruik de OOP-style van mysqli. Beter dan een eigen class.

Correcte uitlijning is ook weer moeilijk.

en waarom controleren met empty() ?

Aarclay schreef:

en waarom controleren met empty() ?

Waarom niet dan?

Raaoul schreef:

Correcte uitlijning is ook weer moeilijk.

Mijn excuses, dat is mijn slechtste gedeelte. xD

Dylanh schreef:

je bent op regel 7 location vergeten

Inderdaad, had ik al verbeterd.

Aarclay schreef:

Gebruik de OOP-style van mysqli. Beter dan een eigen class.

Hoe bedoel je?

Hopelijk komen er ook nog positieve reacties, alvast bedankt voor het kritiek!

- empty laat meer door dan je denkt terwijl je anders verwacht.
- Mysqli is zelf al object oriented: http://nl1.php.net/mysqli_query

Aarclay schreef:

- empty laat meer door dan je denkt terwijl je anders verwacht.
- Mysqli is zelf al object oriented: http://nl1.php.net/mysqli_query

Dankjewel.

Wat is anders een goede vervanging voor empty?

isset?

if(trim($var)==) {
// leeg
}

Zo doe ik dat...

thanks

Aarclay schreef:

if(trim($var)==) {
// leeg
}

Zo doe ik dat...

if(empty(trim($var))) {
// leeg
}

Dit komt toch op hetzelfde neer neem ik aan? Of is het meer dat == korter is?

Wat doet regel 7 van je snippet eigenlijk??

Lees de draad voor je blaat

Zakenman schreef:

Eerdere citaten tonen

Dylanh schreef:

je bent op regel 7 location vergeten

Inderdaad, had ik al verbeterd.

Het is released veel plezier ermee!