How to push list of objects into array in Javascript

Question

I am getting list of objects as below,

const input = [
  {cases: 162151, recovered: 68, time: 1627362000000},
  {cases: 159569, recovered: 39, time: 1627448400000},
  {cases: 150654, recovered: 97, time: 1627534800000},
  {cases: 140750, recovered: 98, time: 1627621200000},
  {cases: 163394, recovered: 69, time: 1627707600000},
  {cases: 160380, recovered: 73, time: 1627794000000}
];

I want to store this in another output array.

let output = [];
let outputElements = {};
    
input.forEach(function(inputElements){    
    outputElements.Time = inputElements.time;
    outputElements.casesCount= inputElements.cases;
    outputElements.recoverCount = inputElements.recovered;    
});
    
output.push(outputElements);

Here, output I am getting only the last object only. {cases: 160380, recovered: 73, time: 1627794000000}

Here, where I am missing to fill the objects properly in output. Thanks in advance.

move output.push(outputElements) inside the loop as the last line. And when you are pushing it, use this sentence 'output.push(Object.assign({}, outputElements)' — Sathyesh
– Sathyesh, Commented Aug 3, 2021 at 6:35

Taco · Accepted Answer · 2021-08-03 14:02:07Z

2

What's wrong if you cloned the input array using the map method:

var input = [
  {cases: 162151, recovered: 68, time: 1627362000000},
  {cases: 159569, recovered: 39, time: 1627448400000},
  {cases: 150654, recovered: 97, time: 1627534800000},
  {cases: 140750, recovered: 98, time: 1627621200000},
  {cases: 163394, recovered: 69, time: 1627707600000},
  {cases: 160380, recovered: 73, time: 1627794000000}
];


var output = input.map(({ 
  time: Time, 
  cases: casesCount, 
  recovered: recoverCount
}) => ({Time, casesCount, recoverCount}));


console.log(output)

edited Aug 3, 2021 at 14:02

Taco

2,9531 gold badge20 silver badges55 bronze badges

answered Aug 3, 2021 at 6:51

Rami Assi

1,1712 gold badges12 silver badges21 bronze badges

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

1 Comment

kstepien Over a year ago

You can also use here destructing syntax for clean code. Additionally, you can remove return from the code.

kstepien · Accepted Answer · 2021-08-03 07:37:02Z

1

You need to move outputElements = {}; and output.push(outputElements); inside forEach

const input = [
  { cases: 162151, recovered: 68, time: 1627362000000 },
  { cases: 159569, recovered: 39, time: 1627448400000 },
  { cases: 150654, recovered: 97, time: 1627534800000 },
  { cases: 140750, recovered: 98, time: 1627621200000 },
  { cases: 163394, recovered: 69, time: 1627707600000 },
  { cases: 160380, recovered: 73, time: 1627794000000 }
];

let output = [];

input.forEach(({ time: Time, cases: casesCount, recovered: recoverCount }) =>
  output.push({ Time, casesCount, recoverCount })
);

console.log(output);

edited Aug 3, 2021 at 7:37

kstepien

1,23111 silver badges19 bronze badges

answered Aug 3, 2021 at 6:38

Nikola Pavicevic

23.6k9 gold badges30 silver badges52 bronze badges

1 Comment

kstepien Over a year ago

in that case you can move time, casesCount, recoverCount to the brackets {}

Vadivel A · Accepted Answer · 2021-08-03 06:41:57Z

0

I got it. Thanks for the comments also.

outputElements = {}; and output.push(outputElements); needs to be inside the for loop. Not outside.

corrected code as below:

output = [];    

input.forEach(function(inputElements){
    outputElements = {};
    outputElements.Time = inputElements.time;
    outputElements.casesCount= inputElements.cases;
    outputElements.recoverCount = inputElements.recovered; 
    output.push(outputElements);   
});

answered Aug 3, 2021 at 6:41

Vadivel A

517 bronze badges

2 Comments

kevinSpaceyIsKeyserSöze Over a year ago

You can actually accept your own answer as a solution if you want to

Bekim Bacaj Over a year ago

how nice, you made it exactly like mine... minus readability.

Bekim Bacaj · Accepted Answer · 2021-08-03 06:44:28Z

0

Here, output I am getting only the last object only. {cases: 160380, recovered: 73, time: 1627794000000}

That's because you are overwriting the outputElements object !

You will need to make the outputElements = {}; statement inside the forEach loop

input.forEach(function(inputElements){  

    outputElements = {};

    outputElements.Time = inputElements.time;
    outputElements.casesCount= inputElements.cases;
    outputElements.recoverCount = inputElements.recovered;

    output.push(outputElements);
});

just as well as the push, before creating the new object to be fed into the array.

answered Aug 3, 2021 at 6:44

Bekim Bacaj

6,0152 gold badges27 silver badges26 bronze badges

Comments

Ingadi · Accepted Answer · 2021-08-03 06:49:04Z

You're only getting the last object {cases: 160380, recovered: 73, time: 1627794000000} because you are overwriting all previous values of outputElements with the forEach loop before pushing to your output so you end up with only one value. You can use map instead of forEach

output = input.map(inputElements => {
  const outputElements = {};
  outputElements.Time = inputElements.time;
  outputElements.casesCount = inputElements.cases;
  outputElements.recoverCount = inputElements.recovered;
  return outputElements;
});

or you could just simply move output.push inside the forEach loop

input.forEach(function (inputElements) {
  const outputElements = {};
  outputElements.Time = inputElements.time;
  outputElements.casesCount = inputElements.cases;
  outputElements.recoverCount = inputElements.recovered;
  output.push(outputElements);
});

Collectives™ on Stack Overflow

How to push list of objects into array in Javascript

5 Answers 5

1 Comment

1 Comment

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

1 Comment

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related