This Junior Developer Has a Lot to Learn (Code Review)
- 주니어 개발자의 코드 리뷰를 통해 JavaScript 및 React의 기본 개념 부족이 드러남 😥
- 변수 명명 규칙(Pascal Case vs Camel Case)과
let
vs const
사용에 대한 이해 부족 😕
- React 컴포넌트의 올바른 명명 규칙(PascalCase)을 지키지 않아 혼란을 야기함 😵
- 세션 유무 확인 없이 DB 쿼리를 실행하여 비효율적인 호출 또는 오류 발생 가능성 존재 😨
redirect
함수를 사용하여 세션이 없을 경우 로그인 페이지로 리다이렉트하는 방식으로 개선 제안 🤩
- Prisma를 사용할 때 타입 안정성을 확보하지 못해
any
타입이 남발되고 있음 😭
- 데이터베이스 호출을 컴포넌트 내에서 직접 수행하는 대신 유틸 함수를 만들어 추상화할 것을 권장 👍
- ORM 사용 시 타입 안정성을 확보하고, 유효하지 않은 컬럼 사용 시 오류를 감지할 수 있도록 설정해야 함 🤔
- 서버 컴포넌트에서 데이터베이스 직접 호출은 가능하지만, 쿼리 자체는 유틸 함수를 통해 추상화하는 것이 좋음 💡