How to simplify nested JavaScript loops

Question

I have written three for loops for displaying data on the screen using JavaScript. How can I simplify the following code?

My Json Format:

    {
        "result": {
            "type": "SUCCESS",
            "data": [{          
                "monthName": "July",
                "monthId": 7,
                "weeks": [{             
                    "days": [{                  
                        "day": "Sunday",
                        "date": "07/31/2016"
                    }],             
                }],
            }],
        }
    }

var monthObj = response.result.data;

for (var i = 0; i < monthObj.length; i++) {  
  for (var t = 0; t < monthObj[i].weeks.length; t++) {
    for (var s = 0; s < monthObj[i].weeks[t].days.length; s++) {
    }
  }     
}

This is my loop, here I have written 3 loops.
Is there any easy way to simplify these loops?

Why do you imagine it might be simplifiable? How do you want to simplify it? Is there some reason you did not add the for-loop tag to this question? — user663031
– user663031, Commented Dec 29, 2016 at 12:09
i have written 3 for loops, My doubt is this possible to write everything in one for loop. — sathish kumar
– sathish kumar, Commented Dec 29, 2016 at 12:16
@AhmadIbrahim Please do not remove the jQuery tag. It expresses the OP's willingness to entertain a jQuery solution. — user663031
– user663031, Commented Dec 29, 2016 at 12:21
Since years/months/days are more or less static data, the only dynamic data you need is the date, so your data might as well be "data" : ["07/31/2016"], which can give you all the other info when converted to a Date object. It would reduce everything down to one loop. (depending on what you're doing with those dates...) — Shilly
– Shilly, Commented Dec 29, 2016 at 12:39

Christian Zosel · Accepted Answer · 2016-12-29 12:04:31Z

2

It depends how your data looks like and what you want to do inside the loop. If you expect to have more than one entry for month, week and day and you want to perform some action on all of them, you will need three loops.

I'd recommend using forEach instead of for though:

const data = {
  "result": {
    "type": "SUCCESS",
    "data": [{
      "monthName": "July",
      "monthId": 7,
      "weeks": [{
        "days": [{
          "day": "Sunday",
          "date": "07/31/2016"
        }],
      }],
    }],
  }
}

data.result.data.forEach(month => {
  month.weeks.forEach(week => {
    week.days.forEach(day => console.log(day))
  })
})

answered Dec 29, 2016 at 12:04

Christian Zosel

1,4241 gold badge9 silver badges16 bronze badges

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

Collectives™ on Stack Overflow

How to simplify nested JavaScript loops

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