Regristratie probleem

Door Bob Ruis op 01-05-2013 11:28

1.160 views

Hallo mensen van het PHPHULP,

Ik ben bezig met een registratie systeem, alleen nu heb ik 1 probleem nog.... Het probleem is: dat wanneer ik de controle op de dubbel wachtwoord uitvoer zegt het na het klikken op de registratie button dat de wachtwoorden niet overeen komen(dit is goed) maar toch staat daaronder de melding van SUCCES je bent aangemeld hoe kan ik ervoor zorgen dat hij alleen de melding laat zien van wachtwoorden komen niet overeen en dat het niet wordt weggeschreven naar de database!

Alvast bedankt voor de moeite!

<?php
if(!empty($_POST['username']) && !empty($_POST['password']) && !empty($_POST['email']) && !empty($_POST['password_check']))
{
$username = mysql_real_escape_string($_POST['username']);
$password = md5(mysql_real_escape_string($_POST['password']));
$email = mysql_real_escape_string($_POST['email']);
$password_check = md5(mysql_real_escape_string($_POST['password_check']));
$checkusername = mysql_query("SELECT * FROM users WHERE Username = '".$username."'");
$checkemail = mysql_query("SELECT * FROM users WHERE EmailAddress = '".$email."'");

if(mysql_num_rows($checkemail) == 1)
{
echo "<h1>Error</h1>";
echo "<p>Sorry, de email is al in gebruik. Ga terug en probeer opnieuw.</p>". header( "refresh:3;");
}

if($_POST['password'] !== $_POST['password_check']);

{
echo "Wachtwoorden komen niet overeen";
}
if(mysql_num_rows($checkusername) == 1)
{
echo "<h1>Error</h1>";
echo "<p>Sorry, De gebruikersnaam is al in gebruik. Ga terug en probeer opnieuw.</p>";//. header( "refresh:3;");
}

else
{
$registerquery = mysql_query("INSERT INTO users (Username, Password, Password_check, EmailAddress) VALUES('".$username."', '".$password."', '".$password_check."', '".$email."')");
}
if($registerquery)
{
echo "<h1>Success</h1>";
echo "<p>Je account is aangemaakt. <a href='index.php'>Klik hier</a> om in te loggen of wacht een aantal seconden.</p>"; //. header( "refresh:3;url=index.php" );
}
else
{
echo "<h1>Error</h1>";
echo "<p>Sorry, je regristratie is mislukt. Ga terug en probeer opnieuw.</p>";//. header( "refresh:3;url=register.php" );
}

}
else
{
?>

- Ariën -

01-05-2013 11:34

Is een elseif niet een beter idee voor je statements?

Bob Ruis

01-05-2013 11:40

- Aar - op 01/05/2013 11:34:03

Is een elseif niet een beter idee voor je statements?

Als je mij kan uitleggen hoe ik dat regel ? Heb niet heel veel verstand van PHP

Wouter J

01-05-2013 11:41

Dan moet je de rest in een else statement plaatsen. Omdat dat handig is raad ik je aan zoiets te doen: (de condities zijn natuurlijk dummy)
<?php
$errors = array();

if (form is submitted) {
if (username is not correct) {
$errors[] = 'De gebruikers naam is niet geldig';
}
if (wachtwords are not equal) {
$errors[] = 'De wachtwoorden komen niet overeen';
}
if (email is not valid) {
$errors[] = 'Geen geldig email adres';
}

if (0 === count($errors)) {
// geen errors gevonden, execute query
}
}
?>
En dan ergens boven je form:

<?php if (0 !== count($errors)) : ?>
    <ul class=error-list>
        <?php foreach ($errors as $error) : ?>
        <li><?php $error ?></li>
        <?php endforeach ?>
    </ul>
<?php endif ?>

Mocht je iets niet begrijpen: Stel alsjeblieft vragen, code kopieren zonder het volledig te begrijpen is het slechtste wat je kan doen.

Nog wat andere tips:
- Ik mis foutafhandeling (kijk een naar http://wouterj.github.io/sql-boilerplate )
- md5 is niet meer veilig, gebruik liever een hoge sha* functie ([php]sha256[/php] bijv.)
- wanneer je een hash functie gebruikt is mysql_real_escape_string niet meer nodig

Reageren

Inloggen om te reageren