Only JavaScript Dynamic Array Field Validation

Question

I think so there is a problem with for statement??

Adjusted code again, but not alert popup is all the time even if all the input fields got values?

Hello I am trying to validate a dynamic array of fields on a form:

<form onsubmit="return checkReq();">
    <input value="" type="hidden" name="slider[]" id=""/>
</form>

with the following JavaScript, but it doesn't work? Could you tell me please what I am doing wrong.

    <script language="javascript">
    function checkReq () {
        var boxes = document.getElementsByName("slider[]");
    var ret = true;
    for (var x = 0; x < boxes.length; x++) {

        if(boxes[x].value == '' || '0'){
            ret = false;
            break;
            } else {ret = true;} 

         }
   if (ret == false)
   {
     alert('Problem'); return ret;        
   }
}
    </script>

Rajesh Dhiman · Accepted Answer · 2014-04-28 12:39:30Z

1

I think this might help.

function checkReq () {
     var boxes = document.getElementsByName("slider[]");
var ret = true;
for (var x = 0; x < boxes.length; x++) {       
    if(boxes[x].value == '' || boxes[x].value == '0'){
        ret = false;
        break;
        } else {ret = true;} 

     }    
   if (ret == false)
   {
     alert('Problem'); return ret;        
   }
}

edited Apr 28, 2014 at 12:39

answered Apr 28, 2014 at 11:17

Rajesh Dhiman

1,8961 gold badge18 silver badges30 bronze badges

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

1 Comment

Fenix Dolores Over a year ago

Now alert is on all the time, even if all the fields got value in them ? strange

Metalstorm · Accepted Answer · 2014-04-28 10:32:15Z

0

You always return after the first loop, so it doesn't go through each element (and thus redundant), is this intended?

answered Apr 28, 2014 at 10:32

Metalstorm

3,3603 gold badges29 silver badges24 bronze badges

Comments

Anoop · Accepted Answer · 2014-04-28 10:46:23Z

0

Try this You are trying to compare element instead of it's value JSFIDDLE

function checkReq () {
    var boxes = document.getElementsByName("slider[]");

    for (var x = 0; x < boxes.length; x++) {

       if(boxes[x].value == '' || boxes[x].value == '0'){

            alert('Problem'); return false;

         } 
         else {return true;} 
    }
}

edited Apr 28, 2014 at 10:46

answered Apr 28, 2014 at 10:32

Anoop

23.2k10 gold badges65 silver badges79 bronze badges

1 Comment

Anoop Over a year ago

@FenixDolores press the submit button

Collectives™ on Stack Overflow

Only JavaScript Dynamic Array Field Validation

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related