Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 경매는 재고보다 많은 양의 사용자 요청을 처리할 수 없다. #59

Merged
merged 5 commits into from
Aug 13, 2024

Conversation

chhs2131
Copy link
Collaborator

@chhs2131 chhs2131 commented Aug 12, 2024

📄 Summary

  • 입찰을 진행하려는 경매가 존재하는지 검증
    • AuctionId에 해당하는 경매가 존재하는지 확인한다.
  • 경매 입찰(구매)을 위해 전달되는 수량을 검증
    • 남은 재고보다 더 많은 수량을 입찰하려는 경우를 검증한다.
    • 인당 구매 수량 제한을 벗어나는지 검증한다.
  • 입찰/환불 측 로직은 이후에 동시성을 반드시 고려하여 개선해야합니다.

🙋🏻 More

close #58

- submitBid에 전달된 auctionId가 유효한지 검증한다.
- 유효하지 않은 경우 A011 에러가 발생한다.
- 남은 재고보다 더 많은 수량을 입찰하려는 경우를 검증한다.
- 인당 구매 수량 제한을 벗어나는지 검증한다.
- 경매 입찰 요청 수량 검증에 대한 A014 에러를 추가한다.
- 경매 재고보다 많은 수량을 입찰(구매)하려는 경우 예외가 발생
- 인당 구매 제한 수량보다 많은 수량을 입찰(구매)하려는 경우 예외가 발생
@chhs2131 chhs2131 added the FEATURE 새로운 기능 또는 변경 label Aug 12, 2024
@chhs2131 chhs2131 self-assigned this Aug 12, 2024
@chhs2131 chhs2131 requested review from a team and yudonggeun August 12, 2024 13:54
yudonggeun
yudonggeun previously approved these changes Aug 13, 2024
Copy link
Contributor

@yudonggeun yudonggeun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋아요

@@ -22,7 +22,7 @@ public class Auction {
@Builder
private Auction(final ZonedDateTime startedAt, final Long sellerId, final String productName,
final long originPrice,
final int stock,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

long 예리한데 👍

Copy link
Collaborator

@minseok-oh minseok-oh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋습니다!!

@minseok-oh minseok-oh merged commit 3aceb0c into dev Aug 13, 2024
@chhs2131 chhs2131 deleted the feat/58-auction-bid-validate-stock branch August 13, 2024 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEATURE 새로운 기능 또는 변경
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] 경매는 재고보다 많은 양의 사용자 요청을 처리할 수 없다
3 participants