Kan jeg gøre det bedre?
Jeg har en kode til at oprette en bruger på min hp, den ser ud som dette:<?
if (isset($_POST['submit'])) {
$name = $_POST['name'];
$adress = $_POST['adress'];
$city = $_POST['city'];
$zip = $_POST['zip'];
$email = $_POST['email'];
$username = strtolower($_POST['username']);
$password = $_POST['password'];
$password2 = $_POST['password2'];
if (empty($name) OR empty($adress) OR empty($city) OR empty($zip) OR empty($email) OR empty($username) OR empty($password) OR empty($password2)) {
echo "<br><br><span style=\"color: #ff0000\">Alle felter skal udfyldes...</span>";
} else {
mysql_connect('***', '***', '***');
mysql_select_db('***');
$result = mysql_query("select username from user where username = '$username'") or die (mysql_error());
$number = mysql_num_rows($result);
if ($number > 0) {
echo "<br><br><span style=\"color: #ff00ff\">Brugernavnet er optaget...</span>";
} else {
if ($password != $password2) {
echo "<br><br><span style=\"color: #ff0000\">Adgangskoderne passer ikke sammen...</span>";
} else {
mysql_query("insert into user (username, password, city, name, zip, adress, email, date) values ('$username', password('$password'), '$city', '$name', '$zip', '$adress', '$email', now())") or die(mysql_error());
echo "<br><br><span style=\"color: #0000ff\">Brugeren er oprettet, du kan nu logge ind...</span>";
}
}
mysql_close();
}
}
?>
Kan jeg gøre det bedre, eller mere sikkert?
