[PHP] Walidacja formularza Opcje

[boro11]

Witam!
Robię sobie walidację formularza w php, ale mam pewien problem, otóz komunikaty mi się wyświetlają niepoprawnie.

Mój kod:

[PHP] pobierz, plaintext 
 
if (isset($captcha) && !empty($captcha)) 
{
	if($captcha==$_SESSION['code'] && !empty($captcha))
		{
			if(isset($nick) && !empty($nick))
				{
					if(isset($country) && !empty($country))
						{	
							if($pmail or $ppmail)
								{
									if(isset($nick) && !empty($nick))
										{
											if(isset($pass) && !empty($pass))
												{
													if($repass==$pass && !empty($captcha))
														{
															if(isset($mail) && !empty($mail))
																{
																	$zap=array(1 => 'SELECT',2 => '*',3 => 'FROM',4 => $pref.'users',5 => 'WHERE',6 => "nick='$nick'");
																	$logs=$db->zap($zap);
																	$row = mysql_fetch_array($logs);
																	if($row['nick']==$nick)
																	echo "<h2>This username is already in use, please use another.</h2></br>";
																	else
																		{
																			$zap=array(1 => 'SELECT',2 => 'nick',3 => 'FROM',4 => $pref.'users',5 => 'WHERE',6 => "ip='$ip'");
																			$log=$db->zap($zap);
																			$row = mysql_fetch_array($log);
																			$pass1=sha1($pass);
																			$cod=$nick.mt_rand(1,5000);
																			$zap=array(1 =>'INSERT INTO',2 => $pref.'users',3 => "(id,nick,pass,mail,ip,lastvisit,session,money,payemail,method,member,advice,
register,country,buyprem,name,cod,ppmail,regforum,lr,signature)",4 => 'VALUES',5 => "('' , '$nick', '$pass1', '$mail', '$ip', '0000-00-00', '', '0.2', '$pmail', '$pay', 'standard', '$ref', '".Date("Y-m-d")."', '$country', '',  '$name', '$cod', '$ppmail','0', '$lr', '')");
																			$db->zap($zap);
																			$zap=array(1 =>'INSERT INTO',2 => $pref.'cheaters',3 => "(id,nick,ip,much)",4 => 'VALUES',5 => "('' , '$nick', '$ip', '0')");	
																			$db->zap($zap);
																			echo "<h2>Your account is created. You can now login.</h2></br>";
																			$update_t = "UPDATE  stats SET user=user+1";
																			mysql_query($update_t) or die(mysql_error());										
																		}  
																} 
															else
															echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
														}
													else
													echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> Password doesn't match. Please try again.</div>";
												}
											else
											echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
										}
									else
									echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
								}
							else
							echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
						} 
					else
					echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
				} 
			else
			echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
		} 
	else
	echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> Wrong captcha code. Please try again.</div>";
}
else
echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
[PHP] pobierz, plaintext 

Komunikat o tym, że należy uzupełnić wszystkie pola pojawia się ciągle, ma ktoś pomysł czemu?

Ten post edytował boro11 1.12.2013, 01:38:01

[com]

pewnie któryś z warunków zamykasz nie tam gdzie trzeba, masz tak porobione wcięcia i długie linie że kod jest na tyle nie czytelny, że nie da się zlokalizować w nim nic... pamiętaj o zasadzie max 120 znaków na linie optymalne 80..

[boro11]

W edytorze (notepad++) to wygląda lepiej bo wcięcia są równo, forum trochę pomieszało, generalnie warunek jest dobry na sprawdzanie?

[com]

Ok wkleiłem w n++, jest trochę lepiej ale nadal ciężko się wgl w tym odnaleźć, generalnie to co widzę, że sprawdzasz kilkukrotnie to samo wystarczy raz wtedy parę ifów zniknie i pewnie ten problem też

[boro11]

Czyli po prostu do jednego if'a "dodać" inne if'y za pomocą && ? Oczywiście chodzi mi o same warunki.

Trochę go ukróciłem według Twojej rady, ale nadal to samo:

[PHP] pobierz, plaintext 
if (isset($captcha) && !empty($captcha)) 
{
	if($captcha==$_SESSION['code'] && !empty($captcha))
		{
			if(isset($nick) && !empty($nick) && isset($country) && !empty($country) && $pmail or $ppmail && isset($nick) && !empty($nick) && isset($pass) && !empty($pass))
				{
					if($repass==$pass && !empty($captcha))
						{
							if(isset($mail) && !empty($mail))
							{
								$zap=array(1 => 'SELECT',2 => '*',3 => 'FROM',4 => $pref.'users',5 => 'WHERE',6 => "nick='$nick'");
								$logs=$db->zap($zap);
								$row = mysql_fetch_array($logs);
								if($row['nick']==$nick)
								{
									echo "<h2>This username is already in use, please use another.</h2></br>";
								}
								else
								{
									$zap=array(1 => 'SELECT',2 => 'nick',3 => 'FROM',4 => $pref.'users',5 => 'WHERE',6 => "ip='$ip'");
									$log=$db->zap($zap);
									$row = mysql_fetch_array($log);
									$pass1=sha1($pass);
									$cod=$nick.mt_rand(1,5000);
									$zap=array(1 =>'INSERT INTO',2 => $pref.'users',3 => "(id,nick,pass,mail,ip,lastvisit,session,money,payemail,method,member,advice,
register,country,buyprem,name,cod,ppmail,regforum,lr,signature)",4 => 'VALUES',5 => "('' , '$nick', '$pass1', '$mail', '$ip', '0000-00-00', '', '0.2', '$pmail', '$pay', 'standard', '$ref', '".Date("Y-m-d")."', '$country', '',  '$name', '$cod', '$ppmail','0', '$lr', '')");
									$db->zap($zap);
									$zap=array(1 =>'INSERT INTO',2 => $pref.'cheaters',3 => "(id,nick,ip,much)",4 => 'VALUES',5 => "('' , '$nick', '$ip', '0')");	
									$db->zap($zap);
									echo "<h2>Your account is created. You can now login.</h2></br>";
									$update_t = "UPDATE  stats SET user=user+1";
									mysql_query($update_t) or die(mysql_error());										
								}  
							}
							else
							{
								echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
							}
						}
						else
						{
							echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> Password doesn't match. Please try again.</div>";
						}
				}
				else
				{
				echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
				}
		} 
		else
		{
		echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> Wrong captcha code. Please try again.</div>";
		}
}
else
{
echo "<div class='alert alert-warning  text-center'><strong>Warning!</strong> All fields are requied.</div>";
}
[PHP] pobierz, plaintext 

Nawiasy są dobrze prawie na 100%, notepad dobrze je podświetla jak sprawdzałem parokrotnie :/

Edit: Okej ostatnie else nie było potrzebne

Ten post edytował boro11 1.12.2013, 02:09:16