Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP & SQL

Nesting IFs lukt niet (Opgelost)

koenvdl - 03/11/2009 20:50 (laatste wijziging 03/11/2009 20:51)

Nieuw lid

bij onderstaande code zou er een registratie in mijn database moeten gebeuren. Sinds dat ik de controle van het emailadres heb ingebouwd loopt er iets grandioos mis.

php code - Bekijk de code zonder highlighting - Klap code in

<?php
if ($submit)
{
    if ($nickname&&$username&&$password&&$password2&&$email)
    {
        if($password == $password2)
        {
            if(strlen($username) > 25 || strlen($nickname) > 25)
            {
                if(strlen($password) > 25 || strlen($password) < 6)
                {
                   	if (strpos($email,"@") === true)
					{
						$var = explode("@",$email);
						if (count($var)==2)
						{
							if(strpos($var[1],".")=== true)
							{
								$connection = mysql_connect('localhost', 'root', '') or die ("Could not connect to the database server!");
               					mysql_select_db("test", $connection) or die ("Could not connect to the database");
                				$register = sprintf("INSERT INTO users (id,username,password,nickname,date,email,registerIP) VALUES('', '%s', '%s', '%s', '%s','%s','%s')",
								mysql_real_escape_string($username),
								mysql_real_escape_string($password),
								mysql_real_escape_string($nickname),
								mysql_real_escape_string($date),
								mysql_real_escape_string($email),
								mysql_real_escape_string($registerIP));
								
                				mysql_query($register);
               		 			die("Successfully registered please <a href='index.php'>Log in</a>!");
							}
						else
							echo "No valid emailaddress. . is missing<br/>";
						}
					else
						echo "No valid emailaddress. @ is missing.<br/>";
                	}
                else
                	echo "Password must be between 6 - 25 characters!";
            	}
            else
            	echo "Username and Nickname maximum is 25 characters";
        	}
        else
            echo "Password does not match!";
    	}
    else
        echo "Please fill in all fields!";
	}
}	
?>

<?php
if ($submit)
{
    if ($nickname&&$username&&$password&&$password2&&$email)
    {
        if($password == $password2)
        {
            if(strlen($username) > 25 || strlen($nickname) > 25)
            {
                if(strlen($password) > 25 || strlen($password) < 6)
                {
                   	if (strpos($email,"@") === true)
					{
						$var = explode("@",$email);
						if (count($var)==2)
						{
							if(strpos($var[1],".")=== true)
							{
								$connection = mysql_connect('localhost', 'root', '') or die ("Could not connect to the database server!");
               					mysql_select_db("test", $connection) or die ("Could not connect to the database");
                				$register = sprintf("INSERT INTO users (id,username,password,nickname,date,email,registerIP) VALUES('', '%s', '%s', '%s', '%s','%s','%s')",
								mysql_real_escape_string($username),
								mysql_real_escape_string($password),
								mysql_real_escape_string($nickname),
								mysql_real_escape_string($date),
								mysql_real_escape_string($email),
								mysql_real_escape_string($registerIP));
 
                				mysql_query($register);
               		 			die("Successfully registered please <a href='index.php'>Log in</a>!");
							}
						else
							echo "No valid emailaddress. . is missing<br/>";
						}
					else
						echo "No valid emailaddress. @ is missing.<br/>";
                	}
                else
                	echo "Password must be between 6 - 25 characters!";
            	}
            else
            	echo "Username and Nickname maximum is 25 characters";
        	}
        else
            echo "Password does not match!";
    	}
    else
        echo "Please fill in all fields!";
	}
}	
?>

6 antwoorden

Gesponsorde links

vinTage - 03/11/2009 20:58 (laatste wijziging 03/11/2009 21:01)
Nieuw lid		je mist sowieso een ) na je email check, maar waarom zo checken, er zijn andere makkelijkere manieren om te checken op een valid email adres. if (strpos($email,"@") === true))

NTS64 - 03/11/2009 20:59

PHP gevorderde

Je doet dit veel te omslachtig imo.
Waarom doe je niet iets in deze aard:

php code - Bekijk de code zonder highlighting - Klap code in

<?php
$errors = array();
if(statement)
{
   $errors[] = "Error 1 omschrijving ";
}
if(statement2)
{
	$errors[] = "Error 2 omschrijving";
}
if(statement3)
{
	$errors[] = "Error 3 omschrijving";
}
// Enzovoorts
// Ten slotte kijk je dan of de array errors keys bevat of niet
if(count($errors) > 0)
{
	echo 'De registratie kan niet voltooid worden, volgende fouten traden op: <br :>';
	foreach($errors as $error)
	{
		echo $error.'<br />';
	}
}
else
{
	// registreren
}
?>

<?php
$errors = array();
if(statement)
{
   $errors[] = "Error 1 omschrijving ";
}
if(statement2)
{
	$errors[] = "Error 2 omschrijving";
}
if(statement3)
{
	$errors[] = "Error 3 omschrijving";
}
// Enzovoorts
// Ten slotte kijk je dan of de array errors keys bevat of niet
if(count($errors) > 0)
{
	echo 'De registratie kan niet voltooid worden, volgende fouten traden op: <br :>';
	foreach($errors as $error)
	{
		echo $error.'<br />';
	}
}
else
{
	// registreren
}
?>

Martijn - 03/11/2009 22:31

Crew PHP

php code - Bekijk de code zonder highlighting - Klap code in

if(voorwaarde1){...;}
elseif(voorwaarde2){...;}
elseif(voorwaarde3){...;}
elseif(etc etc etc  ){...;}
else{ echo"goed";}

if(voorwaarde1){...;}
elseif(voorwaarde2){...;}
elseif(voorwaarde3){...;}
elseif(etc etc etc  ){...;}
else{ echo"goed";}

vinTage - 03/11/2009 23:07
Nieuw lid		@DeviourSoul, dan is die van Woedi nogal wat handiger, dan kan je alle errors in een keer (forloop) tonen ipv per error.

Martijn - 04/11/2009 19:24
Crew PHP		als je het zo wilt doen kun je beter ipv de foreach dit doen: php code - Bekijk de code zonder highlighting - Klap code in echo implode("<br />-",$array); echo implode("<br />-",$array);

vinTage - 04/11/2009 21:07
Nieuw lid		Maar dat beperkt je weer in je vrijheid om eea op te maken met css (tenzij je de opmaak ook in de errors gaat proppen (wat dan weer niet klopt als je met bv gekleurde rijen om en om wilt doen)

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten