onsubmit multiple javascript functions
Does anyone know how to onsubmit multiple functions? What I intented to do was: onsubmit checks if all these functions return true, if all of them return true, then I do the action="checked.html"
<form id="myForm" name="myForm" action="checked.html" method="post"
onsubmit="return validateName() || validatePhone() || validateAddress() ||
validateEmail()">
However, what actually happened was, the computer checks the code from left to right, and the result of the next function will replace the previous one, and so on, until the end. So, even if all the 3 first functions return false, as long as the last one is true, the computer will execute action="checked.html", which is not what I intented to do... can anyone please help me on this, it's been like 4 hours I'm trying to fix it :S Thanks!
Also, when I tried to do something like <form onsubmit="return verify1() && verify2()">
, it doesn't work, all it does is to check verify1(), and not the following ones... :(
CLARIFCATION: On submit, I want four validation functions to be called. Submit should be prevented unless all four functions return true, but all four should always be called even if some of them return false.
Solution 1:
Use &
instead of &&
.
&&
uses short-circuit evaluation and so doesn't evaluate the right-hand operand if the left-hand operand is falsy.
&
always evaluates both operands.
onsubmit="return !!(validateName() & validatePhone()
& validateAddress() & validateEmail());"
EDIT: Sorry, I forgot that &
won't return an actual boolean. Wrap the expression in parentheses and use a double !
to convert it as (now) shown. (You wouldn't need to worry if using the result in an if
test, but the return value from the event handler should be an actual boolean.)
P.S.: Here's a rather clunky demo to show that all four functions get called but produce the overall true or false result that you need: http://jsfiddle.net/nnnnnn/svM4h/1/
Solution 2:
Change your code to use AND instead of OR
onsubmit="return validateName() && validatePhone() && validateAddress() && validateEmail()"
The reason being that if you use OR once that first function returns true javascript won't bother to check the other functions. Also you don't want to use OR because if the first check was false and second is true the form will still submit since at least one of your checks is TRUE.
See fiddle to show this works, both checks execute and show an alert message but because check2 returns false the form doesn't submit - if you want play around with the return results of checks1 and check2 functions and the use of && and ||
After understanding the problem better, you need to use a single &, but because true & true
returns 1 and not true
you need to write it like this
onsubmit="return ((validateName() & validatePhone() & validateAddress() & validateEmail()) == 1)"