[김참솔] Sprint 6 #97
[김참솔] Sprint 6 #97
Conversation
|
스프리트 미션 하시느라 수고 많으셨어요. |
|
cd8d67e commit의 수정사항과 관련해서 질문이 있습니다. 오호 그렇군요. |
There was a problem hiding this comment.
굿굿 개발 환경과 운영 환경을 따로 분리하셨군요 ! 😉👍
| } | ||
| `; | ||
|
|
||
| function Item({ imageUrl, title, price, likeCount }) { |
There was a problem hiding this comment.
(제안) Item보다는 ProductCard와 같이 좀 더 명확한 이름을 쓰는건 어떨까요?
Item이라고 쓰여있으니 어떤 UI인지 헷갈릴 것 같아서 조심스레 제안드려봅니다 ~! 😊
There was a problem hiding this comment.
스프린트 미션에서 /additem 이라는 path를 사용하는걸 보고 일관성을 고려해서 Item 이라는 이름을 프로젝트 전체에서 사용했는데요. 혹시 Product 라는 이름을 사용했을 때 path와 다른 용어를 사용해서 생기는 단점은 없을까요?
| const handleTagKeyUp = (event) => { | ||
| if (event.key !== "Enter") return; | ||
|
|
||
| const tagName = event.target.value.trim(); | ||
| if (tagName.length === 0) return; | ||
|
|
||
| if (tags.includes(tagName)) return; | ||
| setTags([...tags, tagName]); | ||
|
|
||
| setInputValues({ | ||
| ...inputValues, | ||
| tag: "", | ||
| }); | ||
| }; |
There was a problem hiding this comment.
크으.. 코드가 정말 가독성 좋게 잘 짜여져있네요.
"만약, 엔터가 아닐 경우 함수를 종료한다.", "만약, trim 후 빈 값이면 함수를 종료한다", "만약, 기존 태그와 중복이라면 함수를 종료한다"...
자칫 if 블록 지옥이 될 수 있었는데 조기에 종료함으로써 가독성을 참 좋게 작성하셨네요 👍👍
|
수고하셨습니다 참솔님 ! 참솔님은 앞으로도 쭉 성장하시길거라 생각합니다 ! 👍👍 |
2 participants
요구사항
기본
심화
주요 변경사항
/additem주소의 상품 등록하기 page를 구현합니다.styled-components를 사용해서 다시 구현했습니다.스크린샷
멘토에게
AddingItemImageInput)과 이미지 preview(AddingItemImage)가 화면 가능한 영역의 50%를 차지하도록 구현했습니다.AddingItemImageInput과AddingItemImage의 너비는 동일하게 맞춰지는데AddingItemImageInput의 높이가 약간 커지는 문제가 있습니다. (왼쪽 회색 box의 높이가 약간 큽니다.)