Skip to content

Commit 8c51b79

Browse files
committed
3-mandatory-interpret comments fixed
1 parent 7defa63 commit 8c51b79

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,4 +29,6 @@ console.log(result);
2929
//c) movieLength use remainder operator % . movieLength % 60 returns the remaining seconds after dividing the movie length into full minutes.
3030
//d) expression assigned to totalMinutes calculate movie time in whole minutes.
3131
//e) This variable represent total movie time in hh/mm/ss format, we could name it totalTime or totalMovieTime.
32-
//f) Yes, the code works correctly when movieLength is non-negative whole number that represents seconds.
32+
/*f) No, the code does not work correctly for all values.
33+
It works correctly when movieLength is a non-negative whole number representing the movie length in seconds.
34+
Negative numbers produce negative hours, minutes or seconds, and decimal numbers can produce decimal seconds, which does not give the expected hh:mm:ss format. */

0 commit comments

Comments
 (0)