duddlfkd02/mbti-test

김하영님 코드리뷰

Opened this issue · 0 comments

하영님 너무 고생하셨습니다!

// console.log("전송된 formData 확인용!!!!! ====>", formData);

주석지우기!

const { login } = useContext(AuthContext);

useContext 로 AuthContext 를 불러서 사용하고 있는데 context 를 사용하실때 커스텀훅으로 묶어 사용하면 조금 더 간결하게 사용할 수 있어용.

export const useAuth = ()=>{
  return  useContext(AuthContext);
}

const MOCK_DATA = [

MOCK_DATA를 컴포넌트 내부에 선언하면 렌더링 시마다 재생성됩니다. 그래서 컴포넌트 외부에서 선언해서 사용하는편이 더 좋아보여요~

const token = localStorage.getItem("accessToken");

handleClick 함수에서 로컬 스토리지에서 직접 토큰을 확인하기보다는, 이미 AuthContext에서 제공하는 isAuthenticated를 활용하는 것이 코드의 일관성을 높일 수 있습니다.

const [isAuthenticated, setIsAuthenticated] = useState(!!token);

isAuthenticated 가 상태일 필요가 없어보여요!

const isAuthenticated = !!token // 혹은
const isAuthenticated = !!user

const isOwner = result.userId === user.id;

👍

하영님 코드가 정말 깔끔해요. 적절하게 정리되어있고 로직과 이름도 읽기 쉽게 잘 구성되어있어요! Query 도 사용해보시면 좋을것 같네요 :)

어렵다고 말씀해주신 부분들도 깔끔하게 잘 구현해주셨는데요?

+) 테일윈드는 지금 사용해주신것처럼 classname 에다가 다 넣어주시는 방법이 가장 직관적이고 효율적인 방법이 맞습니다 :) 모듈화 X