TodoList-Level1_CodeReview

[richeeee128]

안녕하세욥~~!

[richeeee128]

• 85번째줄 부터 102번째줄, 111번째줄부터 128번째줄까지 중복되는 것 같으니 컴포넌트 분리 통해 재사용성을 높이는 방안은 어떠신가요?

• 25번째줄에서 Id = todos.length +1 , todos.length를 활용할 때 심각한 오류가 발생할 수 있다고 합니다. 문제 사항을 검토해보시면 어떨까요?

• 86,87번째 줄 Span 태그와 br 태그를 사용하여 줄바꿈이 일어난 것을 보았습니다. 다음엔 h1 같은 제목 태그나 p 태그를 사용하여 줄바꿈이 포함되어 있는 태그를 활용해 보시는 건 어떠세요?

• 기능별로 단락을 나누어 이해하기 쉽도록 업로드 되어 보기 편했습니다. 주석도 있어서 무슨 기능을 하고 있는지 확실히 보였습니다.

• 사용자가 빈칸을 넣으면 만들어지지 않도록 한 것도 배울 점이라고 생각했습니다.

• 전체적으로 화살표 함수로 통일한 점도 대단하다고 생각합니다.