Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6주차] Team 포토그라운드 강다혜 & 이희원 미션 제출합니다. #12

Open
wants to merge 67 commits into
base: main
Choose a base branch
from

Conversation

hiwon-lee
Copy link

@hiwon-lee hiwon-lee commented Nov 16, 2024

6주차 미션: Next-Netflix

결과물

Key Questions

무한 스크롤과 Intersection Observer API의 특징에 대해 알아봅시다.

무한 스크롤
무한 스크롤은 스크롤을 무한으로 할 수 있는 기능이다.
페이지 최하단에 도달했을 때 신규 콘텐츠를 로드하는 식으로 무한 스크롤이 구성된다.
이 전에는 페이지네이션을 활용해 콘텐츠를 확인하는 방식을 사용했다.

무한스크롤의 장점 : 별도의 추가 동작이 필요하지 않음.
무한스크롤의 단점 : 한 페이지 내에 많은 콘텐츠가 로드 되므로, 페이지 성능이 느려진다. 많은 컨텐츠 로드 후, 눈여겨봤던 콘텐츠로 다시 돌아가기 어렵다.

Intersection Oberserver API
해당 API는 무한 스크롤을 구현할 수 있도록 돕는다.
Intersection Oberserver(교차 관찰자)는 요소(Element)가 뷰포트(ViewPort)와 교차하고 있는 지 감지한다. 즉, 관찰 중인 요소가 사용자가 보는 화면 영역 내에 들어왔는 지 알려준다.

객체 생성

new Intersection Observer(callback, options)

객체 생성 시 root(관찰 대상이 화면에 들어왔음을 감지하는 영역), rootMargin(감지 영역을 바깥 범위까지 확장), threshold(관찰 대상이 화면 내 얼마나 들어왔을 때 콜백 함수를 콜백할 지) 등의 값을 작성하면 된다.

tanstack query의 사용 이유(기존의 상태 관리 라이브러리와는 어떻게 다른지)와 사용 방법(reactJS와 nextJS에서)을 알아봅시다.

tanstack query(이전 이름 : React Query)
웹 애플리케이션에서 데이터 가져오기, 캐싱 동기화 및 서버 상태 업데이트를 간편하게 만들어주는 리액트 라이브러리다.
즉, React에서 API 요청과 상태 관리를 쉽게 해주는 도구다.
React나 Next.js와 함께 사용하는 경우 서버에서 데이터를 효율적으로 가져오고 캐싱 및 동기화를 관리할 수 있게 해준다.

특징 TanStackQuery 상태관리 라이브러리
사용 목적 서버 상태 관리 (API 데이터 패칭, 캐싱) 클라이언트 상태 관리
주요 기능 데이터 패칭, 캐싱, 무효화, 동기화, 리트라이 애플리케이션의 전역 상태 관리
복잡성 간단 (비동기 로직 추상화) 상태 및 비동기 로직 직접 구현 필요
리덕스 미들웨어 필요성 불필요 필요 (예: Redux-Thunk, Redux-Saga)
캐싱 지원
지원
직접 구현 필요

[ReactJS, Next.js에서의 사용]

ReactJS :

  • CSR을 기본으로 동작하며, 데이터는 클라이언트에서 fetching

Next.js :

  • SSR이나 SSG(Static-Site Generation)을 활용해 초기 상태를 서버에서 준비할 수 있다.
  • Hydrate로 서버에서 패칭한 데이터를 클라이언트에 주입한다.

[사용 방법]

  • 데이터 캐싱
    • 항상 쿼리 키를 지정해 데이터를 가져온다. 이 쿼리 키를 통해 캐시된 데이터를 사용할 지, 새로운 데이터를 가져올 지를 판단한다.
  • 데이터의 신선도
    • 캐시한 데이터를 fresh | stale 상태로 구분해 관리한다.
    • fresh -> 데이터 사용
    • stale -> 서버에 다시 요청해 신선한 데이터를 가져온다.
    • staleTime : 데이터가 상하는 데까지 걸리는 시간
    • isStale : fresh | stale 판단
import { useQuery } from '@tanstack/react-query' 
export default function DelayedData() { 
	const { data, isStale } = useQuery({ 
		queryKey: ['delay'], 
		queryFn: async () => (await fetch('https://api.heropy.dev/v0/delay?t=1000')).json(), 
		staleTime: 1000 * 10 // 10초 후 상함. 즉, 10초 동안 신선함. 
		}) 
	return ( <> 
		<div>데이터가 {isStale ? '상했어요..' : '신선해요!'}</div> 
		<div>{JSON.stringify(data)}</div> 
		</> ) 
}

기본적인 git add, commit, push, pull, merge, rebase 등의 명령어에 대해 알아봅시다(+ git branch 전략이나 다른 git 명령어도 좋습니다!)

Git에서의 흐름을 기준으로 git 명령어에 대해 설명한다.

  1. 작업 디렉토리 (Working Directory):
    • 실제로 코드 작업을 하는 디렉토리다.
    • 파일을 생성하거나 수정하면, 변경 사항은 "Tracked" 상태(추적 중) 또는 "Untracked" 상태(추적되지 않음)가 된다.
  2. 스테이징 영역 (Staging Area):
    • 변경된 파일을 git add 명령어로 스테이징 영역에 올립니다.
    • 스테이징된 파일만이 commit 대상이 됩니다.
  3. 로컬 리포지토리 (Local Repository):
    • 스테이징 영역의 변경 사항을 git commit 명령어로 로컬 리포지토리에 저장
  4. 원격 리포지토리 (Remote Repository):
    • 로컬 리포지토리에서 커밋된 내용을 git push로 원격 리포지토리에 업로드

Merge와 Rebase
특히 협업 할 때 중요하 쓰인다.

특징 Merge Rebase
히스토리 병합 커밋 생성, 브랜치 구조 유지 히스토리가 직선으로 재배치됨
커밋 내역 확인 브랜치별 작업 흐름이 명확히 보임 히스토리가 깔끔하고 간단해짐
협업 안전성 충돌 관리가 비교적 쉬움 충돌 발생 시 작업 복잡도가 증가
사용 사례 협업 중 공유된 브랜치에서 병합 시 사용 개인 작업 정리 또는 feature 브랜치 정리 시 사용

@hiwon-lee hiwon-lee changed the title [6주차] 포토그라운드 미션 제출합니다. [6주차] Team 포토그라운드 강다혜 & 이희원 미션 제출합니다. Nov 16, 2024
Copy link

@hae2ni hae2ni left a comment

Choose a reason for hiding this comment

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

수고하셨습니다아아아 체고체고! 되게 이것저것 뜯어볼게 많았던 코드인 것 같아요,,, 시간이슈인게 너무 슬프네여,,,ㅠㅠ 배포 링크 한번만 확인만 부탁드립니다!

export default function DetailScreen({
data,
}: {
data: { title: string; poster_path: string; overview: string };
Copy link

Choose a reason for hiding this comment

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

혹시 interface로 따로 type을 지정하지 않으신 이유가 있으신가용

Choose a reason for hiding this comment

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

저도 궁금합니다! 자주 쓰일 수 있는 인터페이스라면 파일에 따로 정의해두고 꺼내써도 좋을 것 같아요~

Comment on lines +6 to +12
function shuffleArray(array: MovieProps[]) {
for (let i = array.length - 1; i > 0; i--) {
const j = Math.floor(Math.random() * (i + 1));
[array[i], array[j]] = [array[j], array[i]]; // 요소 교환
}
return array;
}
Copy link

Choose a reason for hiding this comment

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

요렇게 다시 섞은 이유가 있오?

Comment on lines +10 to +15
<div className="topTenIcon">
<p className="top">Top</p>
<p className="ten">10</p>
</div>
<div className="title">#2 in Nigeria Today</div>
</TopTen>
Copy link

Choose a reason for hiding this comment

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

styled component는 그 자체에 모드ㄴ 스타일 요소를 반영해서 한 눈에 알아보기 위해 사용하는 부분도 있으니, className을 사용하시는 것도 좋지만 바로바로 알 수 있는 요소를 사용하시는 것도 좋을 것 같아요!
만약에 겹치는 요소가 많아서 className을 쓰신 거라면,
공통적인 요소를 미리 선언하고, 그걸 끌고 오는 형태로 바꿔도 좋을 것 같습니다!

const Common = css`  
 display: flex;
  justify-content: center;
  align-items: center;
  gap: 3rem;
  margin-bottom: 2.5rem;`
  
  const Ten = styled.p`
${Common}
어쩌구
`

Copy link
Author

Choose a reason for hiding this comment

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

오...역시

font-size: 26.75px;
font-weight: 700;
line-height: 20px;
letter-spacing: -0.07339449226856232px;
Copy link

Choose a reason for hiding this comment

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

...? 롸

font-size: 20.92px;
font-weight: 700;
line-height: 15.64px;
letter-spacing: -0.05740566551685333px;
Copy link

Choose a reason for hiding this comment

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

광기아니묘

Copy link
Author

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

letter-spacing: 0.7511110901832581px;
`;

export default function SearchList({
Copy link

Choose a reason for hiding this comment

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

그 searchList가 searchPage 쪽 컴포넌트인 것 같은데 현재 배포 페이지에는 아예 나오지가 않아요ㅠㅠ 확인 부탁드려요!

return (
<html lang="ko">
<body>
<StyledComponentsRegistry>{children}</StyledComponentsRegistry>
Copy link

Choose a reason for hiding this comment

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

요 친구가 어떤 컴포넌트인지는 아직 확인하지 못했지만 전체적인 레이아웃에서 footter navbar가 화면이랑 떨어져서 나오고 있습니다! 확인해주세용!

</TransparentButton>
</InputContainer>

<SearchList query={query} movieList={movieList} />
Copy link

Choose a reason for hiding this comment

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

요고가 지금 나타나질 않고 있어서,,,ㅠ

@@ -0,0 +1,16 @@
export default function SearchIcon() {
Copy link

Choose a reason for hiding this comment

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

혹시 svg를 그냥 파일로 public 폴더에 넣지 않은 이유가 있으신가요?

<ButtonContainer
className="w-full h-full text-center"
type={type}
// onClick={onClick}
Copy link

Choose a reason for hiding this comment

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

주석은 지워주세용!

Copy link

@jiwonnchoi jiwonnchoi left a comment

Choose a reason for hiding this comment

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

이번주 과제까지 마무리하시느라 너무 수고하셨습니다🥹👍
시간이슈로 코드리뷰를 세세히 하지 못해 너무 아쉽고,, 죄송하고,, 다음엔 꼭 시간들여 뜯어보겠습니다😭 디바운스 적용해주신 부분 저랑은 조금 다르게 쓰쎠서 덕분에 좀 더 찾아보고 갑니다! 또 배포링크에서 홈화면 데이터가 굉장히 빨리 뜨는데 좋으네요,,,⭐
다음 마지막 과제까지 화이팅합시당!!

Choose a reason for hiding this comment

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

github action 저도 써보고 싶었는데 다음엔 시도해봐야겠어요👍👍

Comment on lines +19 to +25
const MENU_LIST = [
{ title: 'Home', icon: HomeIcon, href: '/home' },
{ title: 'Search', icon: SearchIcon, href: '/search' },
{ title: 'Coming Soon', icon: ComingSoonIcon, href: '/soon' },
{ title: 'Downloads', icon: DownloadsIcon, href: '/download' },
{ title: 'More', icon: MoreIcon, href: '/more' },
];

Choose a reason for hiding this comment

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

배포 링크에서 soon, download, more 쪽은 404 에러가 나고 있어서 페이지를 만들어주시면 좋을 것 같아요!

export default function DetailScreen({
data,
}: {
data: { title: string; poster_path: string; overview: string };

Choose a reason for hiding this comment

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

저도 궁금합니다! 자주 쓰일 수 있는 인터페이스라면 파일에 따로 정의해두고 꺼내써도 좋을 것 같아요~

Comment on lines +29 to +35
useDebounce(
() => {
router.replace(`/search?query=${query}`);
},
[query],
1000,
);

Choose a reason for hiding this comment

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

debounce 간단하게 적용하신 점 좋네요!!
검색페이지 처음 들어가면 경로가 query= 인 상태로 끝나서 query가 빈 값일 때는 popular 데이터를 부르는 식으로 api 호출의 경우를 나눠 처리해주셔도 좋을 것 같습니다~!

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.

4 participants