심화주차 개인과제 피드백

[modolee]

더 생각해 보기

• 전체적으로 고민해 본 흔적이 보입니다. 이론으로 바탕으로 실제 코드에도 적용할 수 있도록 노력해 주세요.
• Class Method에 왜 화살표 함수를 써야 되는지는 링크를 참고하세요.

Github

• 이슈, PR, 커밋 메세지은 입문주차 해설 영상을 참고해서 어떻게 작성해야 되는지 확인하시기 바랍니다.

README

• 다음 프로젝트부터는 README 파일에 해당 프로젝트에 대한 간략한 설명과 함께 처음 본 사람도 clone부터 어떻게 하면 실행할 수 있는지에 대한 과정을 적어주면 좋습니다.

환경변수

• .env.example 파일을 만들어서, 프로젝트 실행을 위해 필요한 환경변수를 제공하는 것이 좋습니다.

불필요한 패키지

• 이번 프로젝트에서는 prisma ORM으로 MySQL과 연결하여 사용합니다. mongoose ODM은 MongoDB와 연결할 때 사용하는 패키지로 이번 프로젝트에서는 불필요합니다. https://github.com/Parkseojin2/spa-office-re/blob/34e8c066eb499dee25190ce52fb375e2b215e27d/package.json#L19

RESTful 하지 못한 URL

https://github.com/Parkseojin2/spa-office-re/blob/34e8c066eb499dee25190ce52fb375e2b215e27d/src/routers/resumes.router.js#L13-L17

• 이력서 CRUD는 기본적으로 아래와 같이 5가지 API로 표현됩니다.

  POST /resumes 생성
  GET /resumes 목록 조회
  GET /resumes/:id (또는 :resumeId) 상세 조회
  PATCH(또는 PUT) /resumes/:id 수정
  DELETE /resumes/:id 삭제

API 동작하지 않음

• 레이어 분리는 했지만, 동작하는지 테스트하지 않은 것은 보이네요.
• 아래 코드에서 사용하는 HTTP_STATUS 를 import 하지 않아서 에러가 발생합니다. https://github.com/Parkseojin2/spa-office-re/blob/34e8c066eb499dee25190ce52fb375e2b215e27d/src/controllers/auth.controller.js#L13-L21

내 정보 조회

• 내 정보 조회 시 인증 미들웨어를 거치면서 이미 DB에서 사용자 정보를 조회합니다. 그러므로 controller에서 다시 한번 DB를 조회하지 않아도 됩니다.
• 그리고 Controller에서 바로 Repository 레이어를 호출하는 형태는 3-Layered Architecture에 맞지 않는 형태입니다.

레이어 분리

• 레이어 분리가 완벽히 되지 않았습니다. 해설 영상 참고해서 마무리 지어 보세요.
• 서비스 레이어에서 아직 prisma 코드를 사용하고 있습니다. https://github.com/Parkseojin2/spa-office-re/blob/34e8c066eb499dee25190ce52fb375e2b215e27d/src/services/auth.service.js#L40-L54 https://github.com/Parkseojin2/spa-office-re/blob/34e8c066eb499dee25190ce52fb375e2b215e27d/src/services/auth.service.js#L69-L72

[Parkseojin2]

넵