Skip to content

Commit 14c0fe3

Browse files
Fix the error and answer the question
1 parent 2589dc7 commit 14c0fe3

File tree

2 files changed

+16
-4
lines changed

2 files changed

+16
-4
lines changed

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
const $24HourClockTime = "20:53";
22
const $12hourClockTime = "08:53";
33
console.log($24HourClockTime)
4-
console.log($12hourClockTime)
4+
console.log($12hourClockTime)
5+
//

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

Lines changed: 14 additions & 3 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,22 @@ 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
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?
17-
18+
//The error came from line 5
19+
//Because arguments inside replaceAll() were not separated properly
20+
//After fixing the comma, the program worked
21+
//Output became correct (14.57)
1822
// c) Identify all the lines that are variable reassignment statements
19-
23+
// line 4 and 5
2024
// d) Identify all the lines that are variable declarations
25+
// line 1,2,7 and 8
2126

2227
// e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
28+
// Number(carPrice.replaceAll(",", "")) does two jobs:
29+
30+
//replaceAll(",", "") → removes the commas from the string
31+
//"10,000" becomes "10000"
32+
//Number(...) → converts the cleaned string into a real number
33+
//"10000" becomes 10000

0 commit comments

Comments
 (0)