Skip to content

10주차 -코몬/강민규 워크북 과제제출#78

Merged
onebone merged 1 commit intoKWU-UMC:comonfrom
kmk0207:main
Dec 20, 2025
Merged

10주차 -코몬/강민규 워크북 과제제출#78
onebone merged 1 commit intoKWU-UMC:comonfrom
kmk0207:main

Conversation

@kmk0207
Copy link

@kmk0207 kmk0207 commented Dec 19, 2025

✅ 10주차 워크북 완료 🎉

워크북을 완료한 챌린저분들, 수고 많으셨습니다!
이곳에 학습 진행 중 트러블슈팅(문제 해결 경험), 커밋별 설명, 그리고 소감을 자유롭게 남겨주세요.
서로의 경험을 공유하며 성장해봐요! 🚀


🛠️ 트러블슈팅 (Troubleshooting)


문제


원인 분석


해결 방법



📌 커밋별 설명 (Commit Log)

[커밋 이름](커밋 주소) : 설명 적기 ~~
+ 이미지도 첨부해주시면 좋아요 ~

@YuJin020303
Copy link

  1. App.tsx에 api 호출 로직이 직접 포함되어 있는데, 해당 로직은 커스텀훅으로 만들어서 사용하는 것이 더 좋을 것 같습니다.
  2. TMDB_TOKEN이 하드 코딩 되어있습니다!! .env파일로 빼는게 좋을 것 같습니다.

Copy link
Contributor

@onebone onebone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

10주차 워크북도 고생 많으셨습니다! 서브모듈로 작업해두셨군요👀 여러모로 잘 정돈된 코드인 것 같습니다👍 한 가지 짚어보자면 memo는 props를 비교하는 오버헤드를 추가해서 너무 많이 사용되면 오히려 역효과를 낼 수도 있어요. 예를 들어 MovieModal 같은 경우 보통 사용자가 영화를 클릭했을 때만 렌더링 될 거라서 비용이 낮을 가능성이 높아서 한 번 고려해보면 좋을 것 같아요. 이외에도 매번 props가 바뀔 수밖에 없는 컴포넌트나 children을 받는 컴포넌트도 memo를 쓰는 건 다시 고려해보는 게 좋습니다!

아무쪼록 10주차까지 같이 달려와주셔서 감사드리고 프로젝트에서 좋은 결과 내시기 바랍니다! 💪

@onebone onebone merged commit c7280b6 into KWU-UMC:comon Dec 20, 2025
1 check passed
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.

3 participants