[SOLVED] form validation

[40esp]

Im trying to get a form validation script to work:

 

<?php 

if ((isset($_POST['add_email']) && ($_POST['add_email'] == "")) || ((isset($_POST['select'])) && ($_POST['select'] == ""))) { 

exit();

}

else

{
echo "fgfgdfgdgdfgdfg";

}

?>

The problem is, $_POST['add_email'] can be filled with an email address, and $_POST['select'] could be blank, and it will still process echo "fgfgdfgdgdfgdfg";. even though I've said check  $_POST['select'] in my if statement.. its like its completely ignoring the first part of the script.

 

Any ideas?

[MatthewJ]

You should post the form

[ober]

Throw print_r($_POST); in front of all of that and verify your values.  My guess is that it isn't what you're expecting.

[lemmin]

It is because you are using the OR operator. You are basically only asking that one OR the other is filled in. Simply change "||" to "&&" and it will check both.

[abdfahim]

I would rather go like this

if (isset($_POST['submit_button_name'])){
   if (trim($_POST['add_email']) == "" or trim($_POST['select']) == "") { 
        exit();
   }else{
        echo "done";
   }
}

[abdfahim]

hi lemmin .. he wants to exit if either email or select field is blank ... so he should OR this, not AND. If he change OR to AND as you said, it'll exit only if both email and select field is blank, which is not the thing he wants, I guess.

[40esp]

I got it, Thanks for the help! The print_r($_POST); helped.

[ober]

It is because you are using the OR operator. You are basically only asking that one OR the other is filled in. Simply change "||" to "&&" and it will check both.

Wrong.  He wants to bail if either meets the criteria.  For your scenario, both would have to fail.

[lemmin]

Woops, sorry. I saw those as "if not equal" at first glance.

[abdfahim]

nice to see your problem solved, but remember to use TRIM() function in such case, otherwise if any blank space is there in any field, you will be bluffed.