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

♻️ BaseSemesterEntitySemester VO로 변환 #803

Open
uwoobeat opened this issue Oct 12, 2024 · 0 comments
Open

♻️ BaseSemesterEntitySemester VO로 변환 #803

uwoobeat opened this issue Oct 12, 2024 · 0 comments
Assignees
Labels
♻️ refactor 기능 개선

Comments

@uwoobeat
Copy link
Member

uwoobeat commented Oct 12, 2024

📌 Description

  • 현재 학년도 / 학기를 나타내는 BaseSemesterEntity@MappedSuperClass 를 사용하여 엔티티의 부모 클래스로 다루고 있음
  • 현재 해당 클래스를 상속하는 엔티티로는 Recruitment, RecruitmentRound, Study 가 있음
  • 세 엔티티, 특히 리쿠르팅 계열과 스터디 계열은 거의 공통점이 없고, 이를 추상화한 BaseSemesterEntity에 대하여 같은 층위의 위계구조를 가진다고 보기에도 적절하지 않음.
  • 따라서 상속 대신 합성을 사용하는 방식으로 개선 -> Semester VO에 대한 프로퍼티를 가지게 하는 식으로 개선
  • 기존 학년도 / 학기 필드에 대한 요청 / 응답 DTO의 경우 적절히 대체하거나 / 기존 스키마와 호환되도록 (프론트 측과 협의 필요)
@uwoobeat uwoobeat added the ♻️ refactor 기능 개선 label Oct 12, 2024
@uwoobeat uwoobeat self-assigned this Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 기능 개선
Projects
None yet
Development

No branches or pull requests

1 participant