JavaScript Scope

[bhb23955]

<
// The scope of random is too loose
const random = Math.floor(Math.random() * 3);

const getRandEvent = () => {
if (random === 0) {
return ‘Marathon’;
} else if (random === 1) {
return ‘Triathlon’;
} else if (random === 2) {
return ‘Pentathlon’;
}
};

// The scope of days is too tight
const getTrainingDays = event => {
let days = “”;
if (event === ‘Marathon’) {
days = 50;
} else if (event === ‘Triathlon’) {
days = 100;
} else if (event === ‘Pentathlon’) {
days = 200;
}

return days;
};

const name = ‘Nala’;

// The scope of name is too tight
const logEvent = (name, event) => {
console.log(${name}'s event is: ${event});
};

const logTime = (name, days) => {
console.log(${name}'s time to train is: ${days} days);
};

const event = getRandEvent();
const days = getTrainingDays(event);
// Define a name variable. Use it as an argument after updating logEvent and logTime

logEvent(event);
logTime(days);

/>

The above is my code. It is from the scoping exercise on JavaScript. In one part of the exercise, it asks us to " Pass name as the first argument to logEvent() and logTime() ." However, when I do this, the following is output:

Why does it change the name to Triatholon, Marathon, or Pentathlon and the event and days become undefined? If I remove the parameters, the code runs fine, but the exercise asks you to add the parameters, so I know it’s supposed to work.

[thepitycoder]

A quick one about code formatting:

You can click the </> icon in the menu bar which appears at the top of the text box while you type.

Press Enter to go to a blank line, click that icon, and you will see this:

Just copy and paste your code directly from the editor into that highlighted portion, and the forum will retain the formatting.

The above is my code.

It doesn’t match what’s shown in the screenshot; is it up-to-date?

Why does it change the name to Triatholon, Marathon, or Pentathlon and the event and days become undefined?

In your code, you are calling the functions like so:

logEvent(event);
logTime(days);

In each, you’re providing a single parameter. In the definitions I can see in your screenshot, the functions look like this:

const logEvent = (name, event) => {
  console.log(`${name}'s event is: ${event}`);
};

const logTime = (name, days) => {
  console.log(`${name}'s time to train is: ${days} days`);
};

The output you’re getting is correct, for a single parameter input.

[bhb23955]

I edited the code part.

[thepitycoder]

It would appear my explanation of why you’re seeing the output you are was correct.

you’re providing a single parameter. In the definitions I can see in your screenshot, the functions look like this:

const logEvent = (name, event) => {
  console.log(`${name}'s event is: ${event}`);
};

const logTime = (name, days) => {
  console.log(`${name}'s time to train is: ${days} days`);
};

The output you’re getting is correct, for a single parameter input.

Can you see how to fix it?

[bhb23955]

Oh so in the bottom, when I call the function, I need to put two parameters?

[thepitycoder]

Bingo.

name and event for logEvent, and name and days for logTime.

[bhb23955]

ugh I was so focused on the logEvent and logTime functions that I forgot to look at the bottom where they’re called…