피드백 - 예병수 튜터

[DevelopSoo]

발표 피드백

1. 기본적인 CRUD지만 고급스럽게 만드셨습니다. 사람들은 시각적인 부분에 크게 영향을 받기 때문에 이 부분에 신경을 쓰시는 것도 잘하셨다는 생각이 듭니다.
2. 새로운 모듈 사용법을 공식문서만 보고 어려웠을 것입니다. 특히 주니어에게는요. 이때 예시 코드를 클론 해보고 테스트도 해보면서 학습을 하는 것이 큰 도움이 됐을 것입니다. 꽤 많은 사람들이 이런 수고를 하지 않으려고 하는데요. 새로운 기술의 습득에 적극적이셔서 더 좋은 결과를 얻으신 것 같습니다.

코드 피드백

1. 실제 데이터가 변경되어서 자동으로 세팅하는 게 아니라 억지로 render 시키는 중입니다. 즉, 필요할 때 렌더링하는 것이 아니라 임시방편의 방법이라는 생각이 듭니다. 이 부분은 지우고 버그를 해결하시는 것이 더 좋을 것 같습니다.
   

   nbc_manlody/src/App.tsx

   Lines 22 to 29 in 4536465

   	const [render, setRender] = useState(false);
   	useEffect(() => {
   	onAuthStateChanged(auth, (user) => {
   	if (!user || !accessToken) navigate('/login');
   	setRender(true);
   	});
   	}, []);

2. 타입스크립트를 학습하기 때문에 any가 아니라 적절한 타입을 넣는 것을 추천드립니다. any를 사용하면 javascript를 사용하는 것과 별반 다를게 없습니다.
   

   nbc_manlody/src/components/Profile.tsx

   Lines 16 to 19 in 4536465

   	const [selectefFile, setselectefFile] = useState<any>('');
   	const { userName, userEmail, userImg, setUserProfile } = useUser();
   	const [modalUserImg, setModalUserImg] = useState<any>(userImg);

   
   

   nbc_manlody/src/pages/Home.tsx

   Lines 11 to 13 in 4536465

   	const [homeShowTracks, setHomeShowTracks] = useState<any>([]);
   	const [homeShowArtists, setHomeShowArtists] = useState<any>([]);
   	const [homeShowAlbums, setHomeShowAlbums] = useState<any>([]);

3. ts-ignore도 마찬가지로 type을 지정하지 않겠다는 의미입니다. 이 부분도 추후에 적절한 방법을 찾는 것을 추천드립니다.
   https://github.com/CTDKSKM/nbc_manlody/blob/4536465ff1ba694d24e4d1c0dfb230bb9d6bf949/src/pages/FavoriteSongs.tsx#L24C7-L24C7

[wooriki]

자세한 설명 감사합니다 병수튜터님 !