Can't push item into empty array

Question

I have this function where when I click a button I loop over some divs and get the id of that div along with some checkbox inputs id's if they are marked. To store this values I have a dictionary which is suppose to store the items like this:

{
   1: [4,5],
   2: [2,3]
}

The keys represent the id's of each div and the values are the checkbox items id's that are clicked.

Full code:

$(document).on('click', '.trimite-rasp', function(){
    var formData = {}
    $.each($('.single-question'), function(){
        var listIds = [];
        var qId = $(this).attr('qid');
        console.log(qId);
        $.each($('.form-check-input'), function(){
            if($(this).is(':checked')){
                var thisId = ($(this).attr('aid'));
                $(listIds).push(thisId);
            }
        });

        formData[qId] = listIds;
    });
    console.log(formData);

});

The only problem here is that listIds array is always empty. I can't push the values to that array, and I'm sure the checkbox id is correct because I console.log it and it's taken correctly.

If you had your Console tab of your developer's tools open, you'll most likely see an error something to the effect of push is not a method of JQuery. — Scott Marcus
– Scott Marcus, Commented Jan 30, 2018 at 16:35

Eddie · Accepted Answer · 2018-01-30 16:34:25Z

3

It is because you are using the array as a jQuery selector

use

var thisId = $(this).attr('aid'); 
listIds.push(thisId);

instead of

 $(listIds).push(thisId);

answered Jan 30, 2018 at 16:34

Eddie

26.8k6 gold badges39 silver badges59 bronze badges

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

2 Comments

Ovidiu G Over a year ago

That was the problem. Thanks!

Eddie Over a year ago

Happy to help :)

Rafeeq Mohamed · Accepted Answer · 2018-01-30 16:49:36Z

0

Try this

listIds.push(thisId);

answered Jan 30, 2018 at 16:49

Rafeeq Mohamed

2002 gold badges4 silver badges17 bronze badges

Collectives™ on Stack Overflow

Can't push item into empty array

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related