Unable to read an attribute of last object in array of objects (javascript)

Question

I am working on a basic prototype of rating based to-do app in javascript. There's an object which I am using to store task name and its rating. Example:

taskDetails = [
    {name: "C", rating: 69.3425},
    {name: "A", rating: 60.93875},
    {name: "D", rating: 57.32,
    {name: "B", rating: 59.795}
]

And then there's a function checkAllEqual()

var checkAllEqual = function(){
    var flag = true;
    var temp = [];    
    for(var i = 0; taskDetails.length; i++){
        if(taskDetails[i].rating){     //Line no. 58 of the code
            var rating = taskDetails[i].rating;
            temp.push(parseInt(rating.toFixed(2)));
        }
    }

    console.log(temp);
    return flag;
}

This function checks if all the todos have equal ratings or not. It works fine until 3 iterations but in the last one throws an error that 'rating' is undefined even when 'rating' inside if parenthesis holds correct value! (Checked in the debugger)

I couldn't find the reason. Please help.

The error:

app.js:58 Uncaught TypeError: Cannot read property 'rating' of undefined
    at checkAllEqual (app.js:58)
    at <anonymous>:1:1

You may use a for(const taskDetail of taskDetails) loop to prevent such typos — Jonas Wilms
– Jonas Wilms, Commented Oct 22, 2017 at 9:19

melpomene · Accepted Answer · 2017-10-22 09:12:31Z

1

Your loop condition is just taskDetails.length, so that's an infinite loop that runs right off the end of your array. You want i < taskDetails.length.

answered Oct 22, 2017 at 9:12

melpomene

86.2k8 gold badges97 silver badges156 bronze badges

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

1 Comment

Abhinay Over a year ago

Oops! Silly me :P

Kartik Prakash · Accepted Answer · 2017-10-25 17:43:20Z

0

Change the condition of the loop to i < taskDetails.length

answered Oct 25, 2017 at 17:43

Kartik Prakash

1

Collectives™ on Stack Overflow

Unable to read an attribute of last object in array of objects (javascript)

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related