On Wed, 2004-04-07 at 18:47, Andy B wrote: > I have this very large and long if statement: > if > (!empty($_SESSION['add']['type']) > && !empty($_SESSION['add']['start_date'] > && !empty($_SESSION['add']['end_date']) > && !empty($_SESSION['add']['name']) > && !empty($_SESSION['add']['county']) > && !empty($_SESSION['add']['discription']) > && !empty($_SESSION['add']['StartingDay']) > && !empty($_SESSION['add']['StartingMonth']) > && !empty($_SESSION['add']['StartingYear']) > && !empty($_SESSION['add']['EndingDay']}) > && !empty($_SESSION['add']['EndingMonth']) > && !empty($_SESSION['add']['EndingYear'])) > {//run the insert query} > else > {//show error since one or more fields above are blank} > was wondering if there was really any way to condense that down to something > any better? all fields in the form that those came from are required...
$errors = array(); foreach( $_SESSION['add'] as $key => $value ) { if( isempty( $value ) ) { $errors[] = "You forgot to fill $key." } } if( count( $array ) ) { echo 'Hey MORON! You have the following errors:<br />'; echo '<ul>' .'<li>' .implode( '</li><li>', $errors ); .'</li>' .'</ul>'; } else { // Do SQL INSERT query. } Cheers, Rob. -- .------------------------------------------------------------. | InterJinn Application Framework - http://www.interjinn.com | :------------------------------------------------------------: | An application and templating framework for PHP. Boasting | | a powerful, scalable system for accessing system services | | such as forms, properties, sessions, and caches. InterJinn | | also provides an extremely flexible architecture for | | creating re-usable components quickly and easily. | `------------------------------------------------------------' -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php