주특기 숙련주차 과제 피드백 - 작성자: 류제천 튜터

[rjc1704]

Overview

• 에러로 인해 프로그램이 동작하지 않는 상태라 코드 내용만 보고 피드백 드리겠습니다.
• useState로 관리하던 상태를 리덕스로 리팩터링해보려고 노력한 흔적이 많이 보입니다. 최대한 노력하려고 하신 점 좋습니다.
• 리덕스 사용에 대해서 아직 많이 어려움을 겪고 계시는 것으로 보입니다. 뉴스피드 발제 때 공유된 리덕스 특강 영상을 꼭 다시 한번 확인해 보셨으면 좋겠습니다. https://teamsparta.notion.site/458bb1aa25e64272903fe0f3d75d9a0f
• 큰 작업을 한꺼번에 한 다음 commit하기 보다는 작은 작업을 한 가지 할 때 마다 git commit을 하는 습관을 들여주시면 좋습니다.

코드 레벨 피드백은 아래에 드리겠습니다.

[rjc1704]

https://github.com/jjin2e/react-todolist/blob/5eeeb88757eccedf53e561f6ed11595d6e146c86/src/App.jsx#L37-L42 title과 contents를 참조하겠다고 했는데, title, contents가 정의된 코드가 없습니다.

title과 contents는 input 태그에서 사용되는 상태로 굳이 전역상태로 관리할 필요가 없어서 리덕스 없이 useState로 const [tittle, setTitle] = useState("");를 그대로 사용하셔도 무방합니다.

[rjc1704]

https://github.com/jjin2e/react-todolist/blob/5eeeb88757eccedf53e561f6ed11595d6e146c86/src/redux/config/configStore.js#L2-L3 redux는 동일한 출처의 라이브러리이기 때문에 두줄로 나눠서 쓸 필요가 없고, import { createStore, combineReducers } from "redux"; 이렇게 한줄로 쓰시는 것을 더 권장 드립니다.

[rjc1704]

https://github.com/jjin2e/react-todolist/blob/5eeeb88757eccedf53e561f6ed11595d6e146c86/src/App.jsx#L63-L64 redux를 쓰셨다면 ui를 그리는 state인 todos와 state를 변경하는 setTodos를 props로 넘겨줄 필요가 없습니다. 직접 TodoList 컴포넌트 내에서 useSelector로 todos를 구독하고, App.jsx 내에서 todo 추가 시, TodoList 컴포넌트에서 상태변화를 감지하고 리렌더링됩니다.