[구현] ADD: 유저 금액 충전

yujin-min commented 7 months ago

변경 사항

구조 변경

auth 로직을 users 모듈에서 분리
모듈 지식에 종속되지 않은 헬퍼들은 최상위에 common에 들어가도록 분리
entities 대신 models로 폴더 이름 변경 뒤 entity, vo, domain-service는 그 아래 존재하도록 변경
유저 타입 개념 추가
회사 개념 및 기능을 추가하기 전이라 일단 유저 필드로 디폴트 값 추가
business, personal 2가지 타입 존재
계좌 개념 추가
유,무상에 따라 balance, point 분리
유저와 1:1 관계
유저 생성시 계좌도 같이 생성
유저 제거시 계좌도 같이 제거
기능 추가
users/self/charge
계좌번호를 기준으로 충전하는게 아닌 유저를 통해서 충전하도록 구성

yujin-min commented 7 months ago

모듈 범위를 어떻게 잡아야 할지 정확히 모르겠어서 고민이 있었습니다. 예를 들어 account-dto 자체는 users-controller에서 사용하고 있기에 유저 모듈 안에 들어가 있는 상태인데 사실 그 지식이나 개념 자체는 account 소속이라 어디에 두는게 맞을지 헷갈렸습니다. 이런 경우에 사실 어떻게 보면 users/self/charge 대신 accounts/:account-id/charge로 구성하면 해결될 문제일테니 user에 충전 기능을 둔 것 자체가 문제일수도 있겠네요. 후자의 경우로 변경한다면 유저타입 정보를 토큰에 넣어서 확인해도 되는건지 아니면 테이블에서 직접 가져오는게 나은건지도 고민이 됩니다.

트랜잭션 자체를 인터셉터로 빼서 데코레이터로 사용하는 경우를 종종 봤는데 이게 맞는 건지 헷갈려서 사용하지 않았습니다. 트랜잭션이 이뤄져야 하는 계층의 범위를 넘어서는 느낌이 들어서요. 어떻게 생각하시는지 궁금합니다. 일단은 queryRunner를 인자로 넘겨서 두 서비스를 넘나들도록 추가했습니다.

계좌 기능에 Money, Point 객체를 추가하고 계산 자체는 Service로 빼서 구성했는데 이렇게 구성해도 되는건지 잘 모르겠습니다. money, point를 하나의 객체로 활용할 수도 있었겠다는 생각이 드네요.

이번에도 PR이 늦었네요. 의견을 남겨달라고 하셔서 남겨봅니다. 항상 감사합니다! 🙇 @f-lab-namu