Refactor array of objects

Question

The below code will have input as array of objects and I would like to convert into a different format.

The below code works fine but I need a more refactored shorter format of what I am trying to achieve.

var res =  {"matchObject":"{\"data\":[{\"id\":\"jack1\",\"firstname\":\"jack\",\"lastname\":\"hudson\",\"dob\":\"1990-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":null,\"orgid\":\"001\"},{\"id\":\"jack2\",\"firstname\":\"Jack\",\"lastname\":\"Clinton\",\"dob\":\"1991-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":\"+16464922600\",\"orgid\":\"002\"}]}"};
var parsedObj = JSON.parse(res.matchObject);

var res = [];
for(var key in parsedObj.data){
    var emailObj = {};
    var phoneObj = {}
    if(parsedObj.data[key].email !== null){
        emailObj.matchedRes = parsedObj.data[key].email;
        emailObj.id = parsedObj.data[key].id;
        emailObj.type = "email";
        res.push(emailObj);
    }
    if(parsedObj.data[key].phone !== null){
        phoneObj.matchedRes = parsedObj.data[key].phone;
        phoneObj.id = parsedObj.data[key].id;
        phoneObj.type="phone";
        res.push(phoneObj);
    }  
}
console.log(res);

Desired output:

[ { matchedRes: '[email protected]', id: 'jack1', type: 'email' },
  { matchedRes: '[email protected]', id: 'jack2', type: 'email' },
  { matchedRes: '+16464922600', id: 'jack2', type: 'phone' } ]

In the above code separate objects are created with phone and email for same id.

Seems pretty optimized to me :)

Silvio Biasiol
– Silvio Biasiol

2019-02-14 08:48:31 +00:00
Commented Feb 14, 2019 at 8:48 — Silvio Biasiol
– Silvio Biasiol, Commented Feb 14, 2019 at 8:48

sjahan · Accepted Answer · 2019-02-14 19:00:52Z

3

Here is a solution!

I just did a generic reducer, and then I use it on phone and email.

Then, I just spread the result of both calls to the result array :)

var res =  {"matchObject":"{\"data\":[{\"id\":\"jack1\",\"firstname\":\"jack\",\"lastname\":\"hudson\",\"dob\":\"1990-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":null,\"orgid\":\"001\"},{\"id\":\"jack2\",\"firstname\":\"Jack\",\"lastname\":\"Clinton\",\"dob\":\"1991-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":\"+16464922600\",\"orgid\":\"002\"}]}"};
var parsedObj = JSON.parse(res.matchObject);

const extractData = (obj, type) => obj.reduce((acc, elt) => (
  elt[type] && acc.push({matchedRes: elt[type], id: elt.id, type: type})
, acc),[]);

const result = [...extractData(parsedObj.data, 'email'), ...extractData(parsedObj.data, 'phone')];

console.log(result);

Hope this helps, please do not hesitate to comment if you have any question ;)

edited Feb 14, 2019 at 19:00

answered Feb 14, 2019 at 8:46

sjahan

5,9804 gold badges24 silver badges46 bronze badges

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

5 Comments

CertainPerformance Over a year ago

Best not to abuse the conditional operator as a replacement for if. The conditional operator is when you need a resulting expression - if you don't need an expression, the conditional operator is not appropriate

sjahan Over a year ago

I agree with you, it is often less readable and only appropriate in some cases. But as OP asked for a shorter format... :) I gave my best to the shortest version!

Code Maniac Over a year ago

You can destructuring assignment to make it even shorted stackoverflow.com/questions/54605286/…

user804401 Over a year ago

Thanks you for the overwhelming response. Learned lot of refactoring techniques with Reduce today.

sjahan Over a year ago

@user804401 You should google something like javascript performance map vs reduce, but from my understanding, this will be quite the same. The performance will mainly depends on your mapping/reducing function, but in the end, both functions do the same: iterating on an array, executing the function to map/reduce, then adding the result of this function in a target object. Maybe reduce is slightly heavier (not even sure about that), but I think that when you come to that kind of fine tuning issue, you will probably want to modify your data model more than using something else than reduce!

Code Maniac · Accepted Answer · 2019-02-14 09:00:57Z

2

You can use reduce with destructuring assignment . and check if email or phone is there than add a object accordingly

var res =  {"matchObject":"{\"data\":[{\"id\":\"jack1\",\"firstname\":\"jack\",\"lastname\":\"hudson\",\"dob\":\"1990-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":null,\"orgid\":\"001\"},{\"id\":\"jack2\",\"firstname\":\"Jack\",\"lastname\":\"Clinton\",\"dob\":\"1991-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":\"+16464922600\",\"orgid\":\"002\"}]}"};
var parsedObj = JSON.parse(res.matchObject);

let op = parsedObj.data.reduce((out,{id,email,phone})=>{
  if(email){
    out.push({matchedRes:email,id,type:`email`})
  } 
  if(phone){
  out.push({matchesRes:phone,id,type:`phone`})
  }
  return out
},[])

console.log(op)

If you want to see more use cases of You can destructuring assignment and it's uses you can check this one out

edited Feb 14, 2019 at 9:00

answered Feb 14, 2019 at 8:46

Code Maniac

37.9k5 gold badges44 silver badges65 bronze badges

1 Comment

Code Maniac Over a year ago

Read here hackernoon.com/…

Terry Lennox · Accepted Answer · 2019-02-14 08:51:19Z

2

This should be possible with reduce:

var res =  {"matchObject":"{\"data\":[{\"id\":\"jack1\",\"firstname\":\"jack\",\"lastname\":\"hudson\",\"dob\":\"1990-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":null,\"orgid\":\"001\"},{\"id\":\"jack2\",\"firstname\":\"Jack\",\"lastname\":\"Clinton\",\"dob\":\"1991-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":\"+16464922600\",\"orgid\":\"002\"}]}"};
var parsedObj = JSON.parse(res.matchObject);
const keyFields = ["email", "phone"];

let result = parsedObj.data.reduce((acc, val) => {
    keyFields.forEach(k => {
       if (val[k]) acc.push({ matchedRes: val.email, id: val.id,  type: k});
    });
    return acc;
}, []);

console.log("Result: ", result);

answered Feb 14, 2019 at 8:51

Terry Lennox

30.9k5 gold badges36 silver badges45 bronze badges

4 Comments

Code Maniac Over a year ago

You can destructuring assignment to make it even shorted stackoverflow.com/questions/54605286/…

Terry Lennox Over a year ago

I saw that it your answer. Nice solution!

Terry Lennox Over a year ago

The only slight issue with that is the OP wants the phone, email fields mapped to a matchedRes field, right?

Code Maniac Over a year ago

Yeah, fixed that key naming mate. Thanks for pointing out miss

Silvio Biasiol · Accepted Answer · 2019-02-14 09:30:42Z

If you are looking for a little shorter code but still easy to read for anybody:

var res =  {"matchObject":"{\"data\":[{\"id\":\"jack1\",\"firstname\":\"jack\",\"lastname\":\"hudson\",\"dob\":\"1990-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":null,\"orgid\":\"001\"},{\"id\":\"jack2\",\"firstname\":\"Jack\",\"lastname\":\"Clinton\",\"dob\":\"1991-01-01T00:00:00.000Z\",\"email\":\"[email protected]\",\"phone\":\"+16464922600\",\"orgid\":\"002\"}]}"};
var parsedObj = JSON.parse(res.matchObject);

var res = [];

Object.entries(parsedObj.data).forEach(el => {
    el = el[1]
    if(el.email !== null)
      res.push({
        matchedRes: el.email,
        id: el.id,
        type: "email"
      })
    if(el.phone !== null)
       res.push({
        matchedRes: el.phone,
        id: el.id,
        type: "phone"
    })
})

console.log(res);

Collectives™ on Stack Overflow

Refactor array of objects

4 Answers 4

5 Comments

1 Comment

4 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

5 Comments

1 Comment

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related