ElleryVernon / todo2

https://todo2-three.vercel.app
0 stars 0 forks source link

LV2. 리뷰페어 - 항해99 16기 R반 김용빈 #2

Open rladydqls99 opened 1 year ago

rladydqls99 commented 1 year ago
  1. 폴더명 컴포넌트들이 잘 분리되어 있다고 생각합니다. 역할별로 잘 나누셨을거라고 생각이 드는데, Demo나 Hero라는 폴더들이 어떠한 의미를 가지고 있는지 궁금합니다.

  2. initialState initialState 값이 상태는 배열로 되어 있습니다. 물론 이번 과제에서는 많은 양의 데이터를 다루지 않을 거라고 판단하셔서 배열로 initialState를 나타내셨을 수 있지만, 실제로 담아야 할 todo의 수가 많아질 경우 id를 키로 갖는 객체 형태로 상태를 관리하는 것이 더 효율적인 방법이 될거라고 생각합니다.

  3. 상세보기 페이지 상세보기 페이지는 css가 적용이 안되어 있는데 일부러 안하신건지 궁금합니다.

ElleryVernon commented 1 year ago
  1. Hero는 일반적으로 Landing이라고도 하며, 서비스의 설명이 있는 메인 페이지나, 메인페이지에 있는 서비스 설명 부분을 말합니다.
  2. initialState 는 객체 리스트지만, 각각 uuid의 단점을 개선한 cuid 유형의 고유한 값이 들어있으며, 새로운 객체요소가 추가 될 때 역시 추가하는 부분을 보면 각각 cuid2 라이브러리를 통해 고유한 id 값을 가지게 되어 추가 되고 있습니다.
  3. css를 적용하려다. css가 없어도 요구사항에는 충분히 만족하는거 같아서, 추가로 작성하지 않았습니다.

용빈님 자세히 봐주셔서 감사합니다!

seokjin909 commented 1 year ago

감사합니다!