Javascript Object with Array of objects

Question

I'm wondering if it is possible to create a javascript map with array of objects as content.

My code is as follows:

//Object that replicates a Map
var itemRcptMap = {};

//Receipt Object Holder
var objReceipt = {
     id     : '',
     item   : '',
     qty    : '',
     lotNo  : ''
};

//Iterate an object result w/c is not shown in the example 
for (var i in itemRcptResult)
{
    var id = itemRcptResult[i].id;

    if(!itemRcptMap[id]){
        itemRcptMap[id] = new Array();
    }
    objReceipt.id    = arItemRcptResult[i]id;
    objReceipt.item  = arItemRcptResult[i].items;
    objReceipt.qty   = arItemRcptResult[i].quantity;
    objReceipt.lotNo = arItemRcptResult[i].test;
    itemRcptMap[id] = itemRcptMap[id].push(objReceipt);
}

I'm having the following error

Cannot find function push in object 1.

How can I correct this?

THank you.

Code Simplification:

var itemRcptMap = {};

//Receipt Object Holder
var objReceipt = {
     id     : '',
     item   : '',
     qty    : '',
     lotNo  : ''
};

var id = 5555;
    if(!itemRcptMap[id]){
        itemRcptMap[id] = new Array();
    }
    objReceipt.id    = 1;
    objReceipt.item  = 2;
    objReceipt.qty   = 3;
    objReceipt.lotNo = 4;
    itemRcptMap[id].push(objReceipt);

 //Gets the object with Key 5555
var x = itemRcptMap[5555];  
alert(x.id)

var itemRcptMap = new Array();

Rab Khan
– Rab Khan

2014-08-12 13:42:21 +00:00
Commented Aug 12, 2014 at 13:42 — Rab Khan
– Rab Khan, Commented Aug 12, 2014 at 13:42

PSL · Accepted Answer · 2014-08-12 14:12:01Z

4

When you push to an array, the return value will be the new length of the array (which you are reassigning back to the actual array) so in the next iteration you are not pushing it to the array instead you are trying to access push method of numeric value(number does not have a push method.). Push mutates the source array so just do:-

itemRcptMap[id].push(objReceipt);

You might want to move objReceipt declaration inside of the loop. Other wise you will end up getting the same object in all the arrays.

Based on your comments, possibly you are looking to create a flat map with id (not duplicated) you could do.

//Object that replicates a Map
var itemRcptMap = {};

for (var i=0, l=itemRcptResult.length; i<l; i++){ 
    var id = itemRcptResult[i].id;
    itemRcptMap[id] = itemRcptResult[i];
}

Note if itemRcptResult is an array then dont use for..in loop

edited Aug 12, 2014 at 14:12

answered Aug 12, 2014 at 13:37

PSL

124k21 gold badges257 silver badges243 bronze badges

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

4 Comments

newbie Over a year ago

It still does not work... see my edit for the simplification of my code

newbie Over a year ago

i mean, it works but i still cannot access the object. I've edited my post

PSL Over a year ago

@newbie i dont know what are you trying to do , but whatever you are doing you need to do var x = itemRcptMap[5555][0]; to access the object. Because itemRcptMap[id] is an array which has the the object that you are looking for. One more thing i would recommend is to prefer using [] over new Array();

PSL Over a year ago

@newbie are you looking for map?, then don't use push instead just do itemRcptMap[id] = objReceipt; provided you dont have id duplicated

Collectives™ on Stack Overflow

Javascript Object with Array of objects

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related