Returning new form of array of objects using javascript

Question

My object:

const data = {
  "rows": [
    {
      "code": "ct101",
      "username": "admin",
      "subscriptionExpiresOn": "2019-11-27T09:35:38.916Z",
    },
    {
      "code": "bt101",
      "username": "admin",
      "subscriptionExpiresOn": "2016-11-27T08:20:57.297Z",
    },
  ]
}

There is a key called 'subscriptionExpiresOn' in my object inside rows array,

I want to check condition, If subscriptionExpiresOn is < new Date(). I need output like this:

// Changing the value into 'Expired'
const data = {
  "rows": [
    {
      "code": "ct101",
      "username": "admin",
      "subscriptionExpiresOn": "2019-11-27T09:35:38.916Z",
    },
    {
      "code": "bt101",
      "username": "admin",
      "subscriptionExpiresOn": "Expired",
    },
  ]
}

Here what i tried using map method:

const now = new Date();

data.rows.map((index) => {
  if (index.subscriptionExpiresOn < now) {
    return {
      subscriptionExpiresOn: 'Expired',
    };
  } else {
    return data;
  }
});

But its not working.

Nikhil Aggarwal · Accepted Answer · 2018-12-01 07:49:44Z

2

As you are trying to mutate the same array, you should use Array.forEach. And for date comparison, you will need to convert string into date object.

const now = new Date();
const data = {"rows": [{"code": "ct101","username": "admin","subscriptionExpiresOn": "2019-11-27T09:35:38.916Z"},{"code": "bt101","username": "admin","subscriptionExpiresOn": "2016-11-27T08:20:57.297Z"}]};

data.rows.forEach(o => {
  if(new Date(o.subscriptionExpiresOn) < now) o.subscriptionExpiresOn = 'Expired'
});
console.log(data.rows);

answered Dec 1, 2018 at 7:49

Nikhil Aggarwal

28.5k4 gold badges46 silver badges61 bronze badges

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

11 Comments

Sameer Over a year ago

I am getting 'null' as value inside my 'subscriptionExpiresOn' key.?

Nikhil Aggarwal Over a year ago

@MohamedSameer - What is the expected result in that case? And null as in string "null" or null

Nikhil Aggarwal Over a year ago

@MohamedSameer - What needs to be done in that case? What is the expected output?

Sameer Over a year ago

Ok wait, i think problem is in my side, will update you.

Nikhil Aggarwal Over a year ago

@MohamedSameer - There are many solutions to a problem. Choose the one that suits you. There is no problem with the above too.

|

Blue · Accepted Answer · 2018-12-01 08:07:10Z

You need a few changes. Firstly, you need to convert each row.subscriptionExpiresOn to a date object, so you can compare properly to now. Additionally, you need to return the row in addition to the updated key, so I'm using ES6 spread syntax to accomplish this:

const data = {
  "rows": [
    {
      "code": "ct101",
      "username": "admin",
      "subscriptionExpiresOn": "2019-11-27T09:35:38.916Z",
    },
    {
      "code": "bt101",
      "username": "admin",
      "subscriptionExpiresOn": "2016-11-27T08:20:57.297Z",
    },
  ]
}

const now = new Date();

const mapped = data.rows.map((row) => {
  if (new Date(row.subscriptionExpiresOn) < now) {
    return {
      ...row,
      subscriptionExpiresOn: 'Expired',
    };
  } else {
    return row;
  }
});

console.log(mapped);

Then if you're trying to preserve the structure, just return it in the original format:

return { rows: mapped };

Lazar Ljubenović · Accepted Answer · 2018-12-01 07:47:58Z

It's not working because:

the signature of map is a function whose first argument is the item, not the index;
you're returning only what you want to change. Instead, return the whole object, overwriting the old value with the new;
in the else branch, you're returning the whole original array instead of just returning the item as it is;
you're comparing a stirng with a date.

Here's the working code.

const data = {
  "rows": [
    {
      "code": "ct101",
      "username": "admin",
      "subscriptionExpiresOn": "2019-11-27T09:35:38.916Z",
    },
    {
      "code": "bt101",
      "username": "admin",
      "subscriptionExpiresOn": "2016-11-27T08:20:57.297Z",
    },
  ]
}

const now = new Date();

const result = data.rows.map((item) => {
  if (new Date(item.subscriptionExpiresOn) < now) {
    return {
      ...item,
      subscriptionExpiresOn: 'Expired',
    };
  } else {
    return item;
  }
});

console.log(result)

Collectives™ on Stack Overflow

Returning new form of array of objects using javascript

3 Answers 3

11 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

11 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related