Looping through multiple arrays and converting values using moment JS

Question

I have a function that takes an object, re-sorts the days manually, and converts a string to an array

export const convertHoursOfOperation = (service) => {
  const hours = {
    Monday: service.hours_of_operation["monday"],
    Tuesday: service.hours_of_operation["tuesday"],
    Wednesday: service.hours_of_operation["wednesday"],
    Thursday: service.hours_of_operation["thursday"],
    Friday: service.hours_of_operation["friday"],
    Saturday: service.hours_of_operation["saturday"],
    Sunday: service.hours_of_operation["sunday"]
  };

  for (let hour in hours) {
    hours[hour] !== null ? hours[hour] = hours[hour].split(",") : hours[hour] = 'Closed'

    hours[hour] !== 'Closed' ? hours[hour].forEach((h) => moment(h, "HH:mm").format('h:mm')) : null
  }

  return hours;
}

output from hours[hour] !== null ? hours[hour] = hours[hour].split(",") : hours[hour] = 'Closed'

monday: '09:00, 15:00' => monday: [09:00, 15:00] OR monday: null => monday: 'Closed'

then I am trying to loop through the array if it is an array and convert the values from military to standard time using moment.

so now my desired output should be:

monday: [09:00, 15:00] => monday: [9:00, 3:00]

currently the line hours[hour] !== 'Closed' ? hours[hour].forEach((h) => moment(h, "HH:mm").format('h:mm')) : null is not converting anything. What am I doing wrong here?

You don't appear to be setting your moment() to anything. you are simply running the function. Try like the line above it: hours[hour] !== 'Closed' ? hours[hour].forEach((h) => hours[hour] = moment(h, "HH:mm").format('h:mm')) : null; — mykaf
– mykaf, Commented Mar 25, 2022 at 16:30
At this point anything that is not 'closed' returns undefined — Ben_Sven_Ten
– Ben_Sven_Ten, Commented Mar 25, 2022 at 16:35
@Aditya that has worked! You may submit as answer if you'd like — Ben_Sven_Ten
– Ben_Sven_Ten, Commented Mar 25, 2022 at 16:41

Aditya · Accepted Answer · 2022-03-25 16:44:02Z

1

you are not setting your moment() to anything. Set the value at that particular index. The correct code should be this:

hours[hour] !== 'Closed' ? hours[hour].forEach((h,index) => hours[hour][index] = moment(h, "HH:mm").format('h:mm')) : null;

answered Mar 25, 2022 at 16:44

Aditya

4703 silver badges16 bronze badges

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

Collectives™ on Stack Overflow

Looping through multiple arrays and converting values using moment JS

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related