Combining two arrays return wrong result?

Question

I'm trying to combine two arrays in a specific format, but I cant imagine what I'm doing wrong.

This is the output result that I would like to have:

[ data: [
  {value: 100, name: 'January'},
  {value: 30, name: 'February'},
  {value: 150, name: 'March'},
  {value: 85, name: 'April'},
  {value: 60, name: 'May'},
  {value: 20, name: 'June'}
  ],
  radius: '50%' ]

This is my code:

    var sales = ["100", "30", "150", "85", "60", "20"];
    var months = ["January", "February", "March", "April", "May", "June"];
    var rad = "50%";

    var combined = sales.map(function combine(dataItem, index) {
          return {
              data: [{"value":dataItem, "name":months[index]}],
              radius: rad
          };
                }).filter(function removeEmpty(item) {
                  return item.data.length;
                });
                
    console.log(combined);

"This is the output result that I would like to have" - That would be possible, but you shouldn't misuse an array when you want an object. — Andreas
– Andreas, Commented Apr 25, 2021 at 15:45
@outis thanks for answering, yeah I have jsfiddle I have been trying to fix it for the last hour and nothing jsfiddle.net/zfo3ag08 — CaesarKvs
– CaesarKvs, Commented Apr 25, 2021 at 15:48
@SAM hello, radius will be always 50%, it's "defined value" that needs to be inside the object. However, edited the snippet. Thx — CaesarKvs
– CaesarKvs, Commented Apr 25, 2021 at 15:49

DecPK · Accepted Answer · 2021-04-25 15:51:27Z

3

Just map over it and add the value and name in the object.

var sales = ["100", "30", "150", "85", "60", "20"];
var months = ["January", "February", "March", "April", "May", "June"];

const data = sales.map((sale, i) => ({
  value: parseInt(sale),
  name: months[i],
}));

const result = {
  data,
  radius: "50%",
};

console.log(result);

answered Apr 25, 2021 at 15:51

DecPK

25.4k6 gold badges32 silver badges46 bronze badges

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

1 Comment

jrob11 Over a year ago

This is a better, more concise solution :)

jrob11 · Accepted Answer · 2021-04-25 15:58:23Z

So, in your expected output, you are treating an array like an object. Try this for your expected result:

{data: [
  {value: 100, name: 'January'},
  {value: 30, name: 'February'},
  {value: 150, name: 'March'},
  {value: 85, name: 'April'},
  {value: 60, name: 'May'},
  {value: 20, name: 'June'}
],
radius: '50%'}

As to how to achieve the result, try this:

const sales = ["100", "30", "150", "85", "60", "20"],
months = ["January", "February", "March", "April", "May", "June"];

const combine = (s, m) => {
  const arr = [];
  for (let i = 0; i < Math.min(s.length, m.length); i++)
    arr.push({ value: s[i], name: m[i] });
  return arr;
};

const dataObj = {
  data: combine(sales, months),
  radius: "50%",
};


console.log(dataObj);

Collectives™ on Stack Overflow

Combining two arrays return wrong result?

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related