Update Sprint-1 answers based on reviewer feedback: use increment term, interval notation, type conversion, fix syntax error explanation, and clarify padEnd redundancy

Author: mahmoudshaabo1984

Sprint-1/1-key-exercises/1-count.js

@@ -4,4 +4,5 @@ count = count + 1;
 
 // Line 1 is a variable declaration, creating the count variable with an initial value of 0
 // Describe what line 3 is doing, in particular focus on what = is doing
-// The code adds 1 to the count variable.
+// Line 3 is an increment operation. The = operator is the assignment operator:
+// it evaluates the expression on the right (count + 1), then assigns the result back to count.

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

@@ -9,4 +9,11 @@ const num = Math.floor(Math.random() * (maximum - minimum + 1)) + minimum;
 // Try logging the value of num and running the program several times to build an idea of what the program is doing
 
 console.log(num);
-// num represents a random integer between 1 and 100.
+
+// num represents a random integer in the range [1, 100].
+// Breakdown:
+// 1. Math.random() returns a decimal number in the range [0, 1).
+// 2. (maximum - minimum + 1) calculates the size of the range, which is 100.
+// 3. Math.random() * 100 gives a decimal in the range [0, 100).
+// 4. Math.floor() rounds down to the nearest integer, giving a value in the range [0, 99].
+// 5. Adding minimum (1) shifts the range to [1, 100].

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

@@ -1,9 +1,14 @@
-const cardNumber = "4533787178994213";
-const last4Digits = cardNumber.slice(-4);
+const cardNumber = 4533787178994213;
+const last4Digits = String(cardNumber).slice(-4);
 
 // The last4Digits variable should store the last 4 digits of cardNumber
 // However, the code isn't working
 // Before running the code, make and explain a prediction about why the code won't work
 // 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: The original code will throw a TypeError because .slice() is a string method,
+// and cardNumber is a number, not a string. Numbers do not have a .slice() method.
+// Fix: We use String(cardNumber) to convert the number to a string first,
+// then call .slice(-4) to extract the last 4 characters: "4213".

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

@@ -1,8 +1,8 @@
 let carPrice = "10,000";
 let priceAfterOneYear = "8,543";
 
-carPrice = Number(carPrice.replace(/,/g, ""));
-priceAfterOneYear = Number(priceAfterOneYear.replace(/,/g, ""));
+carPrice = Number(carPrice.replaceAll(",", ""));
+priceAfterOneYear = Number(priceAfterOneYear.replaceAll(",", ""));
 const priceDifference = carPrice - priceAfterOneYear;
 const percentageChange = (priceDifference / carPrice) * 100;
 
@@ -11,12 +11,24 @@ console.log(`The percentage change is ${percentageChange}`);
 // Read the code and then answer the questions below
 
 // a) How many function calls are there in this file? Write down all the lines where a function call is made
-// There are 5 function calls: Number() twice, replaceAll() twice, and console.log().
+// There are 5 function calls:
+// Line 4: replaceAll(",", "") and Number()
+// Line 5: replaceAll(",", "") and Number()
+// Line 9: 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?
-// replaceAll might not work in older Node versions
+// The original code on line 5 was: priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ""));
+// This causes a SyntaxError because there is a missing comma between the two arguments of replaceAll().
+// The original code had replaceAll("," "") but it should be replaceAll(",", "").
+// The comma between "," and "" is needed to separate the two arguments of the function.
+// The fix is simply adding the missing comma: replaceAll(",", "")
+
 // c) Identify all the lines that are variable reassignment statements
-// Lines 4 and 5 are variable reassignment statements.
+// Lines 4 and 5 are variable reassignment statements (carPrice and priceAfterOneYear are reassigned using let).
+
 // d) Identify all the lines that are variable declarations
-// Lines 1, 2, 7, and 8 are variable declarations.
+// Lines 1, 2, 6, 7 are variable declarations (using let or const).
+
 // e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
-// It removes the comma from the string and converts it into a Number type.
+// It removes all commas from the string "10,000" to get "10000",
+// then converts that string into the number 10000 using Number().

Sprint-1/3-mandatory-interpret/3-to-pounds.js

@@ -20,19 +20,29 @@ console.log(`£${pounds}.${pence}`);
 // Step-by-step breakdown:
 
 // 1. const penceString = "399p";
-// Initialises a string variable with the value "399p".
+// Declares a string variable with the value "399p".
 
-// 2. const penceStringWithoutTrailingP = ...
-// It removes the last character ('p') from the string to keep only the numbers.
+// 2. const penceStringWithoutTrailingP = penceString.substring(0, penceString.length - 1);
+// Removes the trailing 'p' character, resulting in "399".
 
-// 3. const paddedPenceNumberString = ...
-// It ensures the string has at least 3 characters by adding a "0" at the beginning if needed (e.g., "50" becomes "050").
+// 3. const paddedPenceNumberString = penceStringWithoutTrailingP.padStart(3, "0");
+// Pads the string to at least 3 characters with leading zeros. "399" stays "399", but "5" would become "005".
 
-// 4. const pounds = ...
-// It extracts the pounds part by taking all characters except the last two.
+// 4. const pounds = paddedPenceNumberString.substring(0, paddedPenceNumberString.length - 2);
+// Extracts all characters except the last two to get the pounds part. "399" becomes "3".
 
-// 5. const pence = ...
-// It extracts the last two characters to represent the pence part and ensures it has two digits.
+// 5. const pence = paddedPenceNumberString.substring(paddedPenceNumberString.length - 2).padEnd(2, "0");
+// Extracts the last two characters for the pence part and pads with trailing "0" if needed.
 
 // 6. console.log(`£${pounds}.${pence}`);
-// It prints the final result in the currency format: £3.99.
+// Outputs the formatted currency: £3.99
+
+// Question: Do we really need .padEnd(2, "0") in this script?
+// No, we do not actually need .padEnd(2, "0") in this script.
+// Because .padStart(3, "0") already guarantees the string has at least 3 characters,
+// .substring(length - 2) will always extract exactly 2 characters for the pence part.
+// For example:
+//   "5p"   -> "5" -> padStart -> "005" -> pence = "05" (already 2 chars, padEnd does nothing)
+//   "50p"  -> "50" -> padStart -> "050" -> pence = "50" (already 2 chars, padEnd does nothing)
+//   "399p" -> "399" -> padStart -> "399" -> pence = "99" (already 2 chars, padEnd does nothing)
+// So .padEnd(2, "0") is redundant here because padStart already ensures enough characters.