Address Sprint 2 review feedback

Author: codebyra

Sprint-2/2-mandatory-errors/1.js

@@ -2,3 +2,8 @@
 
 let age = 33;
 age = age + 1;
+
+// The original code used const to declare age.
+// A const variable cannot be reassigned after declaration.
+// Attempting to reassign age caused a TypeError.
+// Changing const to let fixes the problem because let allows reassignment.

Sprint-2/2-mandatory-errors/2.js

@@ -3,3 +3,8 @@
 
 const cityOfBirth = "Bolton";
 console.log(`I was born in ${cityOfBirth}`);
+
+// The original code tried to use cityOfBirth before it was initialized.
+// This caused a ReferenceError because variables declared with const
+// cannot be accessed before their declaration.
+// Moving the declaration before console.log fixes the problem.

Sprint-2/2-mandatory-errors/3.js

@@ -7,3 +7,14 @@ const last4Digits = cardNumber.toString().slice(-4);
 // Then run the code and see what error it gives.
 // Consider: Why does it give this error? Is this what I predicted? If not, what's different?
 // Then try updating the expression last4Digits is assigned to, in order to get the correct value
+
+// Prediction: I expected the code to fail because cardNumber
+// is a number, and slice() is a string method.
+//
+// Actual result: The original code produced a TypeError
+// because cardNumber.slice is not a function.
+//
+// The fix converts cardNumber into a string using toString()
+// before calling slice(-4) to extract the last four digits.
+//
+// The actual error matched my prediction.

Sprint-2/2-mandatory-errors/4.js

@@ -1,2 +1,7 @@
 const twelveHourClockTime = "8:53pm";
 const twentyFourHourClockTime = "20:53";
+
+// The original variable names started with numbers.
+// JavaScript variable names cannot begin with a number.
+// This caused a SyntaxError because the names were invalid.
+// Renaming the variables to start with letters fixes the problem.

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

@@ -1,4 +1,4 @@
-const movieLength = 8784; // length of movie in seconds
+const movieLength = 90.5;
 
 const remainingSeconds = movieLength % 60;
 const totalMinutes = (movieLength - remainingSeconds) / 60;
@@ -31,5 +31,16 @@ console.log(result);
 // A better variable name would be formattedMovieLength.
 
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
-// Answer: No, it will not format all values correctly.
-// For example, single-digit minutes or seconds will not have a leading zero.
+// // Answer: No, the code does not work correctly for all values.
+//
+// Testing -90 produced 0:-1:-30, which is not a valid time format.
+// Negative durations should not be accepted.
+//
+// Testing 90.5 produced 0:1:30.5, showing that the program
+// does not handle fractional seconds appropriately.
+//
+// The program also does not add leading zeros to single-digit
+// minutes or seconds.
+//
+// The program should validate its input and format the
+// output correctly.