ParkBig
commented
1 year ago 줄 바꿈, 깜빡했네요.. 리뷰 감사합니다!
2022년 10월 4일 (화) 오후 11:02, 이대호_ICT @.***>님이 작성:
-
App.js에서 라우팅 로직 잘 작성하신것 같습니다. 관련해서 하나 조언을 드리자면, 각 페이지를 담당하는 메인컴포넌트를 따로 관리하신건 잘하셨는데, 폴더명을 pages로 지정하면 더 이해하기 편할것 같습니다. (개발자들사이에서의 컨벤션이라고 보시면 됩니다.)
redux 관련 로직에서는 todoArr에서 reducer를 필요에 맞게 잘 사용하신 것 같습니다. 다만 title.js이나 content.js 모듈은 따로 사용하지 않고 useState로 관리하시는게 더 적절할 것 같습니다.
stlyed component로 스타일을 지정할때나 함수를안에 로직을 작성할 때 적절하게 줄바꿈을하시면서 작성하시는게 코드 가독성 향상에 도움이 될 것 같습니다.
수고하셨습니다 :)
— Reply to this email directly, view it on GitHub https://github.com/ParkBig/todoList_with_Redux/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZFRNS7OUN5SGTIRPCR635DWBQ2GRANCNFSM6AAAAAAQ4TN54U . You are receiving this because you are subscribed to this thread.Message ID: @.***>
App.js에서 라우팅 로직 잘 작성하신것 같습니다. 관련해서 하나 조언을 드리자면, 각 페이지를 담당하는 메인컴포넌트를 따로 관리하신건 잘하셨는데, 폴더명을 pages로 지정하면 더 이해하기 편할것 같습니다. (개발자들사이에서의 컨벤션이라고 보시면 됩니다.)
redux 관련 로직에서는 todoArr에서 reducer를 필요에 맞게 잘 사용하신 것 같습니다. 다만 title.js이나 content.js 모듈은 따로 사용하지 않고 useState로 관리하시는게 더 적절할 것 같습니다.
stlyed component로 스타일을 지정할때나 함수를안에 로직을 작성할 때 적절하게 줄바꿈을하시면서 작성하시는게 코드 가독성 향상에 도움이 될 것 같습니다.
수고하셨습니다 :)