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: Unit Domain DoubleUnit, MergedUnit, Repository, Test 추가 #24

Open
wants to merge 1 commit into
base: feat/issue-4/unit_entity
Choose a base branch
from

Conversation

HyeW
Copy link

@HyeW HyeW commented Aug 15, 2022

PR 요약

  1. DoubleUnit, MergedUnit Entity, Repository, Test 추가

변경 사항

  1. SingleUnit파트를 제외한 나머지 부분 Enity, Repository, Test 를 추가합니다.
  2. Test는 SingleUnit이 있다는 전제 하에 만들어 현재 코드로만은 실행되지 않습니다. (혼자서 SingleUnit Class를 만들어 테스트 완료)
  3. 데이터 타입 중 'Type'은 사용자 시점 Unit 분류 기준이고 'Form'은 개발자 시점 Unit 분류 기준입니다.
  4. Repository는 현재 생성, 단건 조회, 삭제만 가능합니다.

클래스 다이어그램

Untitled Diagram (1)

참고 사항

유지보수를 위해 단방향 매핑을 기본으로 하여 코드를 작성합니다.
DoubleUnit, MergedUnit과 MappedUnit은 Double과 Merged를 저장할 때 Mapped가 저장되는 것이 논리 상 맞아 양방향 매핑을 사용합니다.
@HyeW HyeW added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 15, 2022
@HyeW HyeW self-assigned this Aug 15, 2022
@PPakSang
Copy link
Contributor

  1. 태그 도메인 분리
    

추후에 태그 관리(태그명 중복 검사, 최대 갯수 등등) 를 할 때 태그를 유닛 도메인 쪽에 포함하면 유닛 도메인의 책임이 많아질 것 같은데 어떻게 생각하시나요?

  1. MappedUnit
    

추가 배경: Single Unit 과 연관관계를 끊고 Double Unit, Merged Unit 과 생명주기를 같이 하는 MappedUnit 을 추가

현재 코드 상으로 봤을 때 원본 유닛(origin Unit) 과 연관관계를 맺고 있기 때문에 원본 파일 수정 시 mapped Unit 내용도 수정될 것 같은데 의도한 바가 맞나요?

Mapped Unit 에 Double Unit 혹은 Merged Unit 하나만 매핑 -> 현재는 두 필드 전부 연관관(ManyToOne) 를 맺고 있는데 혹시 나중에 어떻게 조회할 예정이신가요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants