Get all form arrays to JS array

Question

I'm having problem to get all form array values to JS array, with one value it working fine:

          var el = document.getElementsByName('p[]');
          for (var i = 0, j = el.length; i < j; i++) {
          var elem = el[i];
          alert(elem.value);
          }

But when I try to add more its not working:

var el = document.getElementsByName('p[]', 'k[]', 'ka[]', 's[]');
              for (var i = 0, j = el.length; i < j; i++) {
              var elem = el[i];
                      alert(elem.value);
                    $.post("test.php", { 'p': [elem.value] });
                      }

Here is my form: form is dynamic (with button click I can add new line with these 4 fields:

   <form name="form">
    <input type="text" name="p[]" id=name" />
    <input type="text" name="k[]" id="quant" size="3" />
    <input type="text" name="ka[]" id="price" size="10" />
    <input type="text" name="s[]" id="sum" size="10" disabled="disabled"/><br />
    </form>

Since you want to get all elements in the form, you can also use var el = document.forms['form'].elements;. — Rob W
– Rob W, Commented Jul 10, 2012 at 20:57
@Bergi because I as I mentioned when I click add button there adds same 4 input fields — Osvalda Kazlaučiūnaitė
– Osvalda Kazlaučiūnaitė, Commented Jul 10, 2012 at 20:59
@RobW does that actually work? I'm not seeing it in on MDC anywhere: developer.mozilla.org/en/DOM/element#Properties — Matt Ball
– Matt Ball, Commented Jul 10, 2012 at 21:04

Community · Accepted Answer · 2017-05-23 11:48:12Z

3

document.getElementsByName doesn't magically take multiple arguments in the way that you're trying to use it. You need to call gEBN multiple times, once for each name (described in more detail by another answer).

Alternately, just loop over the <form>'s inputs:

var form = document.getElementsByName('form')[0];
var inputs = form.getElementsByTagName('input');
for (var i = 0, j = inputs.length; i < j; i++) {
    // snip
}

edited May 23, 2017 at 11:48

CommunityBot

11 silver badge

answered Jul 10, 2012 at 20:56

Matt Ball

361k102 gold badges655 silver badges725 bronze badges

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

3 Comments

Bergi Over a year ago

You forgot to use only the first form from the NodeList.

Osvalda Kazlaučiūnaitė Over a year ago

@Matt Ball Now its working fine, is there a way also add field name like p=value, k=value... I vant this all data to send to php file and add it to mysql.

Matt Ball Over a year ago

@OsvaldaKazlaučiūnaitė if you don't mind using jQuery, .serialize() will do all this work for you.

user1498339 · Accepted Answer · 2012-07-10 21:00:45Z

1

The method get elementsByName takes one string argument, so you should do something like this:

var listOfElements = ['p[]', 'k[]', 'ka[]', 's[]'];
for (var i=0; i<listOfElements.length; i++) {
   var el = document.getElementsByName(listOfElements[i]);
   ...
}

answered Jul 10, 2012 at 21:00

user1498339

6294 silver badges9 bronze badges

Collectives™ on Stack Overflow

Get all form arrays to JS array

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related