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

Farhan Khan
Farhan Khan
17,530 Points
Posted by Farhan Khan
Farhan Khan
17,530 Points

how is this not correct?

I am getting the correct output but it's still giving me a bummer? why? "Something's wrong with this script. The value in the variable money is only 9. But if you preview this script you'll see the "Time to go to the theater" message. Fix this script so that it correctly tests the money and today variables and prints out the proper alert message: "It's Friday, but I don't have enough money to go out""

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 ( today === 'Friday' ||  money < 9) {
  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>

2 Answers

Sam Baines
Sam Baines
4,315 Points
Sam Baines
Sam Baines
4,315 Points

Hi Farhan - I think you need to change the || OR operator to the the && AND Operator in the last else if statement.

Daniel Botta
Daniel Botta
17,956 Points
Daniel Botta
Daniel Botta
17,956 Points 
else if ( money > 10 && today === 'Friday' ) {
  alert("Time for a movie");   
} else if ( today === 'Friday' ||  money < 9) {
  alert("It's Friday, but I don't have enough money to go out");   
}

Is the the second argument supposed to have an || comparison? I'm not sure what the exact output is supposed to be but maybe you need the && caparison like the others? With the || it could not be Friday but have < 9 and it will still display "It's Friday, but I don't have enough money to go out", which would not be true. You also may need to add >= and <= to the last two else if comparisons. Otherwise there will be no test for money values of 10 and 9. Try this...

else if ( money >= 10 && today === 'Friday' ) {
  alert("Time for a movie");   
} else if ( today === 'Friday' &&  money <= 9) {
  alert("It's Friday, but I don't have enough money to go out");   
}