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

[PoC] Abstract Domain Model 정의 #448

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jyoo0515
Copy link
Member

@jyoo0515 jyoo0515 commented Feb 4, 2025

Checklist

  • 이슈들 보다가 해볼만 한 것 같아서 구현해봤습니다

  • toString, hashCode, equals 를 따로 구현하지 않아도 되도록합니다

  • toString 은 reflection 을 사용해서 성능이 조금 떨어지긴 할텐데, 어차피 toString 을 사용하는 곳이 별로 없을거라.. 그냥 조금 더 편한 길로 가봅니다 ㅎㅎ

  • 독립적인 abstract class 로 관리하고 싶었는데, 코틀린에서 abstract class 는 한개밖에 상속을 받지 못해서 TimeAuditingBaseEntity 가 상속받도록 했습니다

    • hierarchy 상 AbstractDomainModel (커스텀 도메인 모델) 보다는 jpa 에 종속적인 TimeAuditingBaseEntity 가 조금 더 higher level 인 느낌?
  • 만약 TimeAuditingBaseEntity 를 상속받는 모델이면 두 기능 (createdAt, updatedAt 이 트래킹되는 기능과, toString 이 다이나믹하게 구현되는 기능) 을 사용하게 되고, 만약 TimeAuditingBaseEntity 를 사용하지 않는다면, AbstractDomainModel 만 가져다가 쓰면 될 것 같습니다

  • resolves add abstract class for domain model #75

  • 충분한 양의 자동화 테스트를 작성했는가?

    • 계단정복지도 서비스는 사이드 프로젝트로 진행되는 만큼 충분한 QA 없이 배포되는 경우가 많습니다. 따라서 자동화 테스트를 꼼꼼하게 작성하는 것이 서비스 품질을 유지하는 데 매우 중요합니다.

@jyoo0515 jyoo0515 requested a review from a team as a code owner February 4, 2025 14:08
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

Successfully merging this pull request may close these issues.

add abstract class for domain model
1 participant