Sleep Debt Calculator Help

brzrkrbee13 · October 4, 2020, 11:38pm

gist.github.com

https://gist.github.com/codecademydev/19fdcd10aac8430e4b1b92d951a72ed0

sleepDebtCalculator.js

const getSleepHours = day => {
  switch(day) {
    case 'monday':
      return 5
      break;
    case 'tuesday':
      return 5
      break;
    case 'wednesday':
      return 7

This file has been truncated. show original

been trying to figure out why this is giving me an answer of “17Error!784” in the console. I’m at a lose…

toastedpitabread · October 5, 2020, 12:33am

Ok so there’s a couple of issues with your code. But nothing that can’t be fixed!

One thing you should review is what happens when a function “returns” a statement.
In your getActualSleepHours function you are invoking getSleepHours, but you’re not storing the variables anywhere.

Also, you have to have brackets for your getActualSleepHours code block.

Welcome to the forums

ryansup · October 5, 2020, 1:11am

I think the getActualSleepHours function is fine as a concise arrow expression.

The output you are getting is suggesting one of your switch comparisons is making it to the default value. Check your spelling carefully

toastedpitabread · October 5, 2020, 1:13am

Ah you’re right, I looked at it too hastily. Good catch.

ryansup · October 5, 2020, 1:19am

All good brotha - I gotchu!

Also,

Since a function stops running as soon as it returns a value, you don’t need the break statements. Having them won’t hurt but they’re not doing anything here. Like @toastedpitabread said, review return and break statements.

brzrkrbee13 · October 5, 2020, 1:45am

Awesome! Thank you everyone for your replies ! Will give it a good once over again.

brzrkrbee13 · October 5, 2020, 10:32pm

Does a return statement need to be at the end of the getActualSleepHours function?

midlindner · October 5, 2020, 10:44pm

No. You have a concise body arrow function. The fat arrow => is essentially saying return. Your issue really is one of spelling. Try calling your getSleepHours() function for each day separately, and logging the result to the console:

brzrkrbee13:

//comment this line out for now
//console.log(getActualSleepHours()); 

//try this:
console.log(getSleepHours('Monday'); //run, then change to 'Tuesday', then 'Wednesday', etc. until you see which day is returning the error

brzrkrbee13 · October 5, 2020, 11:05pm

Thank you, just wanted to be sure. I finally finished the project and get it working properly. Thank you all for your help!