[문지영] sprint6 #204
[문지영] sprint6 #204
Conversation
addiescode-sj
left a comment
addiescode-sj
left a comment
There was a problem hiding this comment.
수고하셨습니다!
주요 리뷰 포인트
- 스타일드 컴포넌트 구별 방식 관련 피드백
- 이미지 업로드 관련 문제 해결
| const preview = URL.createObjectURL(file); | ||
| setPreviewUrl(preview); | ||
|
|
||
| if (previewUrl) { |
There was a problem hiding this comment.
본문에서 질문주신 문제에 대해 답변드리자면,
이미지를 삭제하고 다시 같은 이미지를 업로드할때 함수 내부의 previewUrl은 이전값(같은 값)을 참조하고 있는 상황이기때문에, 해당 조건이 true가 되어 그렇습니다. 따라서 이 상태를 직접 참조하는대신, 파일의 값이 있는지만(비어있는지 아닌지만) 확인하는식으로 조건을 수정하면 될것같네요!
| <StNavLink | ||
| to="/items" | ||
| $isActive={isMarketActive} | ||
| > | ||
| 중고마켓 | ||
| </StNavLink> |
There was a problem hiding this comment.
저번 PR에서 조언드렸던 코멘트인것같은데, 스타일드 컴포넌트를 구분짓기위한 의도라면 완전한 의미를 가진 StyledNavLink로 네이밍하시는게 좋을것같습니다.
혹은, NavLink로 네이밍하고 외부 파일로 분리해 불러올때
import * as S from '...' 이렇게 불러와서 구분하는 방식도 괜찮습니다.
| const [price, handlePriceChange] = useFormatNumber(''); | ||
| const [tags, setTags] = useState([]); | ||
|
|
||
| const isDisabled = name.trim() && description.trim() && price.trim() && tags.length > 0; |
There was a problem hiding this comment.
혹시 조건이 반대로 되어있는것 아닌가요?
|
|
||
| <FormSection> | ||
| <ImageUpload /> | ||
|
|
| const [name, setName] = useState(''); | ||
| const [description, setDescription] = useState(''); | ||
| const [price, handlePriceChange] = useFormatNumber(''); | ||
| const [tags, setTags] = useState([]); |
There was a problem hiding this comment.
굳굳! 각 필드를 개별적으로 업데이트할수있게 나누셨군요 👍
질문에 대한 답변
본문에서 자세히 답변드렸습니다!
커스텀 훅은 보통 이럴 경우에 쓰입니다:
정리하자면 커스텀 훅은 '재사용'과 '로직 분리'가 포인트로,
|
2 participants
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게