issue merging two Javascript object arrays

Question

I have an object array. I just want to add one more object "WITH" an index. I'm not sure how it's done.

This is what I've tried:

//RAW DATA:
this.fetchedData = [{
  "startMinute": 0, //Remove
  "annotation": "All Day",
  "twelveHourFormat": "12am-11:59pm", //Remove
  "timeRange": "00:00-23:59",
  "endMinute": 1439 //Remove
}, {
  "startMinute": 300, //Remove
  "annotation": "Morning",
  "twelveHourFormat": "5am-8:59am", //Remove
  "timeRange": "05:00-08:59",
  "endMinute": 539 //Remove
}]

var newTimeRanges = [];
var temp = []; //Final array(assuming i need one)

newTimeRanges = _.each(this.fetchedData, function(time) {
  delete time.startMinute;
  delete time.twelveHourFormat;
  delete time.endMinute;
});

//DATA AFTER REMOVAL

newTimeRanges = {
  0: {
    annotation: "All Day",
    timeRange: "00:00-23:59"
  },
  1: {
    annotation: "Evening",
    timeRange: "16:00-18:00"
  }
}

//DATA TO BE MERGED
var dirtyData = [{
  "timeRange": "3am-6am",
  "annotation": earlymorning
}];

//Essentially (timeRange+DirtyData)


//Expected Output to be sent for backend
{
  "times": [{
      "annotation": "All Day",
      "timeRange": "00:00-23:59"
    }, {
      "annotation": "Morning",
      "timeRange": "05:00-08:59"
    },
    //Add my new Data here
  ]
}

Now, I want to add or remove another object to that array.

temp.push(newTimeRanges);
temp.push(dirtyData);

This does not merge the records. Instead creates two objects one object and another array.

How can I merge. newTimeRanges with DirtyData. (Like I want 3: "Object")

Is there any efficient way of doing this?

Thanks

why not temp=newTimeRanges; temp.push(dirtyData[0]);//or loop through it — depperm
– depperm, Commented Aug 26, 2016 at 18:42
please add the raw data and the wanted result, like an object literal. — Nina Scholz
– Nina Scholz, Commented Aug 26, 2016 at 18:42
newTimeRanges started as an array, but when you logged it it became an object. How did you populate it? — Barmar
– Barmar, Commented Aug 26, 2016 at 18:45
Why are you initializing newTimeRanges with an empty array if you're just going to immediately replace it with an object? — Barmar
– Barmar, Commented Aug 26, 2016 at 18:52
@Barmar when I used underscoreJS loop to go through the loop and remove few things that converted to object. I had to delete bunch of stuff and store it back. Now I want to append to the newly created object. — TechnoCorner
– TechnoCorner, Commented Aug 26, 2016 at 18:54

Barmar · Accepted Answer · 2016-08-26 18:49:20Z

2

You need to push each property of newTimeRanges separately onto the temp array. Then you can concatenate dirtyData.

$.each(newTimeRanges, function(key, obj) {
    temp.push(obj);
}
temp = temp.concat(dirtyData);

Or you could have made newTimeRanges an array instead of an object, and then you could write:

newTimeRanges = [
    {
        annotation:"All Day",
        timeRange:"00:00-23:59"},
    {
        annotation:"Evening",
        timeRange:"16:00-18:00"}
];

temp = newTimeRanges.concat(dirtyData);

answered Aug 26, 2016 at 18:49

Barmar

789k57 gold badges555 silver badges669 bronze badges

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

Comments

Vinicius · Accepted Answer · 2016-08-26 19:17:16Z

1

If you want to merge 2 arrays into a third array you must use concat, like this:

var temp = newTimeRanges.concat(dirtyData);

See: Array.prototype.concat()

Changing your code to work:

//RAW DATA:
var fetchedData = [{
  "startMinute": 0, //Remove
  "annotation": "All Day",
  "twelveHourFormat": "12am-11:59pm", //Remove
  "timeRange": "00:00-23:59",
  "endMinute": 1439 //Remove
}, {
  "startMinute": 300, //Remove
  "annotation": "Morning",
  "twelveHourFormat": "5am-8:59am", //Remove
  "timeRange": "05:00-08:59",
  "endMinute": 539 //Remove
}];

fetchedData.forEach(function(time) {
  delete time.startMinute;
  delete time.twelveHourFormat;
  delete time.endMinute;
});

console.log(fetchedData);

//DATA TO BE MERGED
var dirtyData = [{
  "timeRange": "3am-6am",
  "annotation": "earlymorning"
}];

var temp = fetchedData.concat(dirtyData);

console.log(temp);

edited Aug 26, 2016 at 19:17

answered Aug 26, 2016 at 18:49

Vinicius

1,1631 gold badge12 silver badges22 bronze badges

5 Comments

Vinicius Over a year ago

@Barmar The question is about arrays... Or it should be.

Barmar Over a year ago

It should be, but read the question. He's replacing the array with an object that happens to have numeric property names. But it's not an array.

Vinicius Over a year ago

Yeah, he is changing it as we speak, so I'm sticking to the title, since it seems that he is just lost with the code.

Vinicius Over a year ago

I mean, it seems that he can accomplish what he wants using arrays.

TechnoCorner Over a year ago

Thanks guys for your help. I got it to work! Sorry, If i confused you guys!

Collectives™ on Stack Overflow

issue merging two Javascript object arrays

2 Answers 2

Comments

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related