Sheffield | 26-ITP-jan | Richard Frimpong | Sprint 1 | Object Destructuring#395
Sheffield | 26-ITP-jan | Richard Frimpong | Sprint 1 | Object Destructuring#395Richiealx wants to merge 5 commits intoCodeYourFuture:mainfrom
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
| total += itemTotal; | ||
|
|
||
| console.log( | ||
| `${itemName} x${quantity} = £${(itemTotal / 100).toFixed(2)}` |
There was a problem hiding this comment.
The output format does not match requirements
There was a problem hiding this comment.
The output is still not the same.
You can use a diff tool to compare your output and the expected results:
QTY ITEM TOTAL
1 Hot cakes 2.32
2 Apple Pie 2.78
1 Egg McMuffin 2.80
1 Sausage McMuffin 3.00
2 Hot Coffee 2.00
4 Hash Brown 1.60
Total: 14.50
QTY ITEM TOTAL
1 Hot Cakes 2.32
2 Apple Pie 2.78
1 Egg McMuffin 2.80
1 Sausage McMuffin 3.00
2 Hot Coffee 2.00
4 Hash Brown 1.60
Total: 14.50
There was a problem hiding this comment.
Thank you for the feedback. I have now adjusted the formatting, spacing, and capitalisation so that the output matches the expected result exactly. I re-ran the file in Node.js and confirmed the output now aligns with the task requirements.
|
Thank you for the review. I updated the destructuring exercises and fixed Exercise 3 so the output format now matches the task requirements more closely. I also re-tested all three exercises in Node.js. |
Luro91
left a comment
There was a problem hiding this comment.
Well done. The formatting is correctly now
Learners, PR Template
Self checklist
Changelist
This PR completes the Object Destructuring exercises for Sprint 1.
All exercises were tested using Node.js and outputs match the expected results.