Skip to content

Commit f20321b

Browse files
author
lintsang
committed
recorrect 3-mandatory-interpret/2-time-format.js
1 parent 910726e commit f20321b

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ console.log(result);
2525
// Line 4 is calculating the quotient of the calculation (8784-remainder) divided by 60 to find the movie length in minute. That will be later used to find the display minute in the Hour:Minute:Second format.
2626

2727
// e) What do you think the variable result represents? Can you think of a better name for this variable?
28-
// I think the original movie length is in second and this code is to calculate the display of movie length in the Hour:Minute:Second format like what what we see in a normal movie information.
28+
// I think the original movie length is in second and this code is to calculate the display of movie length in the Hour:Minute:Second format like what what we see in a normal movie information. I would suggest using durationFormatted as the variable name as it could explain to others clearly what it contains.
2929

3030
// f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
3131
//When testing the output with different values of movie length, the output seems right. No matter a 0 value, small or large value, it can display the the Hour:Minute:Second format well.

0 commit comments

Comments
 (0)