Skip to content

Commit 9d7760a

Browse files
committed
Percentage-change has been copleted successfully
1 parent 8f3d6cf commit 9d7760a

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

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

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ let carPrice = "10,000";
22
let priceAfterOneYear = "8,543";
33

44
carPrice = Number(carPrice.replaceAll(",", ""));
5-
priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ""));
5+
priceAfterOneYear = Number(priceAfterOneYear.replaceAll(",", ""));
66

77
const priceDifference = carPrice - priceAfterOneYear;
88
const percentageChange = (priceDifference / carPrice) * 100;
@@ -12,11 +12,16 @@ console.log(`The percentage change is ${percentageChange}`);
1212
// Read the code and then answer the questions below
1313

1414
// a) How many function calls are there in this file? Write down all the lines where a function call is made
15+
//5 function calls, line 4,5 and 10.
1516

1617
// 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?
18+
// Line 5,SyntaxError: missing ) after argument list just add a comma priceAfterOneYear = Number(priceAfterOneYear.replaceAll(",", ""));
1719

1820
// c) Identify all the lines that are variable reassignment statements
21+
//Line 4 ,5.
1922

2023
// d) Identify all the lines that are variable declarations
21-
24+
//1,2,7, and 8.
2225
// e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
26+
//Replace all commas with an empty string , then convert the result to a number.
27+
//the purpose is change from 10,000 formate to 10000.

0 commit comments

Comments
 (0)