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

feat : 엔티티 세팅 #22

Merged
merged 21 commits into from
Jun 21, 2024
Merged

feat : 엔티티 세팅 #22

merged 21 commits into from
Jun 21, 2024

Conversation

GaBaljaintheroom
Copy link
Contributor

@GaBaljaintheroom GaBaljaintheroom commented Jun 16, 2024

😋 작업한 내용

  • ShowPot 엔티티 세팅

🙏 PR Point

  • 주요 엔티티와 직접적인 연관을 가지고 있다고 생각한 Genre와 Artist, Image, ShowSearch, ArtistSearch 는 객체 참조로 관계를 지었고, 그 외는 UUID로 FK형식으로 관계를 지어보았습니다.
  • JPA에서 발생할 수 있는 문제점 중 하나가 무분별한 객체 참조로 인한 순환이 가능해서 이렇게 해보았는데, 찬기님 생각은 어떻신지 궁금합니다.
  • 회의 때 Image 엔티티는 없었는데, 제가 생각했을 때 Show를 저장할 때 image가 여러개를 저장할 수 있을 것 같다고 생각하여 만들었습니다.

👍 관련 이슈


@GaBaljaintheroom GaBaljaintheroom self-assigned this Jun 16, 2024
@GaBaljaintheroom GaBaljaintheroom added the feat add feature label Jun 16, 2024
Copy link
Contributor

@devmizz devmizz left a comment

Choose a reason for hiding this comment

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

멀티 모듈을 사용하면서 도메인 단위로 모듈을 찢을 때, 도메인의 적절한 크기도 중요해요. 같은 계층에서 도메인 간의 의존이 좋지는 않으니깐요. 서비스가 서비스를 참조하면 내포하고 있는 위험성이 많은 것처럼요. 지금은 도메인을 적절하게 분리하지 않고 테이블 단위로 다 찢어놓은 느낌이에요.

지금의 분리는 편의성도 도메인도 모두 포기한 체, 기계적으로 테이블 단위로 모두 찢은 거 같아요. 조금만 더 고민해보면 좋을 거 같아요! 혹시 다른 의견이 있으면 편하게 주십쇼!

@GaBaljaintheroom
Copy link
Contributor Author

Ticketing에 대해 생각해보았는데, 테이블로 분리까지는 굳이라는 생각이 들었어요. 그래서 일단 Map의 키를 enum -> String으로 유연하게 변경해보았습니다.

@GaBaljaintheroom GaBaljaintheroom merged commit 7179f06 into develop Jun 21, 2024
1 check passed
@GaBaljaintheroom GaBaljaintheroom deleted the feat-entity-setting branch June 21, 2024 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat add feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants