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.

Sign up for GitHub

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

Jump to bottom

리팩토링 #64

Open
CheolhoJeon opened this issue Aug 2, 2022 · 1 comment
Open

리팩토링 #64

CheolhoJeon opened this issue Aug 2, 2022 · 1 comment
Assignees
@byeongsoon @CheolhoJeon

Comments

@CheolhoJeon
Copy link
Contributor

CheolhoJeon commented Aug 2, 2022 •
edited
Loading

  • fixture를 이용한 테스트코드 중복 제거
  • controller, service 메서드 정렬 기준 적용 => 켄벤션화
    • Ex) HTTP 메서드 기준, 다루는 엔터티 기준
@byeongsoon
Copy link
Member

  • 패키지 구조 수정
  • 엔터티에서 Dto 등으로의 의존성 없애기
  • 맵퍼 계속 사용할지 제고
  • 팩토리 클래스 중복 코드 없애기

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@byeongsoon byeongsoon

@CheolhoJeon CheolhoJeon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@byeongsoon @CheolhoJeon