if statement...

[ashleek007]

Hi All,

 

Just working on some form validation and have hit a problem... at the minute each error message displays individually of each other... but I want them to all display at the same time if the statements are true...  for example if someone hasnt put in their Title or Forename or Surname it should show all three error messages... heres the code below...

 

 

if ( $USER_VARS['TITLE'] == "" )
			{
				$USER_VARS['TITLE_ERR'] = "* Please select your Title from the list";
				return 0;
			}


			if ( strlen($USER_VARS['FORE']) > 0 )
			{
				if (!preg_match("/[a-z]+/", $USER_VARS['FORE'])) 
				{
				$USER_VARS['FORE_ERR'] = "* Please ensure First Name contains only letters (A-Z, upper and lower case), spaces, hyphens or apostrophes.";
				return 0;
				}
			}
			else
			{
				$USER_VARS['FORE_ERR'] = "* Please enter a First Name";
				return 0;
			}
			if ( strlen($USER_VARS['SURN']) < 1 )
			{
				$USER_VARS['SURN_ERR'] = "* Please ensure Surname contains only letters (A-Z, upper and lower case), spaces, hyphens or apostrophes.";
				return 0;
			}

			if ($USER_VARS['DOB_DAY'] == "Day")
			{
				$USER_VARS['DOB_ERR'] = "* Please select your Date of Birth";
				return 0;
			}

			if ($USER_VARS['DOB_MONTH'] == "Month")
			{
				$USER_VARS['DOB_ERR'] = "* Please select the month you were born on";
				return 0;
			}

			if ($USER_VARS['DOB_YEAR'] == "19")
			{
				$USER_VARS['DOB_ERR'] = "* Please select the year you were born on";
				return 0;
			}

			if ($USER_VARS['EMPLOYMENT'] == "Select Employment")
			{
				$USER_VARS['EMPLOYMENT_ERR'] = "* Please select your employment status from the list";
				return 0;
			}

 

Thanks for any help you guys can give,

Ash

[OOP]

Hi there,

just remove the return 0 from all the if blocks and after all if statments checks if the error array ( in your case $USER_VARS ) is not empty return it to the caller script.

 

regards,

OOP

[Daniel0]

Just do like this:

$errors[] = "The error message here";

 

Then when you are done checking for errors, see if count($errors) > 0 and then display the errors in the array.

[ashleek007]

cheers all,

 

Daniel0... do you mean like this??

 

			if ($USER_VARS['EMPLOYMENT'] == "")
			{
				$USER_VARS['EMPLOYMENT_ERR'] = "* Please select your employment status from the list";
				$errors[] = $USER_VARS['EMPLOYMENT_ERR'];
			}

			if( count($errors) > 0 )
			{
			echo $errors[];
			}

 

Cheers,

Ash

[Daniel0]

Yes, except it would be something like this:

if(count($errors) > 0)
{
echo implode('<br />', $errors);
}