Lv.2 과제 피드백 - 작성자: 류제천 튜터

[rjc1704]

Overview

• 리덕스 사용을 아주 잘하고 계십니다.
• 커밋메시지에 대한 rule에 익숙해지시면 좋습니다. 일반 기능은 feat: 로 시작, css스타일작업 style: 로 시작, 리드미 작성이나 주석 관련 등 기능과 상관없는 경우는 chore:, 버그 수정 시 fix: 또는 bug: 로 작업해 주시면 커밋 메시지가 많아졌을 때 가독성이 좋아집니다.
• Main 컴포넌트와 Sub 컴포넌트는 하나의 페이지를 의미하는 페이지 컴포넌트이기 때문에 src 폴더 아래에 pages/ 라고 폴더를 만들고 별도로 정리해 주시는 것이 좋습니다.
• 컴포넌트의 파일명은 항상 PascalCase 로 작성되는 것이 좋습니다. main.jsx -> Main.jsx, sub.jsx -> Sub.jsx
• 다음 개인과제 때는 연습차원에서 feat -> dev -> main 순으로 PR 연습도 해보시면 좋겠습니다.
• 리드미에는 이 프로젝트의 목적이 무엇인지 사용한 기술스택에는 어떤 것들이 있는지 간단하게 기술해주시면 좋습니다.
• components/Style.jsx 보다는 별도의 style 폴더를 만들어서 src/style/Style.js 로 해주시는 게 좋습니다. jsx라는 확장자는 jsx 문법이 있는 경우에만 붙여주시는 게 좋습니다. (추후에 배울 typescript에서는 이 룰을 지키지 않으면 에러가 납니다.)

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

[rjc1704]

https://github.com/CodeNoob4089/React_Todolist/blob/eaeb4fef19c870be66975d9ccd5abf562134f971/src/redux/modules/todo.js#L3-L5 redux에서 액션 타입은 일종의 id로써의 역할을 하는 상수이며, 상수를 변수로 처리할 때는 대문자를 기반으로한 SNAKE_CASE를 사용합니다. const ADD_TODO = "todo/ADD_TODO"; todo/ 처럼 앞에 접두사를 붙여주는 습관을 들여주시는 게 좋습니다. dispatch(액션)이 실행됐을 때 store에 등록된 모든 리듀서들이 실행되고 이 때 action.type에 중복이 있으면 의도치 않은 상태변경이 일어날 수 있습니다. 지금은 todo만 다루기 때문에 문제없지만 확장성과 유지보수성을 생각하는 코딩습관이 경쟁력이 됩니다.

[rjc1704]

https://github.com/CodeNoob4089/React_Todolist/blob/eaeb4fef19c870be66975d9ccd5abf562134f971/src/components/Todo.jsx#L15-L15 변수명은 semantic 해야 합니다. data 보다는 todos로 해주시면 좋겠습니다.