Conversation
| const SignupPage = () => { | ||
| const navigate = useNavigate(); | ||
| const [step, setStep] = useState<Step>('email'); | ||
| const [email, setEmail] = useState(''); |
There was a problem hiding this comment.
email, password를 각각 state로 관리하고 있는것으로 보입니다.
{ email: string, password: string}으로 타입 지정된 하나의 state로 묶어서 관리하면 어떨까요?
[form, setForm] = useState<Form(임의 interface)>({ email: "", password: "" }) 로 지정하고
상태를 바꿀 땐
setForm(prev => ({...prev, [바꿀대상]: value}))으로 진행하면 좋을것 같아요
There was a problem hiding this comment.
말씀해주신 대로 email과 password를 각각 관리하는 것보다 SignupForm 인터페이스로 묶어서 하나의 state로 관리하는 게 더 좋은것 같네요
setForm(prev => ({ ...prev, [field]: value })) 패턴으로 수정했습니다!
qkrdmsthff
left a comment
qkrdmsthff
left a comment
There was a problem hiding this comment.
LGTM 동동 이번주차도 수고하셨습니다 ~~ 피드백 늦어져서 죄송해요 ㅠㅠ
|
|
||
| return () => { cancelled = true; }; | ||
| // eslint-disable-next-line react-hooks/exhaustive-deps | ||
| }, [depsKey]); |
There was a problem hiding this comment.
현재 useEffect 안에서 fetchFn 을 호출하나, fetchFn 은 의존성 배열 [depsKey] 에 포함되어 있지 않습니다 fetchFn 도 의존성 배열에 넣거나, 호출부에서 useCallback 으로 감싸서 넘기도록 해결해 볼 수 있겠네요!
5 participants
✅ 워크북 체크리스트
✅ 컨벤션 체크리스트
📌 주안점