Address reviewer feedback for Sprint 1

Author: shafiekwalker7861

Sprint-1/1-key-exercises/4-random.js

@@ -10,5 +10,8 @@ const num = Math.floor(Math.random() * (maximum - minimum + 1)) + minimum;
 // It will help to think about the order in which expressions are evaluated
 // Try logging the value of num and running the program several times to build an idea of what the program is doing
 
-// Math.random() generates a random decimal, 
-// Math.floor() rounds it down, and the expression produces a random whole number between 1 and 100.
+// Math.random() creates a random decimal number.
+// The number will always be between 0 and 1, but it will never actually reach 1.
+
+// Math.floor() rounds down the decimal number that is created after
+// Math.random() is multiplied by the range.

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

@@ -1,5 +1,7 @@
 const cardNumber = 4533787178994213;
-const last4Digits = cardNumber.slice(-4);
+// I fixed the error by replacing the original expression instead of declaring
+// last4Digits a second time. Declaring a const twice causes a SyntaxError.
+const last4Digits = cardNumber % 10000;
 
 // The last4Digits variable should store the last 4 digits of cardNumber
 // However, the code isn't working
@@ -14,4 +16,4 @@ const last4Digits = cardNumber.slice(-4);
 // The error is what I predicted. The slice method is not a function that can be used on numbers, only strings.
 
 // Then try updating the expression last4Digits is assigned to, in order to get the correct value
-const last4Digits = cardNumber % 10000;
+

Sprint-1/3-mandatory-interpret/1-percentage-change.js

@@ -18,15 +18,14 @@ console.log(`The percentage change is ${percentageChange}`);
 // Line 10 calls console.log().
 
 // b) Run the code and identify the line where the error is coming from - why is this error occurring? How can you fix this problem?
-// The error was on line 5. The replaceAll() function call was missing the correct closing bracket/parenthesis.
-// I fixed it by making the line: priceAfterOneYear = Number(priceAfterOneYear.replaceAll(",", ""));
+// I changed the value from a string to a number by wrapping the replaceAll() result in Number().
 
 // c) Identify all the lines that are variable reassignment statements
 // Lines 4 and 5 are variable reassignment statements because its declared on lines 1 and 2.
 
 
 // d) Identify all the lines that are variable declarations
-// line 1 and 2 are variable declarations because they are declared using the let keyword.
+// Lines 1, 2, 7, and 8 are variable declarations because they declare variables using let and const.
 
 // e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
 //// Removes commas and converts the string to a number.

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

@@ -1,4 +1,4 @@
-const movieLength = 8784; // length of movie in seconds
+const movieLength = 3665; // length of movie in seconds
 
 const remainingSeconds = movieLength % 60;
 const totalMinutes = (movieLength - remainingSeconds) / 60;
@@ -32,8 +32,10 @@ result */
 // This gives us the total number of whole minutes in the movie length.
 
 // e) What do you think the variable result represents? Can you think of a better name for this variable?
-// The variable result represents the total length of the movie in hours, minutes, and seconds.
-// A better name for this variable could be "movieDuration".
+// The variable result stores the movie duration as a formatted string (hours:minutes:seconds).
+// A better name would be "formattedMovieLength".
 
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
-// The code works for non-negative whole numbers. Negative or decimal values may produce incorrect or unexpected results.
+// The code works when movieLength is 3665. It outputs "1:1:5", which is correct,
+// but the formatting is not ideal. A better formatted output would be "1:01:05".
+// The code works for whole numbers, but it does not pad single-digit minutes and seconds with leading zeros.