Modify movieLength and enhance comments for clarity

Updated movieLength to a negative value and added comments for clarity on variable declarations and function calls. Suggested improvements for handling negative movie lengths.

Author: Rawan0-0

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

@@ -1,5 +1,5 @@
-const movieLength = 8784; // length of movie in seconds
-
+const movieLength = -1000000; // length of movie in seconds
+//not complete
 const remainingSeconds = movieLength % 60;
 const totalMinutes = (movieLength - remainingSeconds) / 60;
 
@@ -11,17 +11,21 @@ console.log(result);
 
 // For the piece of code above, read the code and then answer the following questions
 
-// a) How many variable declarations are there in this program? 
+// a) How many variable declarations are there in this program?
 // 6 declarations
 // b) How many function calls are there?
 // 1 function call in line 10
 // c) Using documentation, explain what the expression movieLength % 60 represents
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Arithmetic_Operators
-// "operator returns the remainder left over when one operand is divided by a second operand. It always takes the sign of the dividend."
+// it represent the number of seconds left over after taking all the complete minutes
+
 // d) Interpret line 4, what does the expression assigned to totalMinutes mean?
-// it means the number of minutes in a film is calculated by calculating the number of seconds in the film
+//here in the expression we have the movieLength taken from it the number of remainingSeconds, so the operator / 60 represents the conversion of the remaining seconds into minuttes, so we left with complete minutes only
 // e) What do you think the variable result represents? Can you think of a better name for this variable?
 //the variable result represesnt the length of the film in hours,minutes,and seconds. A better name would be movie time.
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
-//  However, it doesn't work when the leading number of a decimal is zero like this 08784 because it interpreted as octal literal and 8 can't work in octal numeration 
-//"The octal number system is a base-8 math and computing system that uses only eight digits: 0, 1, 2, 3, 4, 5, 6, and 7"
+//8784
+//if experimented values into the movieLength variable what will happen? Lets start with the following values:
+//-10 , 0, 10 , 10000000, -1000000
+//0:0:-10, 0:0:0,0:0:10,2777:46:40, -277:-46:-40
+//ok I notice the result get strange with big negative numbers, so I think the code needs to be modified to restrict using negative values