Convert foreach into map function in javascript

Question

This is my function in foreach loop for creating object which has property and value as word and its count, but i want to convert it in map according to es6

function harmlessRamsonNote(noteText,magazineText)
{
  var noteArr = noteText.split(' ');
  var magazineArr = magazineText.split(' ');
  var magazineObj = {};

  magazineArr.forEach(word => {
    if(!magazineObj[word])
    {
      magazineObj[word] = 0;
    }
    magazineObj[word]++;
  });

  console.log(magazineObj);
};

Efe · Accepted Answer · 2018-03-14 10:01:01Z

2

magazineArr.map((word, index, array) => {
   !magazineObj[word] ? magazineObj[word] = 0 : magazineObj[word]++;
})

answered Mar 14, 2018 at 10:01

Efe

5,9862 gold badges25 silver badges37 bronze badges

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

Comments

Suren Srapyan · Accepted Answer · 2018-03-14 10:34:47Z

2

map will return an new item for each item. Instead you can use reduce.

const magazineObj = magazineArr.reduce((acc,word) => {
   acc[word] = (acc[word] || -1) + 1;
}, {});

edited Mar 14, 2018 at 10:34

answered Mar 14, 2018 at 10:01

Suren Srapyan

68.9k14 gold badges128 silver badges119 bronze badges

4 Comments

Akash Salunkhe Over a year ago

it gives error as magazineObj not defined in this line magazineObj[word] = (magazineObj[word] || -1) + 1;

Akash Salunkhe Over a year ago

It still give error as acc[word] = (acc[word] || -1) + 1; ^ TypeError: Cannot read property 'is' of undefined I think its because in first loop value of acc is undefined though i am not sure of this anyways i really appreciate your help and response though

Suren Srapyan Over a year ago

I have not worked with is, maybe it is your error?

Akash Salunkhe Over a year ago

When i call the function with this harmlessRamsonNote('','this is the ramson the the text'); it gives that error, i.e gives error in second word of passed string

Collectives™ on Stack Overflow

Convert foreach into map function in javascript

2 Answers 2

Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related