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

[Rename] DTO 클래스 Request, Response 구분 #576

Open
5 of 11 tasks
miiiinju1 opened this issue Nov 4, 2023 · 0 comments
Open
5 of 11 tasks

[Rename] DTO 클래스 Request, Response 구분 #576

miiiinju1 opened this issue Nov 4, 2023 · 0 comments
Assignees

Comments

@miiiinju1
Copy link
Member

miiiinju1 commented Nov 4, 2023

이슈 개요

  • DTO 클래스 Request와 Response로 구분하여 변경한다.

상세 내용

현재 DTO 클래스들이 Request와 Response의 구분 없이 사용되고 있어 명확하지 않은 인터페이스를 제공하고 있다. 이로 인해 개발 시 DTO의 사용 목적을 쉽게 파악하기 어렵고, 오용의 가능성이 있다. 이를 개선하기 위해 DTO 클래스를 역할에 따라 Request DTO와 Response DTO로 분리하고자 한다.

  • 모든 컨트롤러에서 사용되는 DTO를 검토한다.
  • 각 DTO가 Request 용도인지, Response 용도인지를 결정한다.
  • Request와 Response 용도에 따라 클래스 명을 XxxRequest와 XxxResponse로 명명한다.
  • 변경된 DTO 명을 컨트롤러와 서비스 레이어에서 참조하도록 코드를 수정한다.

이슈 유형 (아래에 해당하는 경우 [x]로 체크해주세요)

  • 버그 리포트 (Bug Report)
  • 기능 요청 (Feature Request)
  • 문서 개선 제안 (Documentation Improvement)
  • 성능 개선 요청 (Performance Improvement)
  • 코드 개선 제안 (Code Refinement)
  • 기타 (Others) - 설명해주세요:

기능 상세 설명 (기능 요청의 경우)

  • 모든 Request DTO는 xxxRequest 형식으로 리네이밍한다.
  • 모든 Response DTO는 xxxResponse 형식으로 리네이밍한다.
  • 리팩토링 후, 단위 테스트를 통해 기능이 정상적으로 동작하는지 검증한다.

추가 컨텍스트

  • 이 작업은 코드의 가독성을 향상시키고, 나중에 DTO를 유지보수하는 데 있어 명확한 기준을 제공할 것이다.

체크리스트

  • 이슈의 유형을 선택했습니다.
  • 이슈의 상세 내용을 충분히 기술했습니다.
  • 문제 재현을 위한 단계를 기술했습니다 (해당하는 경우).
  • 기능의 필요성과 상세 내용을 기술했습니다 (해당하는 경우).
  • 추가 컨텍스트를 제공했습니다 (해당하는 경우).
miiiinju1 added a commit that referenced this issue Nov 5, 2023
Feat/#576 DTO 클래스 Request, Response 구분
@miiiinju1 miiiinju1 self-assigned this Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant