Lv2 과제 페어 코드 리뷰입니다.

[makepin2r]

프로젝트 코드를 읽고 난 리뷰 전달 드립니다. :)

폴더/파일 구조

• 폴더를 생성하여 페이지 단위로 사용하는 컴포넌트를 분류하신 점이 좋았습니다. 해당 방법이 프로젝트 가독성을 높이는 좋은 방식이었다고 생각됩니다.
• 또한 컴포넌트 별로 index.js 와 styled.js 로 정리해주셨습니다. 컴포넌트에 연관된 코드들이 한곳에 모여있어 찾기 쉬우므로 유지 보수에 아주 효율적인 방법으로 보입니다.

변수명/함수명 정의

• 전반적으로 기능 및 UI 종류를 명시한 변수명, 함수명을 정의해주셔서 코드 이해가 수월했습니다.
• 다만 Header 컴포넌트의 경우 input 입력을 받는 기능의 요소인 것 같은데, Header라는 이름에서 레이아웃 요소라는 인상을 받았습니다. 기능이나 역할을 좀더 직관적으로 파악할 수 있는 이름을 채택하셔도 좋을 것 같습니다.
• 함수명의 경우 이벤트 핸들링용인지, 데이터 업데이트용인지 등 특정 기능에 따라 네이밍 규칙을 정하신 것으로 보입니다. 제가 구체적인 룰을 알지 못해 의도를 파악하지 못한 것일 수 있겠으나 onCompletionOnClick 과 같은 경우 'on'이 중복되지 않아도 될 것 같습니다.

Styled Components

• 위에 말씀드린 것처럼 컴포넌트 단위로 하나의 파일에 코드를 관리하신 점이 매우 좋았습니다.
• 다만 일부 스타일 속성의 경우 컴포넌트 파일에 아직 남아있는 경우가 있었는데 (ex. TodoList의 boxStyle, TodoContainer의 8~13/37~40번째 줄 등) 혹시 이유가 있었을까요? 만일 특별한 의도가 없으셨다면 이런 부분들도 styled 파일에 모두 정리해두어도 좋겠다는 생각이 듭니다.

데이터 관련

• 'todos' state에서 Date.now() 함수를 이용해 id의 고유성을 심플한 원리로 유지하신 부분이 인상 깊었습니다.
• 리덕스의 action creator 정의하실 때 payload 명을 들어올 데이터에 맞게 명시해주신 부분도 좋았습니다.
• 리덕스를 활용해 inputData도 전역적으로 관리하신 부분이 보였는데, 해당 데이터는 Header 컴포넌트에서 입력에 사용될 때 말고는 연관된 컴포넌트가 특별히 눈에 띄지 않았습니다. 그러므로 굳이 store로 관리하지 않아도 될 듯 합니다.

전반적으로 깔끔하고 가독성 높은 코드를 작성하시고자 노력하신 부분이 돋보였습니다. 고생하셨습니다. 감사합니다. 😊

[Rose-Rosie-Rose]

정말 제가 놓친 부분들을 너무나도 정확하게 짚어주셔서 너무 감사합니다!! 특히 고민을 많이 했던 이벤트 핸들러 명칭 정하는 부분을 제가 아닌 다른 분이 보았을 때, 어색하다는 것을 알게 되어서 그 부분에 대해서 조금 더 많은 고민을 할 수 있게 해주셔서 너무 감사합니다😊 그리고 말씀해 주신 Header 부분은 정확하게 어떻게 나누면 좋을지 정말 많이 찾아보아서 다른 프로젝트 때는 정확하게 나누어 보겠습니다!! 다시한번 코드리뷰해주셔서 너무 감사합니다😍