PHP Code:
<link rel="stylesheet" href="blue/cq.css" type="text/css">
<?
include ('connect.php');
if (isset($_POST['action']) && $action == 'toevoegen')
{
$db_username = isset($_POST['db_username'])?$_POST['db_username']:'';
$db_password = isset($_POST['db_password'])?$_POST['db_password']:'';
$password2 = isset($_POST['password2'])?$_POST['password2']:'';
$db_email = isset($_POST['db_email'])?$_POST['db_email']:'';
$db_encrypt=md5($db_password);
$select = "SELECT username FROM drregister";
$res_select = mysql_query($select) or die(mysql_error());
while($list=mysql_fetch_object($res_select))
{
$username=$list->username;
$databaseusername = strtoUpper($username);
$registerusername = strtoUpper($db_username);
if ($registerusername == $databaseusername)
{
$i=1;
}
}
if (($i!=1) && ($db_username!="") && ($db_password!="") && ($db_email!=""))
{
if (($db_password==$password2) && (preg_match("#^[_\.0-9a-z-]+@([0-9a-z][0-9a-z-]+\.)+[a-z]{2,3}$#i", $db_email)))
{
mail('Dennisken@pandora.be', 'Buf Clansite', $db_username, "From: Buf ");
$db_date=date("D M j G:i:s Y");
$insert="INSERT INTO buf_register (username,pass,email,date)
VALUES ('$db_username','$db_encrypt','$db_email','$db_dat e')";
$res_insert = mysql_query($insert);
echo "<center><br><br>Your registration is complete!<br><br>";
echo "<center>The Admin will grant you access as fast as possible<br><br>";
}
}
elseif ($i==1)
{
header ("Location: register.php?fout=1");
}
}
if (!$_POST['action'])
{
if (isset($_GET['fout']) && $_GET['fout'] == 1)
{
echo"That username allready exists";
}
echo"<center>";
echo"<form method=POST>";
echo"<k2> Please fill in this form to register!</k2> ";
echo"<br>";
echo"<br>";
echo"<table border=0 rules=none>";
echo"<tr><td align=center><k>Username</k></th>";
echo"<tr><td align=left><input type=text name=db_username class=input>";
echo"<tr><td align=center><k>Password</k>";
echo"<tr><td align=left><input type=password name=db_password class=input>";
echo"<tr><td align=center><k>Enter password again</k>";
echo"<tr><td align=left><input type=password name=password2 class=input>";
echo"<tr><td align=center><k>Email</k>";
echo"<tr><td align=left><input type=text name=db_email class=input>";
echo"</tr>";
echo"<tr><td align=center colspan=2 height=10><input type=submit name=action value=toevoegen class=input></td>";
echo"</table>";
echo"</center>";
}
echo"<a href=index.php>Naar Login</a>";
?>
meeste fouten eruit, kan nog syntaxis foutje van mij inzitten
Enkele punten:
1) Ga werken met true en false, ipv van je domme 1 en niet 1 (n/o).
2) Leer mysql, je kan met 1 query zoeken of die username al bestaat en dan moet je niet zo een extreem lange while gebruiken.
3) VOOR IEDEREEN (aangezien ik dit nog pokke veel zie): Leer eens dat html echo'en met php ZEER onhandig is, het is niet enkel trager, maar ook nog eens moeilijker om html fouten op te sporen, je moet meer escapen, ..., gebruik wat meer <?php en ?> tags of een template systeem
4) http://be2.php.net/globals
5) http://be2.php.net/mysql_error
Da zijn denk ik de voornaamste zaken die u script gewoon ongelofelijk slecht maken. Ma troost u, das normaal als ge een beginner zijt, ik kan mij niet voorstellen dat iemand van in het begin een deftig script als dat maakt (behalve Bart_Simpson### natuurlijk). Enkel: als iemand u raad geeft zoals die $_POST te gebruiken, ga dan eerst opzoeken en proberen voor je zijn raad redelijk snel afwimpelt met de "maar toch" zinnen.