issues
search
richeeee128
/
react_todolist
https://react-todolist-five-xi.vercel.app
0
stars
0
forks
source link
react_todolist_codereview
#1
Open
myeongbin0918
opened
1 year ago
myeongbin0918
commented
1 year ago
안녕하세요~!
myeongbin0918
commented
1 year ago
예쁜 말, 고운 말 쓰기로 해용~💚
myeongbin0918
commented
1 year ago
todo의 initial(초기) state를 {id: 0, title: “”, body: “”}로 구성하셨는데 todo 진행-완료 상태를 표시하는 프로퍼티를(ex: isDone: false) 넣지 않으신 이유가 궁금합니다.
20번째 줄의 clicked라는 이름은 '추가하기' 의도를 나타내기에는 충분하지 않다고 생각합니다. clicked 앞에 add를 붙여 의도를 나타내 보시는 건 어떨까요? + toggleDone도 완료 상태뿐만 아니라 취소 상태도 같이 나타내는 이름이면 좋지 않을까요?
22번째 줄에서 추가되는 todo의 id값을 Date.now()로 주어 기존에 있는 id의 값과 겹치지 않게 한 세심함에 감탄했습니다.
html 부분에서 반복되는 컴포넌트(button.jsx)를 잘 분리해 코드의 줄 수가 줄고 반복되는 부분이 없어져 보기가 편했고 원하는 코드를 찾기도 수월했습니다.
23, 24번째 줄에서 생략할 수 있는 부분은 생략한 게 노련한 개발자 같았습니다.
CSS 꾸미기에 100줄을 넘기지 않고 작업하신 게 대단하다고 생각합니다.
안녕하세요~!