[강수민] Sprint12 #344
[강수민] Sprint12 #344
Conversation
| const imgUrl = files[0]; | ||
| const imgPath = URL.createObjectURL(imgUrl); | ||
| setImg(imgPath); | ||
| setImg(imgUrl); |
There was a problem hiding this comment.
imgUrl을 사용하는 곳이 한 군데 뿐이라면 따로 변수를 만들지 않고 setImg(files[0]) 이런식으로 작성하셔도 됩니다.
| localStorage.removeItem("accessToken"); | ||
| localStorage.removeItem("refreshToken"); | ||
| localStorage.removeItem("userId"); |
There was a problem hiding this comment.
localStorage의 키 값들도 상수로 관리해주셔도 좋습니다.
| images: u_images, | ||
| name: u_name, | ||
| description: u_description, | ||
| price: u_price, | ||
| tags: u_tags, |
There was a problem hiding this comment.
여기도 카멜 케이스 그대로 활용해주셔도 괜찮습니다. uImages, uName 이런식으로요!
| </div> | ||
| </div> | ||
| </div> | ||
| <Modal isOpen={isOpen} closeModal={handleCloseModal}> |
There was a problem hiding this comment.
모달 컴포넌트 내부의 코드가 은근히 길어져서 따로 컴포넌트로 만드셔서 사용해주셔도 좋습니다.
|
|
||
| interface DropDownInquiryProps { | ||
| setIsEditting: (value: boolean) => void; | ||
| onDelete: () => void; |
There was a problem hiding this comment.
setState 하는 함수 대신 handler 함수를 받아오게 하신 점 좋네요~ 더 유연하게 사용할 수 있을 것 같습니다.
| const { data } = useQuery({ | ||
| queryKey: ["products", currentPage, productsPerPage, order, keyword], | ||
| queryFn: () => | ||
| fetchProducts({ |
There was a problem hiding this comment.
fetchProducts 함수를 사용하는 useQuery가 또 있는 것 같은데, 두 경우 모두 사용할 수 있는 훅을 만들어보는 것은 어떨까요?
| // 추후 POST API 연동 예정 | ||
| // console.log(e); | ||
| postProduct({ content: userInput }); | ||
| const { id } = await postProduct({ content: userInput }); |
There was a problem hiding this comment.
post 하는 함수는 useMutation 함수를 활용해보시면 어떨까요?
| onUpdate={() => {}} | ||
| onDelete={() => {}} |
There was a problem hiding this comment.
아직 handler 함수가 개발되지 않은 것이라면 괜찮지만, 그게 아니라 만약 빈 함수를 넣어야 하는 상황이라면 해당 컴포넌트의 핸들러 함수들을 nullable하게 타입 지정해주시는 것도 좋을 것 같습니다.
2 participants
요구사항
기본
중고마켓
상품 상세
상품 등록
심화
주요 변경사항
스크린샷
판다마켓
멘토에게