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

도메인 계층에 정의된 모델에 ORM 엔티티도 같이 정의하고, 관련 클래스 정리 #62

Open
Coalery opened this issue Sep 16, 2024 · 2 comments
Assignees
Labels
refactoring Refactor code request

Comments

@Coalery
Copy link
Member

Coalery commented Sep 16, 2024

도메인 계층과 ORM 엔티티를 분리할 만큼 규모가 크지 않음.

따라서,

  • 도메인 계층에 정의된 모델에 ORM 엔티티(ex @Entity 데코레이터 및 @Column 데코레이터 등)도 같이 정의
  • 도메인 계층에 인터페이스로 정의되어 있던 레포지토리 제거하고 @InjectRepository로 주입 받도록 수정
  • 이에 따라 불필요해지는 클래스 (ex 매퍼 등) 제거
    • 팩토리는 모델 생성 규칙 통일을 위해 남깁니다.
@Coalery Coalery added the refactoring Refactor code request label Sep 16, 2024
@Coalery Coalery self-assigned this Sep 16, 2024
@Coalery
Copy link
Member Author

Coalery commented Oct 9, 2024

트랜잭션 적용을 위해 custom repository를 적용해야 할 필요성이 있을 것으로 보임.
이 경우 도메인에 정의된 모델에 DB 관련 내용이 많이 섞이게 돼서, 과한 책임이 생길 것 같음.

따라서 엔티티와 도메인 모델을 별도로 정의하고, 이 사이 매퍼도 정의해야 할 것으로 보임.

@Coalery Coalery closed this as not planned Won't fix, can't repro, duplicate, stale Oct 9, 2024
@Coalery Coalery reopened this Oct 13, 2024
@Coalery
Copy link
Member Author

Coalery commented Oct 13, 2024

굳이 @Transactional을 사용해야 할까? 생각해보면, 규모가 그리 크진 않아서 아직까지는 굳이라는 느낌이 조금 있는 거 같음.
따라서 기존 안대로 진행하기로 결정.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Refactor code request
Projects
None yet
Development

No branches or pull requests

1 participant