my form is validated when load

[Hoangsta]

when i load up my form, before hitting submit...the form is already showing that some validating needs to be entered. for example, i set the validation to check for firstname if it's blank...

I haven't entered anything yet, and it's giving me errors before I even enter any data...what's up?

$first_name = $POST ['first_name'];
$last_name = $POST['last_name'];
$email_address = $POST['email_address'];
$username = $POST['username'];

$first_name = stripslashes ($first_name);

$last_name = stripslashes($last_name);
$email_address = stripslashes($email_address);
$username = stripslashes($username);

if ((!$first_name) || (!$last_name) || (!$email_address) || (!$username)){
echo 'You did not submit the following required information! <br />';
if(!$first_name){
echo "First Name is a required field. Please enter it below.<br />";
}
if(!$last_name){
echo "Last Name is a required field. Please enter it below.<br />";
}
if(!$email_address){
echo "Email Address is a required field. Please enter it below.<br />";
}
if(!$username){
echo "Desired Username is a required field. Please enter it below.<br />";
}

// include 'join_form.php'; // Show the form again!
/ End the error checking and if everything is ok, we'll move on to
creating the user account /
exit(); // if the error checking has failed, we'll exit the script!
}

[Weedpacket]

Well the obvious problem is that you're not stopping to look if a form is being submitted before you're trying to validate it. Try wrapping the validation code up with if($_POST['foo']){....} where foo would be the name of the submit button.

Also, it's much better to check to see if a posted variable is empty using empty(); the way you're doing it can cause errors later on. So that's if(!empty($_POST['foo']){....}.

[alanzhao]

I prefer to do it this way with form validation,

// name of the submit botton
if ($_POST['submit']) {

  // validate form here
  $first_name = stripslashes(trim($_POST['first_name']));
  $last_name = stripslashes(trim($_POST['last_name']));
  $email_address = stripslashes(trim($_POST['email_address']));
  $username = stripslashes(trim($_POST['username']));

  if ($first_name == "")
    $error[] = "First name is required";
  if ($last_name == "")
    $error[] = "Last name is required";
  if ($email_address == "")
    $error[] = "Email address is required";
  if ($username == "")
    $error[] = "Username is required";

  if (sizeof($error) > 0) {
    echo "You didn't submit the following requirement!<br>";
    for ($i = 0; $i < sizef($error); $i++) {
      echo $error[$i]."<br>";
    }
  }

} else {

  // display form here
  require('join_form.php');

}

hope it helps! :rolleyes:

[Hoangsta]

i have to upgrade my php version soon. but i would like to thank you for taking ur time to code it out. anyways, what's the difference between my codes and urs? besides the "submit" thing...

[chet23]

I stumbled across this thread while looking for something I could use to validate ZIP codes entered into a form field. The only thing the form does is return addresses based on ZIP.

Here's what I'm using now:

if ($_POST['zipInput'] === NULL || $_POST['zipInput'] === "")
    die('Invalid ZIP code entered.');
if (empty($_POST['zipInput']))
    die('Invalid ZIP code entered.');

I'm sure this isn't the tastiest way of getting this to work for me - if only it did work..

If the form field is left empty, it works.
If the form is entered '0', it works.
But if someone enters empty spaces, the validation no worky.

What am i missing?

I need to have this work with both US & Canadian ZIPS.

Thanks..

-Chet

[Hoangsta]

if ($_POST['zipInput'] === NULL || $_POST['zipInput'] === "") 
    die('Invalid ZIP code entered.'); 
if (empty($_POST['zipInput'])) 
    die('Invalid ZIP code entered.');

try it with this

if ($_POST['zipInput'] == NULL || $_POST['zipInput'] == NULL)) 
    die('Invalid ZIP code entered.'); 
if (empty($_POST['zipInput'])) 
    die('Invalid ZIP code entered.');

hope that help. it's really early for my brain...let me know if it doesn't work so i can think about it...

[laserlight]

Do what Weedpacket says, and use [man]empty/man or [man]isset/man.

[fr0stx]

im having this same problem... after click submit.. its nothing happen and it don't send an email?