Skip to content

Create Week10 Mission1#124

Open
geg222 wants to merge 1 commit intomainfrom
geg222
Open

Create Week10 Mission1#124
geg222 wants to merge 1 commit intomainfrom
geg222

Conversation

@geg222
Copy link
Collaborator

@geg222 geg222 commented Jun 2, 2025

📝 미션 번호

10주차 Misson 1

📋 구현 사항

  • 영화 검색 기능
  • React.memo, useCallback, useMemo 등을 활용하여 영화부분만 리렌더링

📎 스크린샷

2025-06-02.3.57.03.mov

✅ 체크리스트

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

🤔 질문 사항

feat: Create Week10 Mission1
@geg222 geg222 self-assigned this 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.

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

Comment on lines +9 to +15
const SearchPage = () => {
const [filters, setFilters] = useState<MovieFilters>({
query: "",
include_adult: false,
language: "ko-KR",
});
const [selectedMovieId, setSelectedMovieId] = useState<number | null>(null);
Copy link
Member

Choose a reason for hiding this comment

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

현재 영화 검색 결과에서 무한 스크롤 또는 페이지네이션이 구현되어 있지 않습니다. 이에 따라 사용자가 검색 결과를 모두 확인하지 못하고 1페이지만 조회 가능합니다. 다른 카테고리별 영화 페이지처럼 페이지네이션을 적용하거나 무한스크롤을 구현하여 개선하면 UX가 크게 향상될 것 같습니다!

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