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

Tadjiev Codes
Tadjiev Codes
9,626 Points
Posted by Tadjiev Codes
Tadjiev Codes
9,626 Points

Something's wrong with this script. The value in the variable money is only 9. But if you preview this script you'll see

Hi, what exactly can be done here?

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 >= 12 || 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>

2 Answers

sebastian tramper
sebastian tramper
7,210 Points
sebastian tramper
sebastian tramper
7,210 Points

hi,

First off all this: || means "or" maybe use && it mean "and"

second to check if something is equal, you use: == not === i believe

Goodluck ! :)

Seth Kroger
Seth Kroger
56,413 Points
Seth Kroger
Seth Kroger
56,413 Points

I'd like to draw your attention to this part:

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

You've already established whether money is more than 10 or it's a Friday ( you should use && for AND to meet both conditions) and that includes 12 or more, so the second condition is always false when it's checked. Given the alert's message, it would make more sense that you only check if it's Friday.

Tadjiev Codes
Tadjiev Codes
9,626 Points
Tadjiev Codes
Tadjiev Codes
9,626 Points

Hi Sir, i tried to add && for AND but still it's not working(