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

Member와 StudyMember 연관관계 변경 #111

Closed
donghyun0304 opened this issue Jul 2, 2024 · 1 comment · Fixed by #112
Closed

Member와 StudyMember 연관관계 변경 #111

donghyun0304 opened this issue Jul 2, 2024 · 1 comment · Fixed by #112
Labels
Priority: Medium 우선순위 중 Status: To Do 처리해야 되는 이슈 Type: Feature/Function 새로운 기능 및 개선 사항

Comments

@donghyun0304
Copy link
Contributor

추가/개선 필요 요소

Member 엔티티와 StudyMember 엔티티의 연관관계를 일대다로 변경

추가/개선 필요 이유

기존에는 Member 엔티티에서 StudyMember를 setter로 주입받아 사용하였다.
성능면에서 주입받아 사용하는 것은 괜찮았으나

  • 코드를 이해하는 비용이 증가

위와 같은 이유로 Member엔티티와 StudyMember 엔티티는 일대다 관계로 변경한다.
성능 문제는 성능으로 인한 문제가 일어날때 고민한다.

@donghyun0304 donghyun0304 added Priority: Medium 우선순위 중 Status: To Do 처리해야 되는 이슈 Type: Feature/Function 새로운 기능 및 개선 사항 labels Jul 2, 2024
Copy link

Branch feature-111 created!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium 우선순위 중 Status: To Do 처리해야 되는 이슈 Type: Feature/Function 새로운 기능 및 개선 사항
Projects
None yet
1 participant