tnrnr910
commented
1 year ago 피드백 감사합니다!
2023년 6월 29일 (목) 12:28, LeeJaeSang @.***>님이 작성:
고생하셨습니다. localStorage 사용으로 Persist 유지할 수 있도록 하신점 좋았습니다.
react 는 .js 보다는 .jsx 확장자를 사용합니다. 리액트 컴포넌트가 .js 로 되어있지 않도록 관리하는게 좋습니다.
이 부분은 하나의 컴포넌트로 분리해서 사용하시는게 더 좋아보입니다. ex) TodoPage
리액트가 어려우셨군요 ㅋㅎㅋㅎㅋㅎ... 동감합니다..
이 부분 코드는 initial 값 자체를 랜더링 이후에 만들어 주고 있습니다. 랜더링 이전의 상태(redux)를 통해 initalState 를 만들어주고 그 초기 상태를 랜더랑 할 수 있도록 변경해주는 것이 좋아보입니다. useEffect 는 말 그대로 sideEffect 를 발생시키기때문에 조금 더 조심스러운 사용이 권장됩니다.
https://github.com/tnrnr910/todoList/blob/14ba16aa82fde83534994beb82edddbff44f6381/src/components/TodoForm/TodoForm.js#L32-L33 이 부분은 clearForm 정도의 함수로 분리해주면 가독성이 더 높아질 수 있습니다.
이 부분은 위에서 filter 를 통해서 조건 분기 이후 랜더링 하는 것이 좋아보입니다.
// 비지니스 로직 const filteredData = data.filter(...);
// UI filteredData.map(...)
actions, reducers, store 는 redux 라는 폴더를 만들어 그 안에 넣었어도 좋을 것 같습니다!
고생하셨습니다!
— Reply to this email directly, view it on GitHub https://github.com/tnrnr910/todoList/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7JK5CVJYIKFX2QBOA6SMKDXNTY6RANCNFSM6AAAAAAZX56RBQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>
고생하셨습니다. localStorage 사용으로 Persist 유지할 수 있도록 하신점 좋았습니다.
react 는 .js 보다는 .jsx 확장자를 사용합니다. 리액트 컴포넌트가 .js 로 되어있지 않도록 관리하는게 좋습니다.
https://github.com/tnrnr910/todoList/blob/14ba16aa82fde83534994beb82edddbff44f6381/src/App.jsx#L20-L34
이 부분은 하나의 컴포넌트로 분리해서 사용하시는게 더 좋아보입니다. ex)
TodoPagehttps://github.com/tnrnr910/todoList/blob/14ba16aa82fde83534994beb82edddbff44f6381/src/components/TodoList/TodoList.js#L32
리액트가 어려우셨군요 ㅋㅎㅋㅎㅋㅎ... 동감합니다..
https://github.com/tnrnr910/todoList/blob/14ba16aa82fde83534994beb82edddbff44f6381/src/components/TodoList/TodoList.js#L23-L41
이 부분 코드는 initial 값 자체를 랜더링 이후에 만들어 주고 있습니다. 랜더링 이전의 상태(redux)를 통해 initalState 를 만들어주고 그 초기 상태를 랜더랑 할 수 있도록 변경해주는 것이 좋아보입니다. useEffect 는 말 그대로 sideEffect 를 발생시키기때문에 조금 더 조심스러운 사용이 권장됩니다.
https://github.com/tnrnr910/todoList/blob/14ba16aa82fde83534994beb82edddbff44f6381/src/components/TodoForm/TodoForm.js#L32-L33 이 부분은 clearForm 정도의 함수로 분리해주면 가독성이 더 높아질 수 있습니다.
https://github.com/tnrnr910/todoList/blob/14ba16aa82fde83534994beb82edddbff44f6381/src/components/TodoList/TodoList.js#L54
이 부분은 위에서 filter 를 통해서 조건 분기 이후 랜더링 하는 것이 좋아보입니다.
actions, reducers, store 는 redux 라는 폴더를 만들어 그 안에 넣었어도 좋을 것 같습니다!
고생하셨습니다!