Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript JavaScript Loops, Arrays and Objects Tracking Data Using Objects The Student Record Search Challenge Solution

jennifer ray
jennifer ray
7,875 Points

Can anyone see any problem with my code? Javascript

My code runs but it does not pull up the grades for a student if all typed in lower case. Can anyone see an error?

var message = '';
var student;
var search;

function print(message) {
  var outputDiv = document.getElementById('output');
  outputDiv.innerHTML = message;
}

function getStudentReport (student) {
  var report = '<h2>Student: ' + student.name + '</h2>';
  report += '<p>Track: ' + student.track + '</p>';
  report += '<p>Points: ' + student.points + '</p>';
  report += '<p>Achievements: ' + student.achievements + '</p>';
  return report;
}


while (true) {
search = prompt('search student records: type a name or "quit" to exit');
  if (search === null || search.toLowerCase() === 'quit') {
      break;
      }
  for (var i = 0; i < students.length; i += 1) {
  student = students[i];
    if ( student.name === search ) {
      message = getStudentReport( student );
      print(message);
    }

}
}

3 Answers

Petros Sordinas
Petros Sordinas
16,181 Points

Change

if ( student.name === search )

To

if (student.name.toLowercase() === search.toLowerCase())

You have to ensure that both strings are lowercase (or uppercase) to make sure they are equal.

Hi Jennifer,

In the line:

if ( student.name === search ) {

Do you want to use .toLowerCase() on search? I don't really know javascript but when you check to see if the user entered 'quit' you tested against search.toLowerCase() - maybe doing the same when comparing against the name may do that for you?

if ( student.name === search.toLowerCase() ) {

Steve.

Also, I can't see where students the array is created so I can't tell if students.length is greater than zero. If it isn't, the for loop won't run, meaning getStudentReport() will never be called. I'm sure that's just in another part of the code - the array must be populated elsewhere.