feedback

[ggomma]

feedback

총평

• 필요한 코드들을 잘 구현하였습니다.
• 개발에 대한 이해도가 높아보입니다.
• readme.md를 작성해주세요.
• 수고하셨습니다.

필요 구현 사항

• 필수
  • [x] API 명세서 작성
  • [x] ERD 작성
  • [x] 회원가입 API
  • [x] 로그인 API
  • [x] Access Token 인증 Middleware
  • [x] 내 정보 조회 API
  • [x] 내 이력서 생성 API
  • [x] 내 이력서 목록 조회 API
  • [x] 내 이력서 상세 조회 API
  • [x] 내 이력서 수정 API
  • [x] 내 이력서 삭제 API
• 선택
  • [x] 역할 인가 Middleware
  • [x] 채용담당자가 모든 이력서 조회 API
  • [x] 채용담당자가 특정 이력서 조회 API
  • [x] 채용담당자가 이력서 상태 변경 API
  • [x] 채용담당자가 이력서 로그 목록 조회 API
  • [x] RefreshToken 인증 Middleware
  • [x] 토큰 재발급 API (🔐 RefreshToken 인증 필요)
  • [x] 로그아웃 API (🔐 RefreshToken 인증 필요)

개선 사항

Readme.md

readme.md 파일은 해당 코드를 처음 접하는 사람들에게 많은 도움을 줍니다. 연습한다고 생각하고 작성해보세요.

불필요한 호출 최소화

아래 코드를 database 조회 이전으로 옮기면 굳이 불필요하게 데이터베이스를 호출할 필요가 없습니다.

https://github.com/lunaradio24/recruitment-platform/blob/11e414e8eac20d3be154c514edf79abfe186deee/src/routers/auth.router.js#L32-L36

사용하지 않는 코드 삭제

아래 코드는 사용하지 않는 코드입니다. 삭제해주세요.

https://github.com/lunaradio24/recruitment-platform/blob/11e414e8eac20d3be154c514edf79abfe186dee묘e/src/routers/auth.router.js#L154

이해하기 쉬운 단어 사용

아래에서 tokenId 보다는 refreshToken 등을 사용하는 것이 직관적입니다. 일반적으로 id는 어떤 의미를 갖기보다는 identifier로서의 역할만 수행합니다.

https://github.com/lunaradio24/recruitment-platform/blob/11e414e8eac20d3be154c514edf79abfe186deee/src/routers/auth.router.js#L162

로그아웃 http method

일반적으로 로그아웃을 위해서는 POST method를 사용합니다.

코드 효율성

아래 코드에서 title, personalStatement 가 들어오지 않을 경우 불필요한 update 작업을 수행합니다. 필요하지 않을 경우 아예 작업을 수행하지 않는 편이 낫습니다.

https://github.com/lunaradio24/recruitment-platform/blob/11e414e8eac20d3be154c514edf79abfe186deee/src/routers/resumes.router.js#L160-L169

코드 효율성

아래 코드의 경우 단지 email을 가져오기 위해 매번 데이터베이스를 조회합니다. 이 경우 차라리 email을 req.user에 함께 넣어주는 편이 더 나을 것 같습니다.

https://github.com/lunaradio24/recruitment-platform/blob/11e414e8eac20d3be154c514edf79abfe186deee/src/routers/users.router.js#L11-L26

환경변수

환경변수를 불러오는 부분은 코드 최상단(여기서는 app.js)에 두는 것이 직관적입니다.

https://github.com/lunaradio24/recruitment-platform/blob/11e414e8eac20d3be154c514edf79abfe186deee/src/utils/auth.util.js#L3

[lunaradio24]

수정했습니다! 덕분에 더 효율적인 코드가 되었네요. 감사합니다!! 👍