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 Basics (Retired) Making Decisions with Conditional Statements Super Conditional Challenge

Kathy Epperson
Kathy Epperson
1,884 Points
Posted by Kathy Epperson
Kathy Epperson
1,884 Points

Why isn't my else if statement working?

I'm supposed to fix this code to get the alert I'm supposed to get to is "It's Friday, but I don't have enough money to go out". What am I doing wrong?

script.js 
var money = 9;
var today = 'Friday'

if ( money >= 100 && today === 'Friday' ) {
  alert("Time to go to the theater");    
} else if ( money >= 50 && today === 'Friday' ) {
  alert("Time for a movie and dinner");    
} else if ( money > 10 && today === 'Friday' ) {
  alert("Time for a movie");   
} else if ( money < 9 || today === 'Friday' ) {
  alert("It's Friday, but I don't have enough money to go out");   
} else {
  alert("This isn't Friday. I need to stay home.");
}
index.html 
<!DOCTYPE HTML>
<html>
<head>
  <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  <title>JavaScript Basics</title>
</head>
<body>
<script src="script.js"></script>
</body>
</html>

1 Answer

Andreas Nyström
Andreas Nyström
8,887 Points
Andreas Nyström
Andreas Nyström
8,887 Points

Hi. All you got to do is change the operators in this. So it looks like this: 

var money = 9;
var today = 'Friday'

if ( money >= 100 && today === 'Friday' ) {
  alert("Time to go to the theater");    
} else if ( money >= 50 && today === 'Friday' ) {
  alert("Time for a movie and dinner");    
} else if ( money > 10 && today === 'Friday' ) {
  alert("Time for a movie");   
} else if ( today === 'Friday' ) {
  alert("It's Friday, but I don't have enough money to go out");   
} else {
  alert("This isn't Friday. I need to stay home.");
}

To clearify: I only changed the OR operator (||) to the AND operator (&&). This means that both statements need to be true in order for it to pass as true. If you use the OR-operator only one statement needs to be true.

I also changed so it says today === 'Friday' which is the same as today EQUAL TO 'Friday'.

Before it said today !== 'Friday' which is the same as today NOT EQUAL TO 'Friday'. Hope this helps you out :).