Skip to content

Commit 80fd388

Browse files
committed
Update comments for clarity and detail in time format calculations
1 parent 3d0981c commit 80fd388

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

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

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,20 @@ 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+
//6 variable declarations.
1516

1617
// b) How many function calls are there?
18+
//1 function call, on line 10.
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+
//The remainder operator returns the remainder left over when movieLength is divided by 60.
2023

2124
// d) Interpret line 4, what does the expression assigned to totalMinutes mean?
25+
// The expression (movieLength - remainingSeconds) / 60 calculates the total number of whole minutes in the movie length. It first subtracts the remaining seconds from the total movie length to get a value that is a multiple of 60, and then divides that value by 60 to convert it from seconds to minutes. It gives it as a whole number and is easier to work with.
2226

2327
// e) What do you think the variable result represents? Can you think of a better name for this variable?
28+
// It represents the movie duration in the format of hours:minutes:seconds. A better name for this variable could be movieDuration.
2429

2530
// f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
31+
// It does not work for negative values. The code assumes that movieLength is a positive integer representing the length of a movie in seconds. If movieLength is negative, the calculations for remainingSeconds, totalMinutes, remainingMinutes, and totalHours will not produce meaningful results, and the final result will not represent a valid time format.

0 commit comments

Comments
 (0)