I believe my code is correct but nothing is being displayed. I dont want to view the solution just yet. any help please

Question

This is just the students array of objects. same as dave's. nothing wrong here but just put here in case needed for reference to my main code

var students = [ 
  { 
   name: 'Dave',
    track: 'Front End Development',
    achievements: 158,
    points: 14730
  },

  {
    name: 'Jody',
    track: 'iOS Development with Swift',
    achievements: '175',
    points: '16375'
  },

  {
    name: 'Jordan',
    track: 'PHP Development',
    achievements: '55',
    points: '2025'
  },

  {
    name: 'John',
    track: 'Learn WordPress',
    achievements: '40',
    points: '1950'
  },

  {
    name: 'Trish',
    track: 'Rails Development',
    achievements: '5',
    points: '350'
  }
];

Here is my main code and Ive looked over it several times and did several edits and everything seems logical but I must be overlooking something. any help is always appreciated. cheers

var message = '';
var student;
var response;

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

while (true) {
  response=prompt("Search student records: type a name or type 'quit' to end");
  if (response.toUpperCase() === "QUIT") {
      break;
      }


for (var i = 0; i < students.length; i += 1) {
  student = students[i];
  if (student.name.toUpperCase() === response.toUpperCase()) {
  message += '<h2>Student: ' + student.name + '</h2>';
  message += '<p>Track: ' + student.track + '</p>';
  message += '<p>Points: ' + student.points + '</p>';
  message += '<p>Achievements: ' + student.achievements + '</p>';
  print(message);
}
}

Answer 1 · 2015-08-10T04:19:18Z

August 10, 2015 4:19am

I checked it out, looks good! Only two things worth mentioning I think. In your while condition, "true" is a perfectly fine argument in this case, you don't have to create a boolean variable for it. And the formatting needs some work! :P

var student;
var response;

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

while (true) {
  response = prompt("Search student records: type a name or type 'quit' to end");
  if (response.toUpperCase() === "QUIT") {
    break;
  } else {
    for (var i = 0; i < students.length; i += 1) {
      student = students[i];
      if (student.name.toUpperCase() === response.toUpperCase()) {
        var message = '<h2>Student: ' + student.name + '</h2>';
        message += '<p>Track: ' + student.track + '</p>';
        message += '<p>Points: ' + student.points + '</p>';
        message += '<p>Achievements: ' + student.achievements + '</p>';
        print(message);
      }
    }
  }
}

As for your issue, it keeps adding them because your message variable keeps storing strings without ever being reset. By placing the variable declaration inside of the if statement, we're basically resetting the variable and message each time. I made the change to fix this in the code above. If you don't like recreating the variable every time (like me), you can turn the first += operator into a =. This will also erase the entire previous string.

Glad you were able to get it working!

Welcome to the Treehouse Community

Looking to learn something new?

Kevin Faust

Kevin Faust

I believe my code is correct but nothing is being displayed. I dont want to view the solution just yet. any help please

Justin Iezzi

Justin Iezzi

Kevin Faust

Kevin Faust

1 Answer

Justin Iezzi

Justin Iezzi

Kevin Faust

Kevin Faust