array iteration in jquery

Question

This is my code

$('select.more-search').each(function(i, e){ 
    more_srch[i] = $(this).attr('name'); 
    $('#'+more_srch[i]+' :selected').each(function(j, selected){ 
      more_sel[i][j] = $(selected).val(); 
    });
});

It shows error in console as TypeError: more_sel[i] is undefined

How to remove this error?

What exactly you need? Have you defined more_sel?

Satpal
– Satpal

2015-07-14 07:53:20 +00:00
Commented Jul 14, 2015 at 7:53 — Satpal
– Satpal, Commented Jul 14, 2015 at 7:53
what are you trying to do here

Arun P Johny
– Arun P Johny

2015-07-14 07:57:19 +00:00
Commented Jul 14, 2015 at 7:57 — Arun P Johny
– Arun P Johny, Commented Jul 14, 2015 at 7:57

Barmar · Accepted Answer · 2015-07-14 08:04:28Z

1

You need to initialize more_sel[i] to an empty array before you can assign to elements.

$('select.more-search').each(function(i, e){ 
    more_srch[i] = $(this).attr('name'); 
    more_sel[i] = [];
    $('#'+more_srch[i]+' :selected').each(function(j, selected){ 
      more_sel[i][j] = $(selected).val(); 
    });
});

Instead of using .each(), you could use .map():

 $('select.more-search').each(function(i, e){ 
    more_srch[i] = $(this).attr('name'); 
    more_sel[i] = $('#'+more_srch[i]+' :selected').map(function(j, selected){ 
      return $(selected).val(); 
    }).get(); // use .get() to turn returned jQuery object into normal array
});

answered Jul 14, 2015 at 8:04

Barmar

789k57 gold badges554 silver badges669 bronze badges

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

Collectives™ on Stack Overflow

array iteration in jquery

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related