corrected issues identified by the reviewer

Author: Mugs3

Sprint-2/1-key-exercises/2-initials.js

@@ -1,11 +1,8 @@
 const firstName = "Creola";
 const middleName = "Katherine";
 const lastName = "Johnson";
-// const initial = "initials";
-// const index = 1;
 
-// console.log('The ${Initial} ${index} is ${firstName.charAt(index)}');
-// Declare a variable called initials that stores the first character of each string.
+
 // This should produce the string "CKJ", but you must not write the characters C, K, or J in the code of your solution.
 
 const initials = firstName[0] + middleName[0] + lastName[0];

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

@@ -2,6 +2,8 @@ const minimum = 1;
 const maximum = 100;
 
 const num = Math.floor(Math.random() * (maximum - minimum + 1)) + minimum;
+// (0.68 * 100 ) + 1
+console.log(num)
 
 // In this exercise, you will need to work out what num represents?
 // Try breaking down the expression and using documentation to explain what it means
@@ -10,4 +12,9 @@ const num = Math.floor(Math.random() * (maximum - minimum + 1)) + minimum;
 
 // Answer: For calculations i utilised BODMAS formula solving numbers in brackets first, multiplication, subtraction and addition
 // I used 0.68 for math.floor(random number) + 1
-// Sum = 69
+// Sum = 69
+// 1.num represents a number which is an Integer that is greater or equal to 1
+// 2.start by (maximum-minimum +1) which the output is 100
+// 3.math.random()*100 returns random number between 0 and 100. Any random number less that 1 can be selected then multiplied by 100
+// 4.math.floor() gives out the largest integer which is less than or equal to the given number which is a decimal
+// The output is displayed in console log

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

@@ -1,9 +1,9 @@
 // trying to create an age variable and then reassign the value by 1
 
-const age = 33;
+let age = 33;
 age = age + 1;
 
-console.log(age)
+console.log(age);
 
 // The TypeError: Assignment to constant variable implies we are trying to reassign the variable twice.
-// This case, I have used let instead in order to allow the variable to be reused.
+// This case, I have used let instead in order to allow the variable to be reused.

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

@@ -1,5 +1,6 @@
-const cardNumber = 4533787178994213;
+const cardNumber = "4533787178994213";
 const last4Digits = cardNumber.slice(-4);
+console.log(last4Digits);
 
 // The last4Digits variable should store the last 4 digits of cardNumber
 // However, the code isn't working
@@ -11,4 +12,4 @@ const last4Digits = cardNumber.slice(-4);
 //Prediction was the code would run successfully without error although with the wrong results due to absence of syntax errors in the file
 // Error returned: TypeError: cardNumber.slice is not a function
 // Lesson learnt here; slice method is only available for strings or arrays not numbers
-// So converted the cardNumber into a string first
+// So converted the cardNumber into a string first

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

@@ -2,7 +2,7 @@ let carPrice = "10,000";
 let priceAfterOneYear = "8,543";
 
 carPrice = Number(carPrice.replaceAll(",", ""));
-priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," , ""));
+priceAfterOneYear = Number(priceAfterOneYear.replaceAll(",", ""));
 
 const priceDifference = carPrice - priceAfterOneYear;
 const percentageChange = (priceDifference / carPrice) * 100;
@@ -12,12 +12,14 @@ 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
-// 2
-//carPrice = Number(carPrice.replaceAll(",", ""));
-//priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ","));
+// 5
+// * number() and .replaceAll() in line 4
+// * number() and .replaceAll()in line 4
+// 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?
 //b) error = SyntaxError: missing ) after argument list
 // (",", ",")); - added , between quoted values
+// The error was coming from line 5 due to a missing comma in the replaceAll() method
 // c) Identify all the lines that are variable reassignment statements
 //carPrice = Number(carPrice.replaceAll(",", ""));
 //priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ","));
@@ -27,4 +29,4 @@ console.log(`The percentage change is ${percentageChange}`);
 //const priceDifference = carPrice - priceAfterOneYear;
 //const percentageChange = (priceDifference / carPrice) * 100;
 // e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
-// e) cleans the amount format by removing characters such as , and leaving only number
+// e) cleans the amount format by removing characters such as , and leaving only number

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

@@ -15,16 +15,19 @@ console.log(result);
 // 6 variables
 
 // b) How many function calls are there?
-// 2
+// 1 (console.log())
 // c) Using documentation, explain what the expression movieLength % 60 represents
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Arithmetic_Operators
 // It means 60 % remainder of Movie length
+// 24 seconds
 
 // d) Interpret line 4, what does the expression assigned to totalMinutes mean?
 // totalMinutes is assigned a value of the result from (movieLength - remainingSeconds) / 60;
+//it means changing the value that was in second into minute by dividing it 60 - making it a whole number
 
 // e) What do you think the variable result represents? Can you think of a better name for this variable?
 // I think its the total movie length with a timer. Based on research it appears to be template literal variable as it mixes static text with dynamic data. Sorry I don't fully understand this bit yet
+// It represents the MovieDuration
 
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
 //any number greater than zero returns a valid positive hour, minute or seconds result. Changing the length to 0 returns 0:0:0. Any negative length returns negative values

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

@@ -2,13 +2,13 @@ const penceString = "399p";
 
 const penceStringWithoutTrailingP = penceString.substring(
   0,
-  penceString.length - 1
+  penceString.length - 1,
 );
 
 const paddedPenceNumberString = penceStringWithoutTrailingP.padStart(3, "0");
 const pounds = paddedPenceNumberString.substring(
   0,
-  paddedPenceNumberString.length - 2
+  paddedPenceNumberString.length - 2,
 );
 
 const pence = paddedPenceNumberString
@@ -26,7 +26,9 @@ console.log(`£${pounds}.${pence}`);
 // To begin, we can start with
 // 1. const penceString = "399p": initialises a string variable with the value "399p"
 // 2. const penceStringWithoutTrailingP = penceString.substring(0): sets penceStringWithoutTRailingP = 399.0 - penceString -1 = 39
+// 2. P will be dropped
 // 3. const paddedPenceNumberString - ensures the figure is 3 characters to taking us back to 399
 // 4 const pounds - removes 2 characters from the amount = 3
 // 5. const pence - adds the amount by 2 characters taking us back to either 39 or 99
-// 6. console displays the figures in pounds and pence = 3.99
+// 5. const pence is initialised in 2 methods 1)extracts the string from index 2)adds 0 hence =  99
+// 6. console displays the figures in pounds and pence = 3.99