Simple Question?

[Trium918]

Do you find anything wrong with the follow script? If so, how can I improve it?

 

<?php
//<-- VALIDATE FORM ENTRY -->
function valid_entry($form_vars) {
   foreach($form_vars as $entry)
   {
      if(empty($entry) || !ereg("^[a-zA-Z0-9]+$", $entry)) {
  	return false;
  }

   }
   return true;
}
?>

Check the form!

<?php
if(isset($submit)) {
// specific form variables to check
// $form_vars = array($name);

  if(!valid_entry($_POST)) {
   print "invalid data!"; 
  } else { 
   print "valid data";
  }
}
?>

[Aureole]

I'd suggest using preg_match() as it is much faster than ereg(), other than that it looks fine.