Work on 3-mandatory-interpret

Author: Vitalii-code

Sprint-1/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,11 +12,18 @@ 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
+//
+// 3 function calls at lines:
+// 4, 5, 10
 
 // 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?
+// Because there is a missing comma at line 5. By adding the missing comma
 
 // c) Identify all the lines that are variable reassignment statements
+// 4, 5
 
 // d) Identify all the lines that are variable declarations
+// 1, 2, 7, 8
 
 // e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
+// it replaces all the occurances of "," to "" so essentially deleting commas so it can be turned into 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 = 10000; // length of movie in seconds
 
 const remainingSeconds = movieLength % 60;
 const totalMinutes = (movieLength - remainingSeconds) / 60;
@@ -12,14 +12,21 @@ console.log(result);
 // For the piece of code above, read the code and then answer the following questions
 
 // a) How many variable declarations are there in this program?
+// 6
 
 // b) How many function calls are there?
+// 1
 
 // c) Using documentation, explain what the expression movieLength % 60 represents
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Arithmetic_Operators
+// it divides the number by 60 so we know how many seconds are remaining by using the modulo operator
 
 // d) Interpret line 4, what does the expression assigned to totalMinutes mean?
+// this is to convert seconds into minutes without losing any seconds becuase we keep them in remainingSecondds
 
 // e) What do you think the variable result represents? Can you think of a better name for this variable?
+// This represents how many hours, minutes and seconds left in the movie. formattedTime.
+//
 
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
+//  Yeah I think this will work for different inputs, becuase we are using modulo so we're not losing any seconds when dividing

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

@@ -1,20 +1,26 @@
+// initialises a string variable with the value "399p"
 const penceString = "399p";
 
+// This cuts the string so we're only saving everything before the p
 const penceStringWithoutTrailingP = penceString.substring(
   0,
   penceString.length - 1
 );
 
+// This adds zeros to the start of the string if the string's length is less than 3
 const paddedPenceNumberString = penceStringWithoutTrailingP.padStart(3, "0");
+// this only saves the string minus the two letters at the end.
 const pounds = paddedPenceNumberString.substring(
   0,
   paddedPenceNumberString.length - 2
 );
 
+// this extracts the last two characters and adds zeroes to the end
 const pence = paddedPenceNumberString
   .substring(paddedPenceNumberString.length - 2)
   .padEnd(2, "0");
 
+// prints the result
 console.log(`£${pounds}.${pence}`);
 
 // This program takes a string representing a price in pence