전남대 FE 이은진 3주차 과제 Step2 #43
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tlsehdfl
reviewed
Jul 12, 2024
| // GoodsMockData를 21번 반복 생성 | ||
| useEffect(() => { | ||
| const fetchGoods = async () => { | ||
| setErrorMessage(null); |
There was a problem hiding this comment.
이미 null 인 상태라 null 로 초기화 할 필요 없을거 같습니다.
Suggested change
| setErrorMessage(null); |
There was a problem hiding this comment.
시간 내주셔서 리뷰해주심에 항상 감사드립니다 !
실시간 급상승 랭킹에서 남성이 위시로 받은 페이지에서, 아이템을 가져올 때 오류가 발생하여
catch (err) {
setErrorMessage('데이터를 불러오는데 실패하였습니다.');
} 코드가 실행됩니다.
이후 다른 페이지를 누르면 불러 올 상품 데이터가 있더라도, ErrorMessage에 '데이터를 불러오는데 실패하였습니다.' 이 문자열이 들어있어 제대로 상품들이 출력 되지 않아 코드에 setErrorMessage(null); 이 문장을 넣어 해결하였습니다!
위 방식으로 문제를 해결하였는데, 혹시 이 방법이 setErrorMessage(null); 이 코드를 없애고도 위 문제를 해결 할 수 있거나, 더 좋은 방식이 있다면 답변 부탁드립니다
| import { getCurrentTheme, ThemeHeroSection } from '@/components/features/Theme/ThemeHeroSection'; | ||
| import { RouterPath } from '@/routes/path'; | ||
| import { ThemeMockList } from '@/types/mock'; | ||
| import {ThemeHeroSection } from '@/components/features/Theme/ThemeHeroSection'; |
There was a problem hiding this comment.
Suggested change
| import {ThemeHeroSection } from '@/components/features/Theme/ThemeHeroSection'; | |
| import { ThemeHeroSection } from '@/components/features/Theme/ThemeHeroSection'; |
| setGoods(data); | ||
| setisLoading(false); | ||
| } catch (err) { | ||
| setisLoading(true); |
There was a problem hiding this comment.
불필요해 보입니다.
Suggested change
| setisLoading(true); |
There was a problem hiding this comment.
finally 절에서 로딩 상태를 false 로 두면 될 거 같아요.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 멘토님 !
3주차 과제를 진행하면서 오류를 찾는데 시간이 지체되어 제출하는데 시간이 오래 걸렸습니다 ..🥺
코드를 중간중간 많이 고쳐 commit을 작은 단위로 나누어 하지 못한 점 알려드립니다.
불필요한 사항이나, 고쳐야 할 부분이 있으면 말씀 부탁드립니다!
항상 시간 내어 리뷰해 주셔서 감사드립니다! 이번 주도 잘 부탁드립니다 ㅎㅎ