constant declarations with forEach loop

Question

What's wrong with the code? How to get a doubled value in doubleSalaries?

const salaries= [
  { amount: 10 },
  { amount: 45 },
  { amount: 90 }
];

const doubleSalaries = salaries.forEach(salaries => ({
...salary,
 amount: salary.amount * 2
}));

Does this answer your question? [JavaScript: Difference between .forEach() and .map()](https://stackoverflow.com/questions/34426458/javascript-difference-between-foreach-and-map) — Lionel Rowe, Sep 07 '20 at 19:01

Gorynych · Accepted Answer · 2020-09-07T19:15:38.953

1

You should map the array

const salaries= [
  { amount: 10 },
  { amount: 45 },
  { amount: 90 }
];

const doubleSalaries = salaries.map(salary => ({amount: salary.amount * 3}));

console.log(doubleSalaries);

edited Sep 07 '20 at 19:15

answered Sep 07 '20 at 19:02

Gorynych

100
4

Still a typo , no `salary` in callback – charlietfl Sep 07 '20 at 19:03
thnx a lot) fixed – Gorynych Sep 07 '20 at 19:05

score 1 · Answer 2 · answered Sep 07 '20 at 19:03

1

You can use the map() method:

const salaries= [
  { amount: 10 },
  { amount: 45 },
  { amount: 90 }
];

const doubleSalaries = salaries.map(x => x.amount * 3);

console.log(doubleSalaries)

answered Sep 07 '20 at 19:03

sonEtLumiere

4,461
3
8
35

constant declarations with forEach loop

2 Answers2