아이템 구매 로직 추가 #63
아이템 구매 로직 추가 #63
Conversation
onegqueen
added
documentation
| @Column(name = "purchase_log_id") | ||
| private Long id; | ||
|
|
||
| @Column |
There was a problem hiding this comment.
크레딧 service 단에서 순환참조가 발생하길래 살펴보니
creditService에 구매로그엔티티가 추가되는 로직 이외에는 user를 조회하는 경우는 없는데 구매로그엔티티에서 userEntity를 필드로 가져버리니까 credit도메인에서 user를 참조할 수 밖에 없게 되서 id만 갖도록 수정했습니다.
fetch 타입을 Lazy로 해서 join컬럼의 조회성능문제는 해결하더라도 이런문제가 발생될수 있더라구요,,
There was a problem hiding this comment.
움 유저 엔티티랑 여기 엔티티랑 순환참조인걸로 이해했는데, 여기서는 양방향 관계가 필요한거죠?
| @Transactional | ||
| public PurchaseResponse purchaseItem(Long userId, PurchaseRequest request) { | ||
| ItemInfo purchasedItem = itemService.addUserItem(userId, request.itemId()); | ||
| int balance = updateUserCredit(userId, -purchasedItem.price(), request.creditType()); |
There was a problem hiding this comment.
유저 크레딧이 증가되면 양수, 감소하면 음수로 amount를 업데이트 시키는데 코드에 음수가 들어가니 보기가 안좋은거 같긴해서.. 변경타입(입금/출금)을두고 엔티티 내에서 입금이면 +시키고 출금이면 -시키는 편이 좋을까요 ?
There was a problem hiding this comment.
확실히 파라미터보단 별도 메서드로 만들어주는건 좋은것 같아용
| @Transactional | ||
| public PurchaseResponse purchaseItem(Long userId, PurchaseRequest request) { | ||
| ItemInfo purchasedItem = itemService.addUserItem(userId, request.itemId()); | ||
| int balance = updateUserCredit(userId, -purchasedItem.price(), request.creditType()); |
There was a problem hiding this comment.
확실히 파라미터보단 별도 메서드로 만들어주는건 좋은것 같아용
| @Column(name = "purchase_log_id") | ||
| private Long id; | ||
|
|
||
| @Column |
There was a problem hiding this comment.
움 유저 엔티티랑 여기 엔티티랑 순환참조인걸로 이해했는데, 여기서는 양방향 관계가 필요한거죠?
🤨 Motivation
🔑 Key Changes
아이템 구매 로직 구현
아직 결제기능 구현에 대한 내용은 미정이지만 두가지 타입으로 구매할 수 있도록 구현해두었습니다.
아이템을 구매하면 구매한 아이템과 잔액정보를 반환하며, 인벤토리에도 구매한 아이템이 추가됩니다.
🙏 To Reviewers