Array concat returns empty array

Question

I'm adding several items to an array using concat in an event-handler as follows:

var selectedValues = [];
$.each($('#selected-levels').data("kendoListBox").dataSource.data(), function(i, item) {
    selectedValues.concat([ item.id ])
});

return {
    "selected" : selectedValues
};

this always returns {level-selected: Array(0)} even though I have checked that there are some items in the dataSource (by stepping through with debugger)

why don't the items appear in the array?

gurvinder372 · Accepted Answer · 2018-03-13 09:20:27Z

21

concat doesn't mutate the array, you need to set the value back to selectedValues

selectedValues = selectedValues.concat([ item.id ])

Or use push

selectedValues.push( item.id )

answered Mar 13, 2018 at 9:20

gurvinder372

68.6k11 gold badges78 silver badges98 bronze badges

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

Comments

Pac0 · Accepted Answer · 2018-03-13 09:23:27Z

1

Ironically, the answer was already hidden in your question, in your question title more exactly.

Actually concat returns the correct new array you want... you just never used the return value !

You have to be aware that the array is not modified in place, but a fresh copy is returned.

So selectedValues.concat([ item.id ]) should be replaced by selectedValues = selectedValues.concat([ item.id ]) if you want to do anything.

answered Mar 13, 2018 at 9:23

Pac0

23.3k9 gold badges76 silver badges85 bronze badges

Collectives™ on Stack Overflow

Array concat returns empty array

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related