array push function not working on google chrome

Question

I want to fetch multiple checkboxes values from one div. My code executes successfully on firefox but in other browsers it doesn't work. My code looks like

var amenity_array = [];
var listofParameters = $("#room-amenity input:checkbox");
for (var index in listofParameters) {
    if ($(listofParameters[index]).attr('checked')) {
        var ste = $(listofParameters[index]).attr('value');
        amenity_array.push(ste);
    }
}
alert(amenity_array);

in the above code amenity_array alerts within the braces but out of this it doesn't work on chrome.

It also doesn't work. I want to use these amenities in another place. — RAVI PRAKASH AWASTHI
– RAVI PRAKASH AWASTHI, Commented Aug 27, 2015 at 6:35

Community · Accepted Answer · 2017-05-23 10:26:39Z

2

Couple of suggestions/bugs:

Make sure your selector is correct to select checkboxes
Use :checked to select only the checkboxes that are checked
Don't use for...in for looping over array
You can use each() to get the checked checkboxes and add them in your array
Make sure that at-least one checkbox is selected, otherwise the array will have no elements in it

Code:

var amenity_array = [];

$('#room-amenity input:checkbox:checked').each(function() {
    amenity_array.push($(this).val());
});

console.log(amenity_array);

edited May 23, 2017 at 10:26

CommunityBot

11 silver badge

answered Aug 27, 2015 at 6:32

Tushar

87.4k21 gold badges164 silver badges182 bronze badges

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

1 Comment

Tushar Over a year ago

@RAVIPRAKASHAWASTHI You need to open browser console to see result

Collectives™ on Stack Overflow

array push function not working on google chrome

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related