Cleanest way to loop over an array of objects and replace a property

Question

What is the cleanest way to assign a person's seat while looping over the available seats, when after assigning a person's seat, the movie ticket status must be marked unavailable. Would be open to using lodash methods as well.

const movieTickets = [
  {
    seat: "16B"
    status: "available"
  },
  {
    seat: "16c"
    status: "available"
  },
  {
    seat: "16D"
    status: "available"
  }
]

const people = [
   { 
     name: "Bob"
     seat: ""
   },
   { 
     name: "Susan"
     seat: ""
   },
   { 
     name: "Timmy"
     seat: ""
   }
]

/**
*  Current solution
* The problem is that ticket is not marked as unavailable, and I'm unsure of the cleanest way to do that, would love suggestions here
**/
 
const assignedPeople = people.map(person => {
  person.seat = movieTickets.find(ticket => ticket.status === "available").seat

  return person;
});

Nitheesh · Accepted Answer · 2021-09-15 15:11:13Z

2

Update your movieTickets array with the same process.

const movieTickets = [
  { seat: "16B", status: "available" },
  { seat: "16c", status: "available" },
  { seat: "16D", status: "available" }
];

const people = [
  { name: "Bob", seat: "" },
  { name: "Susan", seat: "" },
  { name: "Timmy", seat: "" }
]
const assignedPeople = people.map(person => {
  const node = movieTickets.find(ticket => ticket.status === "available");
  person.seat = node.seat;
  node.status = "unavailable";
  return person;
});
console.log(assignedPeople);
console.log(movieTickets);

If you dont want to create a new array, you could update the original array itself.

const movieTickets = [
  { seat: "16B", status: "available" },
  { seat: "16c", status: "available" },
  { seat: "16D", status: "available" }
];

const people = [
  { name: "Bob", seat: "" },
  { name: "Susan", seat: "" },
  { name: "Timmy", seat: "" }
]
people.forEach(person => {
  const node = movieTickets.find(ticket => ticket.status === "available");
  person.seat = node.seat;
  node.status = "unavailable";
});
console.log(people);
console.log(movieTickets);

edited Sep 15, 2021 at 15:11

answered Sep 15, 2021 at 15:08

Nitheesh

20.1k3 gold badges28 silver badges52 bronze badges

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

1 Comment

Ash Rhazaly Over a year ago

Dope, didn't see this in front of me, thanks!

Nitheesh · Accepted Answer · 2021-09-16 03:49:01Z

Assign the found ticket to a variable. Then you can mark it unavailable before assigning the seat to the person.

Also, this allows you to check that a ticket was found. Your code will get an error if there are more people than available tickets, because it will try to access the seat property of null.

const movieTickets = [
  { seat: "16B", status: "available" },
  { seat: "16c", status: "available" },
  { seat: "16D", status: "available" }
];

const people = [
  { name: "Bob", seat: "" },
  { name: "Susan", seat: "" },
  { name: "Timmy", seat: "" }
];

const assignedPeople = people.map(person => {
  let ticket = movieTickets.find(ticket => ticket.status === "available");
  if (ticket) {
    ticket.status = "unavailable";
    person.seat = ticket.seat;
  }
  return person;
});

console.log(assignedPeople);

Nitheesh · Accepted Answer · 2021-09-16 03:50:09Z

0

You can easily achieve the result using map and find

const movieTickets = [
  { seat: "16B", status: "available" },
  { seat: "16c", status: "available" },
  { seat: "16D", status: "available" }
];

const people = [
  { name: "Bob", seat: "" },
  { name: "Susan", seat: "" },
  { name: "Timmy", seat: "" }
];

const newPeopleData = people.map((p) => {
  const isSeatAvailable = movieTickets.find((o) => o.status === "available");
  if (isSeatAvailable) {
    isSeatAvailable.status = "unavailable";
    return { ...p, seat: isSeatAvailable.seat };
  }
  return p;
});

console.log(newPeopleData);
console.log(movieTickets);

/* This is not a part of answer. It is just to give the output fill height. So IGNORE IT */
.as-console-wrapper { max-height: 100% !important; top: 0; }

edited Sep 16, 2021 at 3:50

Nitheesh

20.1k3 gold badges28 silver badges52 bronze badges

answered Sep 15, 2021 at 15:15

DecPK

25.4k6 gold badges32 silver badges46 bronze badges

Collectives™ on Stack Overflow

Cleanest way to loop over an array of objects and replace a property

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related