Skip to content

Create Week10 Mission01#123

Open
woojo230 wants to merge 1 commit intomainfrom
woojo230
Open

Create Week10 Mission01#123
woojo230 wants to merge 1 commit intomainfrom
woojo230

Conversation

@woojo230
Copy link
Collaborator

@woojo230 woojo230 commented Jun 2, 2025

📝 미션 번호

10주차 미션 1

📋 구현 사항

  • React.memo, useMemo, useCallback 사용하여 검색 최적화

📎 스크린샷

2025-06-02.135923.mp4

✅ 체크리스트

  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • 불필요한 주석이 제거되었나요?
  • 코드 스타일이 일관적인가요?

🤔 질문 사항

@woojo230 woojo230 requested a review from hyesngy June 2, 2025 05:05
@woojo230 woojo230 self-assigned this Jun 2, 2025
@woojo230 woojo230 changed the title Create Week1 Mission01 Create Week10 Mission01 Jun 2, 2025
Copy link
Member

@hyesngy hyesngy left a comment

Choose a reason for hiding this comment

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

10주차 워크북까지 정말 긴 여정이었습니다.

그동안 모두 고생 많으셨습니다! 👏🏻👏🏻👏🏻 짧지 않은 기간동안 의미 있는 시간이 되었기를 바랍니다.

이제 기본 워크북은 모두 완주하셨지만, 11-12주차 워크북도 도전해보시길 추천드립니다! 11주차에서는 Vercel을 활용한 배포와 CI/CD 구성을 다루고, 12주차에서는 WebSocket을 이용한 실시간 통신과 Cypress를 활용한 E2E 테스트를 다룹니다.

앞으로도 꾸준히 학습하고 성장하시길 응원하며, 데모데이까지 화이팅 입니다! 👍🏻👍🏻👍🏻

Copy link
Member

Choose a reason for hiding this comment

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

현재 components 폴더 내에서 movie, signup 등으로 도메인별 분리가 잘 되어 있고, 세부 컴포넌트 분리 및 전체적인 구조가 명확하여 가독성이 매우 좋아요 ! 👍🏻

Copy link
Member

Choose a reason for hiding this comment

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

home.tsx에서 React.memo, useCallback, useMemo를 적절히 활용하여 불필요한 리렌더링을 방지하고 있고, MovieItem과 MovieSearchItem에서 memo 처리로 성능 최적화가 효율적으로 구현이 잘 되어 있습니다. 👍🏻

Copy link
Member

Choose a reason for hiding this comment

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

🚀Challenge 미션에 따라 프로젝트 전체의 성능 최적화나 UX 개선에도 도전해보세요! 코드 품질 측면에서도 중복 제거, 책임 분리, 예외 처리 개선 등을 통해 가독성과 유지보수성이 좋은 코드로 리팩토링 해보면 어떨까요 😊

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