Review JavaScript Fundamentals by Building a Gradebook App - Step 4

JavaScript
1

Tell us what’s happening:

My first return line will always be considered as undefined. Only my second return line works as intended, thus I cannot complete the course. Is this a bug, or am I doing something wrong?

Your code so far

function getAverage(scores) {
  let sum = 0;

  for (const score of scores) {
    sum += score;
  }

  return sum / scores.length;
}

function getGrade(score) {
  if (score === 100) {
    return "A++";
  } else if (score >= 90) {
    return "A";
  } else if (score >= 80) {
    return "B";
  } else if (score >= 70) {
    return "C";
  } else if (score >= 60) {
    return "D";
  } else {
    return "F";
  }
}

function hasPassingGrade(score) {
  return getGrade(score) !== "F";
}


// User Editable Region

function studentMsg(totalScores, studentScore) {
const average = getAverage(totalScores);
const grade = getGrade(studentScore);
if (hasPassingGrade(grade) == true) {
  return 'Class average: ' + average + '. Your grade ' + grade + '. You passed the course.';
} else {
  return 'Class average: ' + average + '. Your grade ' + grade + '. You failed the course'
}
}
console.log(studentMsg([56, 23, 89, 42, 75, 11, 68, 34, 91, 19], 100));
console.log(studentMsg([92, 88, 12, 77, 57, 100, 67, 38, 97, 89], 37));

// User Editable Region

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/128.0.0.0 Safari/537.36

Challenge Information:

Review JavaScript Fundamentals by Building a Gradebook App - Step 4

2

hi and welcome to the forum:

hasPassingGrade is getting a letter grade here because you are passing it the value in the grade variable. But it is expecting a number instead. Also you should not use == here as the returned result of hasPassingGrade is a boolean so you can just test it like a truthy without the comparison to true.

Also your messages are not an exact match to the messages the test wants. You are missing a period in one of them and a colon in each.

3

Thank you for the quick response and sorry for my late answer! I sadly couldn’t fix the issue of only the ‘else’ line being defined, by changing the hasPassingGrade line. Instead I opted for a different approach by integrating the hasPassingGrade function in one single return statement (I got rid of ‘if/else’) and it worked.