Validation for multiple textarea in javascript

Question

I have a page which has 2 text areas and wanted to validate each text area if the characters being typed is more than max length upon clicking submit button. Problem with the code below when clicking submit it can only detect if textarea1 is more than max length but not the other textarea. What I wanted to do is it can determine which textarea is exceeding maxlength so users can correct it. Thanks in advance.

 <html>
<head>

<script>
function testing()
{
    var txtlength=document.getElementById("txt1").value.length;
    if(txtlength>50)  
    {
        alert("Max1 length reached ");
        return false; 
    }
    else
    {
        return true; 
    }

    var txtlength1=document.getElementById("txt2").value.length;
    if(txtlength1>50)  
    {
        alert("Max2 length reached ");
        return false;
    }
    else
    {
        return true;
    }

}
</script>
</head>
<body>
<form action="test1.html" method="post" onsubmit="return testing();">
    <textarea id="txt1"></textarea>
    <textarea id="txt2"></textarea>
    <input type="submit" value="Submit"/>
</form>
</body>
</html>

return exits the function. The second block of code is never reached. — elclanrs
– elclanrs, Commented Feb 20, 2014 at 3:39

andri · Accepted Answer · 2014-02-20 03:49:59Z

2

the return true statement should only be done once, at the end of the code.

the final code should looks something like this :

<script>
function testing()
{
    var txtlength=document.getElementById("txt1").value.length;
    if(txtlength>50)  
    {
        alert("Max1 length reached ");
        return false; 
    }

    var txtlength1=document.getElementById("txt2").value.length;
    if(txtlength1>50)  
    {
        alert("Max2 length reached ");
        return false;
    }

    return true;
}
</script>

answered Feb 20, 2014 at 3:49

andri

1,0211 gold badge9 silver badges16 bronze badges

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

Comments

flx · Accepted Answer · 2014-02-20 04:31:31Z

2

function testing()
{
    var txtlength=document.getElementById("txt1").value.length;    
    var txtlength1=document.getElementById("txt2").value.length;
    if(txtlength>50)  
    {
        alert("Max1 length reached ");
        return false; 
    }
    else if(txtlength1>50)  
    {
        alert("Max2 length reached ");
        return false;
    }
        return true;  
}

edited Feb 20, 2014 at 4:31

flx

14.2k11 gold badges59 silver badges73 bronze badges

answered Feb 20, 2014 at 4:14

Raja Danish

2351 silver badge7 bronze badges

1 Comment

dimas Over a year ago

Thanks Raja but i have to give the points to andri since he answered first.

ltalhouarne · Accepted Answer · 2014-02-20 03:41:36Z

1

Change your following function:

var txtlength=document.getElementById("txt1").value.length;
    if(txtlength>50)  
    {
        alert("Max1 length reached ");
        return false; 
    }
    else
    {
        return true; 
    }

    var txtlength1=document.getElementById("txt2").value.length;
    if(txtlength1>50)  
    {
        alert("Max2 length reached ");
        return false;
    }
    else
    {
        return true;
    }

to:

var txtlength=document.getElementById("txt1").value.length;
        if(txtlength>50)  
        {
            alert("Max1 length reached ");
            return false; 
        }
        else
        {
            var txtlength1=document.getElementById("txt2").value.length;
            if(txtlength1>50)  
            {
            alert("Max2 length reached ");
            return false;
            }
            else
            {
            return true;
            }
        }

answered Feb 20, 2014 at 3:41

ltalhouarne

4,6382 gold badges25 silver badges32 bronze badges

2 Comments

elclanrs Over a year ago

How about an else if statement?

dimas Over a year ago

Thanks BelgianMyWaffle

RobG · Accepted Answer · 2014-02-20 04:12:04Z

1

Rather than hard–coding the form ID, pass a reference to the from from the listener:

<form ... onsubmit="return testing(this);">

In the function, loop over the form's controls and check them:

function testing(form) {
  var control;

  for (var i=0, iLen=form.elements.length; i<iLen; i++) {
    control = form.elements[i];

    if (control.tagName.toLowerCase() == 'textarea' && control.value.length > 50) {
       alert(control.name + ' must be 50 characters or less');
       return false;
    }
  }
}

}

answered Feb 20, 2014 at 4:12

RobG

148k32 gold badges180 silver badges216 bronze badges

1 Comment

dimas Over a year ago

Thanks Robg this is an interesting solution :)

stacktogo · Accepted Answer · 2014-02-20 04:53:08Z

0

Another way to do is the same as Raja Dani post, to place a new var called errormessage as empty string above txtlength,txtlength1, replace the alert with adding those messages in errormessage var, output alert if the errormessage length > 0 and false else true? :)

That is just not like the best solution but another way to solve it.

I am new here, tried to add posts on edit (you guys make it so complex) to only add a comment and not edit it cause I cannot comment other posts as I have low rep.

answered Feb 20, 2014 at 4:53

stacktogo

456 bronze badges

1 Comment

dimas Over a year ago

in time you'll gain more points so. They both have the same answer with andri

Collectives™ on Stack Overflow

Validation for multiple textarea in javascript

5 Answers 5

Comments

1 Comment

2 Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

1 Comment

2 Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related