Conversation
…ithub-actions [Fix] delete merged branch github action
src/api/itemApi.js
Outdated
There was a problem hiding this comment.
해당 함수가 사용되는 곳을 확인해보니까 목적으로 사용되고 있더라구요. 아마도 템플릿코드가 이렇게 되어있었나요?
함수는 웬만하면 하나의 목적을 가져야 합니다. 그러나 현재에는 product와 comment를 불러오는거에 둘다 쓰이고 있어요.
if문이 두개인걸 봐도 알 수 있습니다. ㅎㅎ 그것보다 두 함수로 분리해서 getProductById , getCommentById와 같이 따로 관리하는게 더 낫다고 생각합니다.
| if (!item) { | ||
| return <Container>상품을 불러오는 중입니다.</Container>; | ||
| } |
There was a problem hiding this comment.
아이템이 없을때에는 "불러오는 중"이라기보다 아이템이 없다는 메세지가 더 적절 할 것 같아요 !
There was a problem hiding this comment.
Styled-Components들이 한 파일이 점점 많아져서 파일이 길어지면, Styled-Components 부분만 따로 떼어내서 다른 파일에 관리하는게 더 좋을 것 같아요. 파일의 길이가 길어지면 가독성이 좀 떨어지기 시작합니다 ㅎㅎ
| function CommentsSection() { | ||
| const { productid } = useParams(); | ||
| const [comments, setComments] = useState([]); | ||
| const [inputValue, setInputValue] = useState(""); |
There was a problem hiding this comment.
특히 프론트엔드에서 변수명을 정할때 개발자적인 관점보다는 비즈니스로직(기능적)관점에서 정하는게 좀 더 나은것 같더라구요. 그래서 여기는 inputValue보다는 comment가 더 어울릴 것 같습니다.
| <TextInput | ||
| placeholder="개인정보를 공유 및 요청하거나, 명예 훼손, 무단 광고, 불법 정보 유포시 모니터링 후 삭제될 수 있으며, 이에 대한 민형사상 책임은 게시자에게 있습니다." | ||
| type="text" | ||
| onChange={handleInputChange} | ||
| /> |
There was a problem hiding this comment.
TextArea로 선언해주신 점 잘 하셨습니다 !
| /> | ||
| <NicknameContainer> | ||
| <Nickname>{comment.writer.nickname}</Nickname> | ||
| <Time>{TimeCheck(comment.createdAt)}</Time> |
There was a problem hiding this comment.
리액트 리렌더링과 관련없는 유틸성 함수는 따로 빼서 관리하신 점 잘 하셨습니다 !!
There was a problem hiding this comment.
다만 함수인 만큼 동사로 시작하는 네이밍이 좋을 것 같아요. ex checkTime / calculateCommentTime
체크리스트 [기본]
상품 상세
상품 상세 페이지 주소는 “/items/{productId}” 입니다.
response 로 받은 아래의 데이터로 화면을 구현합니다.
=> favoriteCount : 하트 개수
=> images : 상품 이미지
=> tags : 상품태그
=> name : 상품 이름
=> description : 상품 설명
상품 문의 댓글
문의하기에 내용을 입력하면 등록 버튼의 색상은 “3692FF”로 변합니다.
response 로 받은 아래의 데이터로 화면을 구현합니다
=> image : 작성자 이미지
=> nickname : 작성자 닉네임
=> content : 작성자가 남긴 문구
=> description : 상품 설명
=> updatedAt : 문의글 마지막 업데이트 시간
스크린샷
멘토에게