인프런 커뮤니티 질문&답변

wxd8339님의 프로필 이미지
wxd8339

작성한 질문수

따라하며 배우는 노드, 리액트 시리즈 - 영화 사이트 만들기

무비앱 시리즈 #6 Load More Button 만들기 ( 따라하며 배우는 리액트 노드 )

maingImage

해결된 질문

작성

·

245

2

loadmore 누르면 메인 이미지도 바뀌게 되네요
 
이미지를 page1번상태로 고정하고싶은데 어떻게 하면 될까요?

답변 4

1

안녕하세요.

강사님께서 적으신 코드를 보면 loadMoreItems와 useEffect에서 각각 

setMainMovieImage(...[response.results[0]])

로 main이미지를 바꿔주고 있습니다.

따라서 해당 부분을 맨 처음 한번만(page1을 불러올 때) 포함하면 됩니다. 매번 반복하지 않구요.

 

useEffect(() => {
        const endpoint = `${API_URL}movie/popular?api_key=${API_KEY}&language=en-US&page=1`;
        fetch(endpoint)
            .then(response => response.json())
            .then(response => {
                console.log(response)

                setMovies([...Movies, ...response.results])
                setMainMovieImage(...[response.results[0]])
                setCurrentPage(response.page)
            })
    }, [])

    const fetchMovies = (endpoint) => {

        fetch(endpoint)
            .then(response => response.json())
            .then(response => {
                console.log(response)

                setMovies([...Movies, ...response.results])
                //setMainMovieImage(...[response.results[0]])
                setCurrentPage(response.page)
            })
    }

 

위의 코드를 따라 적으시면 원하시는 구현대로 동작하나 코드가 반복되어 우아하지 못한 코드라 느껴집니다.

 

그래도 코드를 줄이려는 노력은 해봤습니다.

아직은 리액트가 어렵게 느껴지네요.

 

useEffect(() => {
        const endpoint = `${API_URL}movie/popular?api_key=${API_KEY}&language=en-US&page=1`;
        fetch(endpoint)
            .then(response => response.json())
            .then(response => {
                setMainMovieImage(...[response.results[0]])
            })
        fetchMovies(endpoint)
    }, [])

    const fetchMovies = (endpoint) => {

        fetch(endpoint)
            .then(response => response.json())
            .then(response => {
                console.log(response)

                setMovies([...Movies, ...response.results])
                //setMainMovieImage(...[response.results[0]])
                setCurrentPage(response.page)
            })
    }

 

작성자님께서 코드를 우아하게 줄일 수 있는 방법을 알게 되신다면 공유해주세요 ^^ 

아니면 언젠가 여기에 John Ahn님의 가호가 있길...

wxd8339님의 프로필 이미지
wxd8339
질문자

ㅠㅠ 친절하게 설명해주셔서 너무 감사드립니다!!

중복되는 코드가 신경쓰인다면 if 문으로 page == 1일때만 setMainMovieImage를 호출하게 하셔도 될 듯 합니다.

0

fetchMovie를 처음 로드 할 때, current page가 0 (use State)거나 endpoint에서 page가 1이니까,

setMainMovieImage를 if문으로 호출하게 하셔도 될 것 같습니다.

const fetchMovies = (endpoint) => {
fetch(endpoint)
.then(response => response.json())
.then(response => {
console.log(response);
setMovies([...Movies, ...response.results])
if(CurrentPage === 0) {
setMainMovieImage(response.results[0])
}
setCurrentPage(response.page)
})
}

0

코드는 좀 더럽지만ㅠ 저는 이런식으로 url값을 1번 이미지의 url값으로 해줬습니다.

wxd8339님의 프로필 이미지
wxd8339
질문자

좋은방법 공유 감사드려요!!

위 코드로 쓰면 제일 인기있는 영화가 바뀔때 수동으로 바꿔줘야하겠죠?

0

<MainImage
          mainImage={mainImage}
          image={`${IMAGE_BASE_URL}w1280/iQFcwSGbZXMkeyKrxbPnwnRo5fl.jpg`}
        />
wxd8339님의 프로필 이미지
wxd8339

작성한 질문수

질문하기