[Feat] Minigame refact & r2dbc 추가 #21
Conversation
Test Results12 tests 12 ✅ 1s ⏱️ Results for commit e2c325a. ♻️ This comment has been updated with latest results. |
…into minigame-refact
|
|
||
| @Table("game_results") | ||
| @Builder | ||
| @NoArgsConstructor |
There was a problem hiding this comment.
찾아보니 매핑이 되는 것으로 확인하였습니다. 매핑은 되나 DDL이 지원되지는 않는 것 같습니다. 만약 DB에 해당 테이블이 있다면 매핑은 됩니다.
There was a problem hiding this comment.
reactiveCrudRepository가 있는데 따로 만든 이유가 있나요?
There was a problem hiding this comment.
service에서 repository를 바라 볼 때 인터페이스를 바라보게 하고 싶어 그렇게 하였습니다. 해당 repository를 바로 사용해도 되지많은 mocking을 하든 아니면 repository가 변경이 되든 할 경우 유연하게 대처하고 싶어 의존성을 낮출려는 시도였습니다.
| .flatMap(gameResult -> { | ||
| gameResult.update(gameUpdateDTO); |
There was a problem hiding this comment.
flatMap은 무적인가요? 해당 코드에서 Map을 쓰면 안되나요?
There was a problem hiding this comment.
아직 리액티브 프로그래밍이 익숙하지 않아 거의 flatMap에 의존하여 사용하고 있는거 같습니다(거의 무적으로 쓰는듯...). 저번에 말해주신 것처럼 코스트가 크다고 했던 걸로 기억하여 대체할만한 것을 찾아 리팩토링 해보겠습니다.
✏️ 작업 개요
minigame 로직에 대한 리팩토링 및 r2dbc 추가 작업
⛳ 작업 분류
🔨 작업 상세 내용
💡 생각해볼 문제