solution #1777
solution #1777
Conversation
There was a problem hiding this comment.
Review Summary
Great work on this counter application! I'm happy to approve your solution as it meets all the requirements. Your use of functional updates with setCount is excellent practice, ensuring state updates are reliable.
As a small suggestion for future improvement, you can make your event handlers a bit cleaner. For example, in your buttons, you're using onClick={() => addOne(count)}. Since the addOne function doesn't take any parameters and already has access to the count state, you can simplify this to just onClick={addOne}. This is a minor point but is a more common and concise pattern in React.
Keep up the great work!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| <button | ||
| type="button" | ||
| className="App__add-one" | ||
| onClick={() => addOne(count)} |
There was a problem hiding this comment.
The addOne function is defined without any parameters, so passing count here is unnecessary. The function can access the count state variable directly from its closure. This also applies to the onClick handlers for the other two buttons. You can simplify this call to onClick={addOne}.
2 participants
DEMO LINK