Iteration over javascript objects

Question

Write a function that takes Season number and episode number as input and gives all the information about that particular episode as output

After taking inputs from the user as season number and episode number, it doesn't give output as the information about that particular episode

let BigBang =  { 
    "_embedded": {
        "episodes": [
          {
            "id": 2913,
            "name": "Pilot",
            "season": 1,
            "number": 1,
            "airdate": "2007-09-24",
            "airtime": "20:30",
            "airstamp": "2007-09-25T00:30:00+00:00",
            "runtime": 30,


            "_links": {
              "self": {
                "href": "http:\/\/api.tvmaze.com\/episodes\/2913"
              }
            }
          },
          {
            "id": 2914,
            "name": "The Big Bran Hypothesis",
            "season": 1,
            "number": 2,
            "airdate": "2007-10-01",
            "airtime": "20:30",
            "airstamp": "2007-10-02T00:30:00+00:00",
            "runtime": 30,
            "image": {
              "medium": "http:\/\/static.tvmaze.com\/uploads\/images\/medium_landscape\/4\/12369.jpg",
              "original": "http:\/\/static.tvmaze.com\/uploads\/images\/original_untouched\/4\/12369.jpg"
            },
       }



    let season = prompt('Enter Season number');                               
    let number = prompt('Enter Episode number');                            
    let AllInfo = (season,number) => {                                   
        for(let current in BigBang._embedded.episodes) {                    
            if(BigBang._embedded.episodes[current].season === season) {                                              
                if(BigBang._embedded.episodes[current].number === number) { 
                let Detail = BigBang._embedded.episodes[current];
                    alert(Detail);
                }
            }
        }                            
    AllInfo(season,number);                         
    }

Can you show us what you have tried?

Thum Choon Tat
– Thum Choon Tat

2018-03-28 06:18:08 +00:00
Commented Mar 28, 2018 at 6:18 — Thum Choon Tat
– Thum Choon Tat, Commented Mar 28, 2018 at 6:18
It;s in the code above. scroll down you will see

Kunal
– Kunal

2018-03-28 06:20:29 +00:00
Commented Mar 28, 2018 at 6:20 — Kunal
– Kunal, Commented Mar 28, 2018 at 6:20
Sorry I missed that

Thum Choon Tat
– Thum Choon Tat

2018-03-28 06:20:44 +00:00
Commented Mar 28, 2018 at 6:20 — Thum Choon Tat
– Thum Choon Tat, Commented Mar 28, 2018 at 6:20

CertainPerformance · Accepted Answer · 2018-03-28 06:20:51Z

Try using .find instead, it'll make the code a lot cleaner:

    let BigBang = {
      "_embedded": {
        "episodes": [{
            "id": 2913,
            "name": "Pilot",
            "season": 1,
            "number": 1,
            "airdate": "2007-09-24",
            "airtime": "20:30",
            "airstamp": "2007-09-25T00:30:00+00:00",
            "runtime": 30,
            "_links": {
              "self": {
                "href": "http:\/\/api.tvmaze.com\/episodes\/2913"
              }
            }
          },
          {
            "id": 2914,
            "name": "The Big Bran Hypothesis",
            "season": 1,
            "number": 2,
            "airdate": "2007-10-01",
            "airtime": "20:30",
            "airstamp": "2007-10-02T00:30:00+00:00",
            "runtime": 30,
            "image": {
              "medium": "http:\/\/static.tvmaze.com\/uploads\/images\/medium_landscape\/4\/12369.jpg",
              "original": "http:\/\/static.tvmaze.com\/uploads\/images\/original_untouched\/4\/12369.jpg"
            },
          }
        ]
      }
    }
    //const inputSeason = prompt('Enter Season number');
    const inputSeason = 1;
    //const inputNumber = prompt('Enter Episode number');
    const inputNumber = 2;

    const foundEpisode = BigBang._embedded.episodes.find(({ season, number}) => {
      return season === inputSeason && number === inputNumber;
    });
    if (foundEpisode) console.log(foundEpisode);
    else console.log('No matching season/number found!');

Benjamin Lüscher · Accepted Answer · 2018-03-28 06:22:38Z

0

You could do that easier with the find() method on your array.

let episode = BigBang._embedded.episodes.find((e) => {
    return e.season === season && e.number === number;
});
if (episode) {
    alert(episode.name);
}

answered Mar 28, 2018 at 6:22

Benjamin Lüscher

4125 silver badges14 bronze badges

Comments

Chinmoy Samanta · Accepted Answer · 2018-03-28 06:24:32Z

I have debugged your code and saw that you call AllInfo function within AllInfo .So there is recursive call happen in your code. So remove calling of AllInfo from AllInfo function, your issue will be fixed. Try the following code.

let BigBang =  { 
"_embedded": {
    "episodes": [
      {
        "id": 2913,
        "name": "Pilot",
        "season": 1,
        "number": 1,
        "airdate": "2007-09-24",
        "airtime": "20:30",
        "airstamp": "2007-09-25T00:30:00+00:00",
        "runtime": 30,


        "_links": {
          "self": {
            "href": "http:\/\/api.tvmaze.com\/episodes\/2913"
          }
        }
      },
      {
        "id": 2914,
        "name": "The Big Bran Hypothesis",
        "season": 1,
        "number": 2,
        "airdate": "2007-10-01",
        "airtime": "20:30",
        "airstamp": "2007-10-02T00:30:00+00:00",
        "runtime": 30,
        "image": {
          "medium": "http:\/\/static.tvmaze.com\/uploads\/images\/medium_landscape\/4\/12369.jpg",
          "original": "http:\/\/static.tvmaze.com\/uploads\/images\/original_untouched\/4\/12369.jpg"
        },
   }]}};



let season = 1;                               
let number = 2;                            
let AllInfo = (season,number) => {                              
    for(let current in BigBang._embedded.episodes) {                    
        if(BigBang._embedded.episodes[current].season === season) {                                              
            if(BigBang._embedded.episodes[current].number === number) { 
            let Detail = BigBang._embedded.episodes[current];
                alert(JSON.stringify(Detail,null,4));
            }
        }
    }                                                     
}
AllInfo(season,number);

Collectives™ on Stack Overflow

Iteration over javascript objects

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related