Skip to content

Conversation

@PPakSang
Copy link
Contributor

DIR Entity, Repository, Test 추가하였습니다.

Repository 는 생성, 조회만 추가하였고, 현재 PageInDIR, UnitInDIR 연관관계 매핑은 포함되어 있지 않습니다.

기존 설계에서 한 가지 수정한 것은 클라이언트에게 제공될 탐색 경로가 필요하다고 판단되어 path 를 추가했습니다.

부모 DIR 이 /root/parentDIR 의 경로에 존재하고, 해당 경로에 sample.unit 을 추가하고 싶다면

parent path : /root/
parent DIR Name : parentDIR

child path : parent path + parent DIR Name + "/"
child unit name : sample

이 됩니다.

모든 DIR 내 조회 생성 로직은 create(child path, child unit name) 로 이루어져 있습니다.

@PPakSang PPakSang added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 27, 2022
@PPakSang PPakSang requested review from HyeW and dkdud9261 July 27, 2022 07:14
@PPakSang PPakSang self-assigned this Jul 27, 2022
@PPakSang
Copy link
Contributor Author

추가 할 로직

  1. Unit, Page 연관관계 매핑
  2. 연관관계 매핑에 따른 ID 값을 활용한 경로 추출 로직 추가

@PPakSang
Copy link
Contributor Author

PPakSang commented Jul 27, 2022

getSingleResult() 는 다른 레이어에서 예측 불가능한 예외를 발생시킬 위험이 존재
수정 검토 필요

금일 회의 결과에 따른 service layer 생성 전 setter 추가 금지 및 단일 레코드 조회 시 getSingleResult 는 다수의 Exception handling 이 필요하기 때문에 Optional 을 반환할 수 있는 패턴으로 수정했습니다.
@PPakSang PPakSang merged commit 259c063 into develop Aug 2, 2022
@PPakSang PPakSang deleted the feat/issue-4/dir_entity branch August 2, 2022 14:33
@PPakSang PPakSang restored the feat/issue-4/dir_entity branch August 2, 2022 14:34
@PPakSang PPakSang linked an issue Aug 2, 2022 that may be closed by this pull request
@PPakSang PPakSang changed the title FEAT : DIR Entity, Repository, Test 추가 Feat: DIR Entity, Repository, Test 추가 Aug 2, 2022
@PPakSang PPakSang changed the title Feat: DIR Entity, Repository, Test 추가 Feat: DIR Entity, Repository, Test 추가 (연관관계 매핑 전) Aug 2, 2022
@PPakSang PPakSang deleted the feat/issue-4/dir_entity branch August 2, 2022 16:05
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.

Domain Entity

2 participants