Consider the following list of objects:
const blogs = [
{
title: "React patterns",
author: "Michael Chan",
url: "https://reactpatterns.com/",
likes: 7,
},
{
title: "Go To Statement Considered Harmful",
author: "Edsger W. Dijkstra",
likes: 5,
},
{
title: "Canonical string reduction",
author: "Edsger W. Dijkstra",
likes: 12,
},
{
title: "First class tests",
author: "Robert C. Martin",
likes: 10,
},
{
title: "TDD harms architecture",
author: "Robert C. Martin",
likes: 0,
},
{
title: "Type wars",
author: "Robert C. Martin",
likes: 2,
}
]
what I need to do, is write a function which returns the blog object, with the author that has the most blogs that he wrote, in the following format (which is also the expected result in this case):
{
author: "Robert C. Martin",
blogs: 3
}
This is the first solution I came up with, involving 2 reducers
const mostBlogs = blogs => {
const formatted_blogs = blogs.reduce((acc, cur_blog)=>{
if (!acc[cur_blog.author]){
acc[cur_blog.author] = {
author: cur_blog.author, blogs: 1
}
} else {
acc[cur_blog.author].blogs++;
}
return acc;
}, {});
return Object.keys(formatted_blogs).reduce((acc, cur)=>{
if (formatted_blogs[cur].blogs > (acc.blogs||0))
return ( {...formatted_blogs[cur]} )
return acc;
}, {});
}
I was wondering how can I make it better, using only one reducer which would be more efficient and smarter. While I was typing out the question here, I actually came up with another solution using only 1 reducer. Would love to hear your opinions of it, could this be optimized? Is it smart/efficient enough? or can I do something different here which would be a better practise?
const mostBlogs = blogs => {
const { most_blogs } = blogs.reduce((acc, cur_blog)=>{
// if author doesn't exist in acc.all, add a new formatted obj
if (!acc.all[cur_blog.author])
acc.all[cur_blog.author] = { author: cur_blog.author, blogs: 1 };
// if exists, simply increment the blogs prop count in acc.all
else
acc.all[cur_blog.author].blogs++;
// check if cur_blog blogs prop in acc.all is higher than
// existing one in acc.most_blogs blogs prop, overwrite obj if so
if (acc.all[cur_blog.author].blogs > (acc.most_blogs.blogs||0))
acc.most_blogs = acc.all[cur_blog.author];
return acc;
}, {all: {}, most_blogs: {} });
return most_blogs;
}