How to check multiple function in JavaScript

Question

I have many functions written in JavaScript such as userName(), password() etc. But on submitting on the form, I have a single function return formValidator();.

I want function to call all the other functions and check whether they are returning false or not. If they all are true than it should return true, otherwise it should return false.

function userName(){
      var x=document.forms["signupform"]["firstname"].value;
            if(x=="" ||x==null){
           document.getElementById("firstname").style.border="1px solid #F50A0A";
           document.getElementById("firstname").style.boxShadow="2px 2px 5px #F50A0A";
           return false;
      }else {return true;}
}

function password(){
 var z=document.forms["signupform"]["password"].value;
      if(z=="" ||z==null){
            document.getElementById("password").style.border="1px solid #F50A0A";
           document.getElementById("password").style.boxShadow="2px 2px 5px #F50A0A";
            return false;
      }else {return true;}
}

What have you tried?

j08691
– j08691

2012-05-30 15:36:33 +00:00
Commented May 30, 2012 at 15:36 — j08691
– j08691, Commented May 30, 2012 at 15:36

James Allardice · Accepted Answer · 2012-05-30 15:36:31Z

6

You can call each function, using the && operator to indicate that each one must return something that evaluates to true. If any one of them returns something that evaluates to false, the entire statement will evaluate to false:

function formValidator() {
    return userName() && password() && somethingElse() && etc();
}

answered May 30, 2012 at 15:36

James Allardice

166k22 gold badges335 silver badges316 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

James Allardice Over a year ago

What do you mean by "not working"? What happens? Any errors in the JavaScript console?

Bergi · Accepted Answer · 2012-05-30 15:37:58Z

1

var validators = [userName, password, ...];
function formValidator() {
    return validators.every(function(fn){return fn();});
}

or just use the AND operator:

function formValidator() {
    return userName() && password() && ...;
}

answered May 30, 2012 at 15:37

Bergi

671k162 gold badges1k silver badges1.5k bronze badges

Comments

rlemon · Accepted Answer · 2012-05-30 17:06:12Z

0

It should be as simple as:

function formValidator()
{
    var username = userName();
    var password = password();

    return username && password; // or just return userName() && password();
}

edited May 30, 2012 at 17:06

rlemon

17.7k14 gold badges96 silver badges126 bronze badges

answered May 30, 2012 at 15:37

brenjt

16.3k14 gold badges80 silver badges120 bronze badges

2 Comments

rlemon Over a year ago

if( condition_which_returns_boolean_true_or_false ) { return true is redundant. just return the condition evaluation.

rlemon Over a year ago

yes it is, I just wanted you to update it to remove the redundancy. I am not one of the downvoters.

Collectives™ on Stack Overflow

How to check multiple function in JavaScript

3 Answers 3

1 Comment

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related