김하영님 코드리뷰
Opened this issue · 0 comments
하영님 너무 고생하셨습니다!
Line 12 in 96ac59d
주석지우기!
Line 8 in 96ac59d
useContext 로 AuthContext 를 불러서 사용하고 있는데 context 를 사용하실때 커스텀훅으로 묶어 사용하면 조금 더 간결하게 사용할 수 있어용.
export const useAuth = ()=>{
return useContext(AuthContext);
}
Line 19 in 96ac59d
MOCK_DATA를 컴포넌트 내부에 선언하면 렌더링 시마다 재생성됩니다. 그래서 컴포넌트 외부에서 선언해서 사용하는편이 더 좋아보여요~
Line 10 in 96ac59d
handleClick 함수에서 로컬 스토리지에서 직접 토큰을 확인하기보다는, 이미 AuthContext에서 제공하는 isAuthenticated를 활용하는 것이 코드의 일관성을 높일 수 있습니다.
mbti-test/src/context/AuthContext.jsx
Line 9 in 96ac59d
isAuthenticated 가 상태일 필요가 없어보여요!
const isAuthenticated = !!token // 혹은
const isAuthenticated = !!user
👍
하영님 코드가 정말 깔끔해요. 적절하게 정리되어있고 로직과 이름도 읽기 쉽게 잘 구성되어있어요! Query 도 사용해보시면 좋을것 같네요 :)
어렵다고 말씀해주신 부분들도 깔끔하게 잘 구현해주셨는데요?
+) 테일윈드는 지금 사용해주신것처럼 classname 에다가 다 넣어주시는 방법이 가장 직관적이고 효율적인 방법이 맞습니다 :) 모듈화 X