Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: BE 리팩토링 (Recruit) & ResponseEntity 추가 #150

Merged
merged 2 commits into from
Nov 30, 2022

Conversation

June1010
Copy link
Member

@June1010 June1010 commented Nov 29, 2022

Feature

  • 배경

    • 기존 1

      • 현재는 컨트롤러 레이어에서 응답코드와 data값을 직접 구성하여 클라이언트에 응답을 내려줌.
      • 이는, 로직과 응답값이 혼재해있어 어떤 속성들이 클라이언트에게 보내지는지 확인하는지 힘들고,
      • 매번 응답을 직접 구성하는 것 역시 번거로움.
    • 변경사항 1

      • ResponseEntity와 ResponseDto를 두고, 컨트롤러에서는 서비스에서 반환된 값을 ResponseDto로 변환.
      • ResponseEntity의 정적 메서드의 인자로 ResponseDto를 넘기면, dto와 상태코드를 묶어 직렬화해서 JSON 형태로 응답
      • 관련 recruit 코드 (recruit 모듈) 변경
    • 기존 2

      • 인터셉터에서 액세스 토큰의 헤더 포함 여부에 따라 userId를 body에 넣어주고 있는데, get 요청 같은 경우 body에 접근하여 값을 가져오는게 부자연스러움.
    • 변경사항 2

      • get 요청은 params 안에 값을 넣어주게끔 변경
      • 관련 recruit 코드 (getOne) 로직 일부 변경
  • 목적

    • Response 처리 용이
    • Dto, Entity 역할 분리
    • Swagger 추후 적용 용이

과정

  • ResponseEntity 구현
  • 응답용 Dto 추가 (Recruit)
  • 관련 컨트롤러 코드 수정
  • HttpBodyInterceptor 로직 수정
  • 관련 컨트롤러, 서비스 코드 수정

결과 (스크린샷 등

스크린샷 2022-11-30 오전 6 42 27

)

스크린샷 2022-11-30 오전 6 43 13

스크린샷 2022-11-30 오전 6 43 33

스크린샷 2022-11-30 오전 6 47 57

관련 issue 번호 (링크)

#149

테스트 방법

Postman

Commit

Copy link
Member

@pushedrumex pushedrumex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@gchoi96 gchoi96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@gchoi96 gchoi96 merged commit 2b35709 into develop Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants