Corrections after first review in 1.1, 1.4, 3.1, 3.2, 3.3

Author: hoimir

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

@@ -5,4 +5,8 @@ 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
 
-// Line 3 is updating the value of the count variable. The = operator is an assignment operator, which assigns the value on the right (count + 1) to the variable on the left (count). In this case, it takes the current value of count (which is 0), adds 1 to it, and then assigns the result (1) back to count. So after this line executes, count will have a new value of 1.              
+// Line 3 is updating the value of the count variable. The = operator is an assignment operator, 
+// which assigns the value on the right (count + 1) to the variable on the left (count). In this case, 
+// it takes the current value of count (which is 0), adds 1 to it, and then assigns the result (1) 
+// back to count. So after this line executes, count will have a new value of 1.
+// It has incremented with 1.              

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

@@ -14,4 +14,4 @@ const num = Math.floor(Math.random() * (maximum - minimum + 1)) + minimum;
 //The random number will always be an integer because Math.floor() is used to round
 //down the result.
 //At the end the value "minimum" is added to guarantee that the generated "num" is at least 1
-//"num" is a random number generator with results between 1 and 100. 
+//"num" is a random number generator with results between 1 and 100, including 1 but excluding 100 

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

@@ -17,14 +17,14 @@ console.log(`The percentage change is ${percentageChange}`);
 
 carPrice = Number(carPrice.replaceAll(",", ""));
 priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ""));
-
+Number; //called twice
 const priceDifference = carPrice - priceAfterOneYear;
 const percentageChange = (priceDifference / carPrice) * 100;
 
 
 // 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?
 
-// There is a comma missing in: replaceAll("," ""));  
+// There is a comma missing between te arguments in: replaceAll("," ""));  
 // It should be: replaceAll(",", "")); 
 
 

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

@@ -19,7 +19,7 @@ console.log(result);
 
 // b) How many function calls are there?
 
-// There are 2: console.log and the template literal for the result string 
+// There is 1: console.log 
 
 
 // c) Using documentation, explain what the expression movieLength % 60 represents
@@ -41,7 +41,9 @@ const totalMinutes = (movieLength - remainingSeconds) / 60;
 
 // "result" represents the result of the calculations, being the total length of the movie 
 // transcribed to hours/minutes/seconds. It would have been much clearer to not give the mathematical 
-// output as the name but instead the usability of this output, such as "movieLengthInHours"
+// output as the name but instead the usability of this output, such as "movieLengthHMS" or the clearer 
+// but rather lengthy movieLengthHoursMinutesSeconds
+
 
 
 

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

@@ -13,7 +13,6 @@ const pounds = paddedPenceNumberString.substring(
 
 const pence = paddedPenceNumberString
   .substring(paddedPenceNumberString.length - 2)
-  .padEnd(2, "0");
 
 console.log(`£${pounds}.${pence}`);
 

Sprint-2/Project-CLI-Treasure-Hunt

@@ -0,0 +1 @@
+Subproject commit 4374e4e6b862b0a110e806b6115f3f6b3c22b26a