practice until to-pounds

Author: Yonas Gebre

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,21 @@ 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 4 fucntion calls 
+  //Number(carPrice.replaceAll(",", "")); 2 calls Function Number and Function replaceAll
+  //Number(priceAfterOneYear.replaceAll(",""")); 2 calls Function Number and Function replaceAll 
 // 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?
-
+// line 5 priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ""));
+//  ^^^SyntaxError: missing ) after argument list
+// add semicolon after te the first argument  in the replaceAll method to fix the error
 // c) Identify all the lines that are variable reassignment statements
+//line 4 to varaible carprice //carPrice = Number(carPrice.replaceAll(",", ""));
+//line 5 to variable priceAfterOneYear //priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ""));
 
 // d) Identify all the lines that are variable declarations
-
+//line 1 to variable carPrice //let carPrice = "10,000";
+//line 2 to variable priceAfterOneYear //let priceAfterOneYear = "8,543";
+//line 7 to variable priceDifference //const priceDifference = carPrice - priceAfterOneYear;
+//line 8 to variable percentageChange //const percentageChange = (priceDifference / carPrice) * 100;
 // e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
+// it replace all the commas with empty string and convert the string to number.

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

@@ -1,4 +1,4 @@
-const movieLength = 8784; // length of movie in seconds
+const movieLength = 60; // length of movie in seconds
 
 const remainingSeconds = movieLength % 60;
 const totalMinutes = (movieLength - remainingSeconds) / 60;
@@ -12,14 +12,22 @@ 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?
+// There are 6 variable declarations.
 
 // b) How many function calls are there?
+// one function call in the program, which is console.log(result);
 
 // c) Using documentation, explain what the expression movieLength % 60 represents
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Arithmetic_Operators
-
+//modulo operator returns left over after division. the expession movieLength % 60 returns the
+// remaining seconds after dividing the movieLength by 60. In this case, it returns 24, which is the remaining seconds after dividing 8784 by 60.
 // d) Interpret line 4, what does the expression assigned to totalMinutes mean?
-
+// the expression covert the movieLength from seconds to minutes.
+// it subtracts the remaining seconds from the movieLength and divides the result by 60 to get the total minutes. In this case, it returns 146, which is the total minutes of the movie length.
 // e) What do you think the variable result represents? Can you think of a better name for this variable?
-
+// the total length of movie in hours,minutes and seconds.
+// a better name movieLengthHMS would be more clear.
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
+// No, it will not work for all values of movieLength.
+// if movieLength is above 24 hours, the code will create an incorrect format of hours.
+// example 52:26:24 which don't make sense.

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

@@ -1,16 +1,17 @@
-const penceString = "399p";
+const penceString = "399p";// initialises a string variable with the value "399p"
 
 const penceStringWithoutTrailingP = penceString.substring(
   0,
   penceString.length - 1
-);
+);// removes the last character "p" from the penceString and stores the result in a new variable called penceStringWithoutTrailingP. The substring method is used to extract a portion of the string, starting from index 0 and ending at the second-to-last index (length - 1).
 
 const paddedPenceNumberString = penceStringWithoutTrailingP.padStart(3, "0");
+//padStart method is used to add leading zeros to the penceStringWithoutTrailingP until it reaches a length of 3 characters. The result is stored in a new variable called paddedPenceNumberString.
 const pounds = paddedPenceNumberString.substring(
   0,
   paddedPenceNumberString.length - 2
 );
-
+//
 const pence = paddedPenceNumberString
   .substring(paddedPenceNumberString.length - 2)
   .padEnd(2, "0");