Skip to content

Conversation

@ksh0323a
Copy link

counter-hw 에 과제코드 작성하였습니다.

@yesjin-git
Copy link
Owner

yesjin-git commented Dec 15, 2019

전체적으로 매우 매우 잘해주셨습니다. 역대 과제 해오신 분들 중에서 가장 잘짜주신 것 같아요 ㅎㅎ
A+드립니다.

개선할 부분 한가지만 말씀드리면 clearInterval을 stop 이외에 component will unmount 상황에서도 해주실 필요가 있습니다.
https://reactjs.org/docs/state-and-lifecycle.html

https://stackoverflow.com/questions/43508744/how-do-componentwillunmount-work-with-clearinterval

위 예제와 아래 문서 참고하시면 좋을 것 같아요.

@ksh0323a
Copy link
Author

전체적으로 매우 매우 잘해주셨습니다. 역대 과제 해오신 분들 중에서 가장 잘짜주신 것 같아요 ㅎㅎ
A+드립니다.

개선할 부분 한가지만 말씀드리면 clearInterval을 stop 이외에 component will unmount 상황에서도 해주실 필요가 있습니다.
https://reactjs.org/docs/state-and-lifecycle.html

https://stackoverflow.com/questions/43508744/how-do-componentwillunmount-work-with-clearinterval

위 예제와 아래 문서 참고하시면 좋을 것 같아요.

넵. 리뷰 감사합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants