Advanced calculator #2922
Advanced calculator #2922
Conversation
src/makeCalculator.js
Outdated
| return this; | ||
| }, | ||
| operate(callback, number) { | ||
| calculator.result = callback(number); |
There was a problem hiding this comment.
result of operation assinged to result 2 times. First time inside arithmetic operation and second time directly from callback return
src/makeCalculator.js
Outdated
| const calculator = { | ||
| result: 0, | ||
| add(number) { | ||
| calculator.result += number; |
There was a problem hiding this comment.
fields from parent object should be called using keyword this
Same to all below
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Excellent work on your calculator implementation! Your solution meets all the functional and style requirements outlined in the task. The code is clean, logical, and demonstrates a strong understanding of the concepts involved. Your use of .call() within the operate method to manage the context of the callback functions is a particularly great solution. I'm happy to approve this submission.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
3 participants
Solution