Skip to content

Commit 4340677

Browse files
authored
Enhance comments for clarity on time format calculations
Added comments to clarify variable declarations, function calls, and expressions related to movie length.
1 parent 2d13ab8 commit 4340677

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

Sprint-1/3-mandatory-interpret/2-time-format.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,21 @@ console.log(result);
1212
// For the piece of code above, read the code and then answer the following questions
1313

1414
// a) How many variable declarations are there in this program?
15+
//[ChunYanWong] 6
1516

1617
// b) How many function calls are there?
18+
//[ChunYanWong] 1
1719

1820
// c) Using documentation, explain what the expression movieLength % 60 represents
1921
// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Arithmetic_Operators
22+
// [ChunYanWong] the remainder - remaining seconds
23+
2024

2125
// d) Interpret line 4, what does the expression assigned to totalMinutes mean?
26+
// [ChunYanWong] the movielength rounded to the neatest minutes
2227

2328
// e) What do you think the variable result represents? Can you think of a better name for this variable?
29+
// [ChunYanWong] the novie length in hours, minutes and seconds and resultHHMMSS may be better
2430

2531
// f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
32+
// [ChunYanWong] No, decimal number cannot be handled correctly

0 commit comments

Comments
 (0)