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 AJAX Basics (retiring) Programming AJAX Check for the correct ready state

Karan kyanam
Karan kyanam
7,293 Points
Posted by Karan kyanam
Karan kyanam
7,293 Points

Solution for code challenge

Is my code incomplete? Do I need to check the status of all other errors?

app.js 
var xhr = new XMLHttpRequest();
xhr.onreadystatechange = function() {
if (xhr.readyState === 4 && status === 200)
};
xhr.open('GET', 'sidebar.html');
xhr.send();
index.html 
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <title>AJAX with JavaScript</title>
  <script src="app.js"></script>
</head>
<body>
  <div id="main">
    <h1>AJAX!</h1>
  </div>
  <div id="sidebar"></div>
</body>
</html>

2 Answers

Ken Alger
STAFF
Ken Alger
Treehouse Teacher
Ken Alger
Ken Alger
Treehouse Teacher

Karan;

Yes, it is not complete. You need to include {} at the end of your if statement line, to include a place for what happens if your conditional returns true. Something along the lines of:

if (conditionalStatement) {
   //do something amazing in here
}

For Task 1 we aren't yet told what amazing thing we want to do, so we can leave it blank.

Happy coding and Happy Holidays,

Ken

Karan kyanam
Karan kyanam
7,293 Points
Karan kyanam
Karan kyanam
7,293 Points

Thank you. Got confused with that. Also typed in status instead of xhr.status. Happy holidays to you too.