Angular variable is NaN or undefined












0














I need to get team in my PlayerComponent class. When I use this line of code:



this.teamId = this.player.teamId;


I get "ERROR TypeError: Cannot read property 'teamId' of undefined".



I use this code in html to retrieve team name from team object.



<p>Team:  {{ team?.name }}</p>


Why is this happening? My player loads correctly.



export class PlayerComponent implements OnInit {
player: Player;
playerId: number;
team: Team;
teamId: number;

constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
private route: ActivatedRoute) { }

ngOnInit() {
this.loadPlayer();
this.loadTeam();
}

loadPlayer() {
this.playerId = this.route.snapshot.params['id'];
this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
this.player = player;
}, error => {
this.alertify.error(error);
});
}

loadTeam() {
this.teamId = this.player.teamId;
this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
this.team = team;
}, error => {
this.alertify.error(error);
});
}
}









share|improve this question
























  • You need to call loadTeam from the subscribe callback of playerServer.getPlayer.
    – The Head Rush
    Nov 12 at 14:15










  • It worked. Thank you! :)
    – Martin
    Nov 12 at 14:20
















0














I need to get team in my PlayerComponent class. When I use this line of code:



this.teamId = this.player.teamId;


I get "ERROR TypeError: Cannot read property 'teamId' of undefined".



I use this code in html to retrieve team name from team object.



<p>Team:  {{ team?.name }}</p>


Why is this happening? My player loads correctly.



export class PlayerComponent implements OnInit {
player: Player;
playerId: number;
team: Team;
teamId: number;

constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
private route: ActivatedRoute) { }

ngOnInit() {
this.loadPlayer();
this.loadTeam();
}

loadPlayer() {
this.playerId = this.route.snapshot.params['id'];
this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
this.player = player;
}, error => {
this.alertify.error(error);
});
}

loadTeam() {
this.teamId = this.player.teamId;
this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
this.team = team;
}, error => {
this.alertify.error(error);
});
}
}









share|improve this question
























  • You need to call loadTeam from the subscribe callback of playerServer.getPlayer.
    – The Head Rush
    Nov 12 at 14:15










  • It worked. Thank you! :)
    – Martin
    Nov 12 at 14:20














0












0








0







I need to get team in my PlayerComponent class. When I use this line of code:



this.teamId = this.player.teamId;


I get "ERROR TypeError: Cannot read property 'teamId' of undefined".



I use this code in html to retrieve team name from team object.



<p>Team:  {{ team?.name }}</p>


Why is this happening? My player loads correctly.



export class PlayerComponent implements OnInit {
player: Player;
playerId: number;
team: Team;
teamId: number;

constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
private route: ActivatedRoute) { }

ngOnInit() {
this.loadPlayer();
this.loadTeam();
}

loadPlayer() {
this.playerId = this.route.snapshot.params['id'];
this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
this.player = player;
}, error => {
this.alertify.error(error);
});
}

loadTeam() {
this.teamId = this.player.teamId;
this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
this.team = team;
}, error => {
this.alertify.error(error);
});
}
}









share|improve this question















I need to get team in my PlayerComponent class. When I use this line of code:



this.teamId = this.player.teamId;


I get "ERROR TypeError: Cannot read property 'teamId' of undefined".



I use this code in html to retrieve team name from team object.



<p>Team:  {{ team?.name }}</p>


Why is this happening? My player loads correctly.



export class PlayerComponent implements OnInit {
player: Player;
playerId: number;
team: Team;
teamId: number;

constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
private route: ActivatedRoute) { }

ngOnInit() {
this.loadPlayer();
this.loadTeam();
}

loadPlayer() {
this.playerId = this.route.snapshot.params['id'];
this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
this.player = player;
}, error => {
this.alertify.error(error);
});
}

loadTeam() {
this.teamId = this.player.teamId;
this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
this.team = team;
}, error => {
this.alertify.error(error);
});
}
}






angular variables undefined






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 12 at 14:18

























asked Nov 12 at 14:11









Martin

84




84












  • You need to call loadTeam from the subscribe callback of playerServer.getPlayer.
    – The Head Rush
    Nov 12 at 14:15










  • It worked. Thank you! :)
    – Martin
    Nov 12 at 14:20


















  • You need to call loadTeam from the subscribe callback of playerServer.getPlayer.
    – The Head Rush
    Nov 12 at 14:15










  • It worked. Thank you! :)
    – Martin
    Nov 12 at 14:20
















You need to call loadTeam from the subscribe callback of playerServer.getPlayer.
– The Head Rush
Nov 12 at 14:15




You need to call loadTeam from the subscribe callback of playerServer.getPlayer.
– The Head Rush
Nov 12 at 14:15












It worked. Thank you! :)
– Martin
Nov 12 at 14:20




It worked. Thank you! :)
– Martin
Nov 12 at 14:20












1 Answer
1






active

oldest

votes


















0














That's because you are loading the team before the player is loaded. Keep in mind that http requests working asynchronously.



To solve your problem you can put the this.loadTeam() inside the subscription of the getPlayer() function.



export class PlayerComponent implements OnInit {
player: Player;
playerId: number;
team: Team;
teamId: number;

constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
private route: ActivatedRoute) { }

ngOnInit() {
this.loadPlayer();
}

loadPlayer() {
this.playerId = this.route.snapshot.params['id'];


this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
this.player = player;
this.loadTeam();
}, error => {
this.alertify.error(error);
});
}

loadTeam() {
this.teamId = this.player.teamId;

this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
this.team = team;
}, error => {
this.alertify.error(error);
});
}
}





share|improve this answer





















    Your Answer






    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "1"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53263937%2fangular-variable-is-nan-or-undefined%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown

























    1 Answer
    1






    active

    oldest

    votes








    1 Answer
    1






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    0














    That's because you are loading the team before the player is loaded. Keep in mind that http requests working asynchronously.



    To solve your problem you can put the this.loadTeam() inside the subscription of the getPlayer() function.



    export class PlayerComponent implements OnInit {
    player: Player;
    playerId: number;
    team: Team;
    teamId: number;

    constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
    private route: ActivatedRoute) { }

    ngOnInit() {
    this.loadPlayer();
    }

    loadPlayer() {
    this.playerId = this.route.snapshot.params['id'];


    this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
    this.player = player;
    this.loadTeam();
    }, error => {
    this.alertify.error(error);
    });
    }

    loadTeam() {
    this.teamId = this.player.teamId;

    this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
    this.team = team;
    }, error => {
    this.alertify.error(error);
    });
    }
    }





    share|improve this answer


























      0














      That's because you are loading the team before the player is loaded. Keep in mind that http requests working asynchronously.



      To solve your problem you can put the this.loadTeam() inside the subscription of the getPlayer() function.



      export class PlayerComponent implements OnInit {
      player: Player;
      playerId: number;
      team: Team;
      teamId: number;

      constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
      private route: ActivatedRoute) { }

      ngOnInit() {
      this.loadPlayer();
      }

      loadPlayer() {
      this.playerId = this.route.snapshot.params['id'];


      this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
      this.player = player;
      this.loadTeam();
      }, error => {
      this.alertify.error(error);
      });
      }

      loadTeam() {
      this.teamId = this.player.teamId;

      this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
      this.team = team;
      }, error => {
      this.alertify.error(error);
      });
      }
      }





      share|improve this answer
























        0












        0








        0






        That's because you are loading the team before the player is loaded. Keep in mind that http requests working asynchronously.



        To solve your problem you can put the this.loadTeam() inside the subscription of the getPlayer() function.



        export class PlayerComponent implements OnInit {
        player: Player;
        playerId: number;
        team: Team;
        teamId: number;

        constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
        private route: ActivatedRoute) { }

        ngOnInit() {
        this.loadPlayer();
        }

        loadPlayer() {
        this.playerId = this.route.snapshot.params['id'];


        this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
        this.player = player;
        this.loadTeam();
        }, error => {
        this.alertify.error(error);
        });
        }

        loadTeam() {
        this.teamId = this.player.teamId;

        this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
        this.team = team;
        }, error => {
        this.alertify.error(error);
        });
        }
        }





        share|improve this answer












        That's because you are loading the team before the player is loaded. Keep in mind that http requests working asynchronously.



        To solve your problem you can put the this.loadTeam() inside the subscription of the getPlayer() function.



        export class PlayerComponent implements OnInit {
        player: Player;
        playerId: number;
        team: Team;
        teamId: number;

        constructor(private playerService: PlayerService, private teameService: TeamService, private alertify: AlertifyService,
        private route: ActivatedRoute) { }

        ngOnInit() {
        this.loadPlayer();
        }

        loadPlayer() {
        this.playerId = this.route.snapshot.params['id'];


        this.playerService.getPlayer(+this.playerId).subscribe((player: Player) => {
        this.player = player;
        this.loadTeam();
        }, error => {
        this.alertify.error(error);
        });
        }

        loadTeam() {
        this.teamId = this.player.teamId;

        this.teameService.getTeam(+this.teamId).subscribe((team: Team) => {
        this.team = team;
        }, error => {
        this.alertify.error(error);
        });
        }
        }






        share|improve this answer












        share|improve this answer



        share|improve this answer










        answered Nov 12 at 14:21









        SplitterAlex

        1,191812




        1,191812






























            draft saved

            draft discarded




















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.





            Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


            Please pay close attention to the following guidance:


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53263937%2fangular-variable-is-nan-or-undefined%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Bressuire

            Vorschmack

            Quarantine