Refactor | CAKK-58 | User 비즈니스에 Facade 패턴 적용

[lcomment]

Issue Number

CAKK-58

Description

User 관련 로직에 Facade Pattern을 적용하고, 미사용 Writer를 제거하였습니다. Writer는 리팩토링 종류 후 Deprecated 될 예정인데, Reader에 대한 고민이 있습니다.

1. Reader도 Facade로 네이밍 변경이 필요해 보입니다.
2. Querydsl로 작성된 일부 쿼리를 도메인 모델 패턴로 리팩토링 하는 것도 좋아보입니다.

관련하여 코멘트 주세요.

etc

[github-actions]

Test Results

 39 files  + 5   39 suites  +5   26s ⏱️ +7s
214 tests +20  214 ✅ +20  0 💤 ±0  0 ❌ ±0 
215 runs  +21  215 ✅ +21  0 💤 ±0  0 ❌ ±0 

Results for commit 3919bf3. ± Comparison against base commit 3aef33c.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

@@              Coverage Diff              @@
##             develop     #197      +/-   ##
=============================================
- Coverage      89.72%   89.18%   -0.55%     
+ Complexity       327      326       -1     
=============================================
  Files            110      110              
  Lines            983      980       -3     
  Branches          36       36              
=============================================
- Hits             882      874       -8     
- Misses            85       91       +6     
+ Partials          16       15       -1     

Files with missing lines	Coverage Δ	Complexity Δ
...in/java/com/cakk/api/service/user/SignService.java	100.00% <100.00%> (ø)	5.00 <0.00> (ø)
...in/java/com/cakk/api/service/user/UserService.java	100.00% <100.00%> (ø)	3.00 <1.00> (ø)

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3aef33c...3919bf3. Read the comment docs.

[YongsHub]

Reader에 로직이 들어가다보니 Facade가 필요하다는 건가요?

[lcomment]

Reader에 로직이 들어가다보니 Facade가 필요하다는 건가요?

그런 것도 있고,,, 이거 domain 모듈에 domain:core로 하나 만들어서 facade를 담는 모듈 자체로 추상화 하는건 너무 오버 엔지니어링 일까요? read 작업에 redis도 필요할거 같은데(인기 케이크 조회 등) 너무 세분화 하는거 같아서 고민이 있네요.

[YongsHub]

Reader에 로직이 들어가다보니 Facade가 필요하다는 건가요?

그런 것도 있고,,, 이거 domain 모듈에 domain:core로 하나 만들어서 facade를 담는 모듈 자체로 추상화 하는건 너무 오버 엔지니어링 일까요? read 작업에 redis도 필요할거 같은데(인기 케이크 조회 등) 너무 세분화 하는거 같아서 고민이 있네요.

일단 모듈까지 가지는 말고 새로운 기능 개발할 때 확장 가능성을 보고 다시 얘기해봐요!