Skip to content

Create Week10 Mission 1#121

Open
wantkdd wants to merge 2 commits intomainfrom
wantkdd
Open

Create Week10 Mission 1#121
wantkdd wants to merge 2 commits intomainfrom
wantkdd

Conversation

@wantkdd
Copy link
Member

@wantkdd wantkdd commented Jun 2, 2025

📝 미션 번호

10주차 Misson 1

📋 구현 사항

  • 검색 기능 구현
  • useCallback, memo 사용하여 불필요한 리렌더링 제거

📎 스크린샷

bandicam.2025-05-31.14-46-08-006.mp4
bandicam.2025-05-31.14-44-52-535.mp4

✅ 체크리스트

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

🤔 질문 사항

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 테스트를 다룹니다.

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

);
};

export default memo(MovieFilter);
Copy link
Member

Choose a reason for hiding this comment

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

movie-filter.tsx에서 React.memo를 적용하여 불필요한 리렌더링을 방지한 점과, useCallback으로 handleFilterChange 함수를 최적화한 점에서 검색 기능이 효율적으로 잘 구현된 것 같습니다!

Copy link
Member

Choose a reason for hiding this comment

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

현재 UI와 로직이 한 파일에 모두 들어가 있는 페이지들이 꽤 있는데, 🛼 Standard Mission에 따라 컴포넌트를 기능 단위로 분리해서 관리하면 가독성, 재사용성, 유지보수 측면에서 훨씬 좋을 것 같습니다!

Copy link
Member

Choose a reason for hiding this comment

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

현재 회원가입 페이지 파일이 2개 있는데 중복된 파일이므로, 하나로 통합하여 사용되지 않는 파일은 제거해주세요!

Copy link
Member

Choose a reason for hiding this comment

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

component 폴더 내에 navbar, 페이지네이션, 로딩스피너, 영화 관련 컴포넌트 등 다양한 컴포넌트들이 동일한 레벨에 위치되어 있습니다. 이를 도메인 또는 기능별로 폴더를 나누어 구조화하면 구조가 더 명확해질 것입니다!

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