Hallo,
Welche Version von PHP kommt bei Dir zum Einsatz?
Seit einigen Jahren ( spätestens seit Beginn der v5 ) ist es sinnvoller, $_POST oder $_GET anstatt $_REQUEST zu verwenden ( je nachdem, über welche Methode das Formular die Daten versendet ).
Als Entwickler Deiner Scripte, weißt Du, über welchen Kanal die Daten kommen!
Du benötigst auch kein stripslashes(), da Du mit mysqli_real_escape_string() eh die Daten maskierst. Entweder es wird so gefunden oder nicht.
Es gibt keinen Grund, dem "bösen" User behilflich zu sein, indem man nicht gewünschte Zeichen aus der Eingabe entfernt.
Dann empfehle ich PreparedStatements zu verwenden. Da ich PDO verwende, kann ich Dir das nur aus dem Ärmel geschüttelt für MySQLi beispielhaft zeigen.
Dein Code sähe mit den bisher angesprochenen Veränderungen in etwa so aus:
require('db.php');
if (isset($_POST['username'])){
$username = mysqli_real_escape_string($con,$username);
$zipcode = mysqli_real_escape_string($con,$zipcode);
$email = mysqli_real_escape_string($con,$email);
$password = mysqli_real_escape_string($con,$password);
$trn_date = date("Y-m-d H:i:s");
$stmnt = mysql_prepare( $con, "INSERT into `users` (id, username, password, email, trn_date) VALUES (?, ?, ?, ?, ?)" );
mysqli_bind_param( $con, $zipcode, $username, md5($password), $email, $trn_date );
mysqli_stmt_execute( $stmnt );
if ( mysqli_stmt_affected_rows($stmnt) )
echo "<div class='content'><h3>You are registered successfully.</h3><br/>Click here to <a href='login.php'>Login</a></div>";
} else { /* ...was auch immer hier kommt... */ }
Alles anzeigen
Ich habe hierfür mal den prozeduralen Stil verwendet, wie Du.
Ich empfehle allerdings für MySQLi den objektorientierten Stil anzugewöhnen.
Weiterhin solltest Du Passwörter statt mit md5() mit password_hash() und password_verify() handeln.
Gruß Arne
