Conversation
bk-git-hub
left a comment
bk-git-hub
left a comment
There was a problem hiding this comment.
유미님 과제 하시느라 고생 많았습니다!
리액트가 익숙하지 않은 상태에서 과제를 진행하시느라 쉽지 않으셨을 것 같은데, 전체적으로 잘 해주셨습니다
파일별로 코멘트를 남겨두었으니, 너무 부담 갖지 말고 차근차근 확인해보시면 좋겠습니다
다음 주 과제에서는 커밋도 가이드를 참고하면서 한 번 천천히 연습해보시면 좋을 것 같습니다.
https://www.notion.so/Git-Commit-Guide-327793efb80180bcb8f9c357c580a48d
| ); | ||
| } | ||
|
|
||
| export function TodoCard() { |
There was a problem hiding this comment.
TodoCard가 카드 1개를 담당하는 대신, 내부에서 todoItems 배열 전체를 돌면서 여러 카드를 렌더링하고 있습니다. 이름상으로는 TodoList가 목록을 돌고 TodoCard가 카드 1개만 담당하는 구조가 더 자연스러울 것 같아요
| return result; | ||
| }; | ||
|
|
||
| return <>{rendering()}</>; |
There was a problem hiding this comment.
rendering 함수는 동작은 잘 하지만, React에서는 보통 배열을 JSX로 바로 반환하는 방식이 더 읽기 쉽습니다. 다음주 읽을 거리에서 리액트에서 실제로 리스트를 어떻게 다루는지가 포함되어있으니 읽어보시고 수정하면 좋을 것 같아요
There was a problem hiding this comment.
TodoHeader, TodoList, TodoCard를 파일단위로 분리하면 가독성과 유지보수면에서 더 좋을 것 같아요
|
|
||
| export function TodoList() { | ||
| return ( | ||
| <div id="todoList"> |
There was a problem hiding this comment.
현재 리스트의 div 를 ul / li 로 바꾸면 조금 더 시맨틱한 마크업이 될 수 있어요
2 participants
✅ 제출 정보
✅ 체크리스트 (필수)
main이 아니라 내 GitHub 핸들 브랜치(<handle>)다<handle>-week-xx형식이다🧩 구현 내용 요약
TodoHeader(제목),TodoList(카드 영역),TodoCard(개별 카드)❓ 궁금한 점