[FEAT] 아바타 대사 API 간단 버전 구현

05AM commented 7 months ago

☀️ 작업 사항

아바타 대사 API 간단 버전을 구현했습니다.

☀️ 관련 이슈

related : #116

☀️ 참고 사항

패키지 변경이 있어 이슈 번호가 붙은 커밋 위주로 보시기를 추천 드립니다.
아바타 대사의 경우 조건에 따라 대사가 달라지는 로직이 있는데 구현에 시간이 좀 걸릴 것 같아 조건 없이 랜덤으로 반환하는 api를 먼저 구현하였습니다.
추후 수정하여 pr 올리겠습니다.
또한 현재 45자를 넘어 60자에 임박하는 아바타 대사가 존재하여 일단 컬럼 데이터형을 varchar(60)으로 설정했습니다.
디자인 파트와 상의하여 글자 수를 늘릴 수 있다면 늘리고, 어렵다면 대사를 수정하여 45자로 변경하는 식으로 진행하겠습니다.
avatar의 서비스와 프로바이더들을 넣는 패키지명을 ddd 아키텍처에 따라 application이라고 명명했는데 이것에 대한 논의가 필요할까요?

hye-on commented 7 months ago

avatar의 서비스와 프로바이더들을 넣는 패키지명을 ddd 아키텍처에 따라 application이라고 명명했는데 이것에 대한 논의가 필요할까요?

다른 패키지들은 service로 묶고 있으니 통일하면 좋을 것 같습니다. application을 선호하신다면 다른 패키지들의 이름도 같이 바꾸면 좋을 것 같습니다.

hye-on commented 7 months ago

pr 올릴게 있어서 머지 하겠습니다!. 고생하셨습니다.

05AM commented 7 months ago

@hye-on

이상하다 생각했더니 제가 혜온님 코멘트를 편집했네요! 코멘트 다시 달았습니다

application을 선호하신다면 다른 패키지들의 이름도 같이 바꾸면 좋을 것 같습니다.

네 개인적인 의견으로는 provider와 service로 나누게 되었으니 이후를 고려해서 리팩토링 시에 service 패키지 이름을 application으로 바꾸는게 좋을 것 같은데 어떻게 생각하시나요?

hye-on commented 7 months ago

네 개인적인 의견으로는 provider와 service로 나누게 되었으니 이후를 고려해서 리팩토링 시에 service 패키지 이름을 application으로 바꾸는게 좋을 것 같은데 어떻게 생각하시나요?

좋아요 그럼 provider도 application에 들어가게 될까요?

ㄴ command ( 빈칸) ㄴ application ( 빈칸) ㄴ domain ㄴ query

이렇게 해도 좋을 것 같습니다.

hye-on commented 7 months ago

@05AM

05AM commented 7 months ago

네 개인적인 의견으로는 provider와 service로 나누게 되었으니 이후를 고려해서 리팩토링 시에 service 패키지 이름을 application으로 바꾸는게 좋을 것 같은데 어떻게 생각하시나요?

좋아요 그럼 provider도 application에 들어가게 될까요?

ㄴ command ( 빈칸) ㄴ application ( 빈칸) ㄴ domain ㄴ query

이렇게 해도 좋을 것 같습니다.

개인적인 생각으로는 query든 command 든 표현 계층과 구분되는 응용 계층이라고 생각이 되어 applcation 내부에 query와 command가 있다면 좋을 것 같습니다.

application이 infra, domain과 같이 하나의 계층을 나타내는 패키지라고 생각합니다. controller와 command 패키지가 같은 층에 있는 것은 조금 어색할 수 있을 것 같습니다.

provider가 이제 queryService가 된 것 말씀하시는거라고 생각하고 말씀드리자면 application 내부에 포함되면 좋을 것 같습니다.

hye-on commented 7 months ago

개인적인 생각으로는 query든 command 든 표현 계층과 구분되는 응용 계층이라고 생각이 되어 applcation 내부에 query와 command가 있다면 좋을 것 같습니다.

https://github.com/madvirus/ddd-start2/tree/main/src/main/java/com/myshop/order 저희가 스터디한 책의 깃레포를 보면 command, query로 한번 나누고 그 안에 application과 domain이 각각 있더라고요

이렇게 command와 query의 하위 레이어에 각각 application이 있으면 좋을 것 같습니다.

찬미님 의견대로 application 하위에 query,command로 분리한다면 서비스들을 묶는 패키지 이름을 뭐라고 하면 좋을까요?( 안 묶게 될까요?) 혹시 의견에 참고하신 레포 있으면 공유부탁드립니다!

application ㄴcommand ㄴ(서비스 묶는 패키지 이름) ㄴquery ㄴ(서비스 묶는 패키지 이름)

Wake-up-together-TogetUp / togetup-server