baram55 / group-artist-fan-letter-box-2 Goto Github PK

명환님 redux thunk 가 어려우셨나봅니다 사용을 안해주셨네욤.

사실 이 부분 전체가 redux + thunk 로 들어갔었어야 해요.
리덕스에서 비동기를 관리하는 부분을 다시 한 번 봐볼필요가 있을듯 싶습니다.

여기서 구조분해할당으로 {data} 로 받아오는게 더 좋아보여요.

이 부분은 util 함수로 빼내어서 관리해줘도 좋을 것 같아요.

axios 를 바로 가져다 사용하는 것 보다 axios 인스턴스를 만들어서 핸들링하는게 더 좋습니다.

axios create 를 이용해서 특정 인스턴스를 생성해서 해보세요!

폼을 이용하고 있을때에는 state 를 이용하는 것 보다는, 비제어 형식으로 사용해도 좋을 것 같아요. input 시 마다 불필요한 렌더링을 굳이 해줄필요는 없을 것 같습니다.

try catch 를 잡을때에 전체적인 로직을 다 묶어주어도괜찮습니다.
response 를 위에 선언해서 사용하지 않고 try catch 블록 안으로 넣어도 좋아보입니다.

이런 비지니스 로직등을 분리해서 묶어주는게 더 좋을 것 같아요. 분리도가 떨어지니 코드가 길어지고 가독성도 떨어집니다. 이제 함수 분리하는 연습을 해보시는게 좋아요여요

명환님 async 에 대한 이해도가 많이 떨어지시지는 않는것 같아요. thunk 에 대한 부분은 가볍게 짚고 넘어가고 react query 쪽을 확실히 잡아가보면 좋을 것 같아요!