West Midlands | May 2026 | Muhammad-Burhan Mustafa | Sprint 1 | Module-Structuring-and-Testing-Data#1442
Conversation
This comment has been minimized.
This comment has been minimized.
5 similar comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
| // trying to create an age variable and then reassign the value by 1 | ||
|
|
||
| const age = 33; | ||
| var age = 33; |
There was a problem hiding this comment.
Should use let instead of var.
Note: Feel free to use AI to find out why use of 'var' is not recommended.
There was a problem hiding this comment.
We usually do not use var because its scope rules and redeclaration rules can cause bugs. Thats why let and const are usually used instead. This change has been made : )
| // e) What do you think the variable result represents? Can you think of a better name for this variable? | ||
|
|
||
| // Answer: The expressions uses the modulo function to work out the remainder value which in this case works out the remaining seconds of the total movie length |
There was a problem hiding this comment.
Part (e) has a second question. Could you also answer that?
There was a problem hiding this comment.
Note: The file where I left this comment was 2-time-format.js. I just noticed you updated 1-percentage-change.js instead.
There was a problem hiding this comment.
apologies this has been rectified now
cjyuan
left a comment
There was a problem hiding this comment.
Could you also address this comment: #1442 (comment) ?
| // e) What do you think the variable result represents? Can you think of a better name for this variable? | ||
|
|
||
| // Answer: The expressions uses the modulo function to work out the remainder value which in this case works out the remaining seconds of the total movie length |
There was a problem hiding this comment.
You deleted question (e) in your file. The original question is;
// e) What do you think the variable result represents? Can you think of a better name for this variable?
The variable which question (e) refers to is the one named result on line 9:
const result = `${totalHours}:${remainingMinutes}:${remainingSeconds}`;
|
Changes look good. |
Self checklist
Changelist
Completed all tasks in repos solved all tasks in sprint 1