diff --git a/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameController.java b/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameController.java index c89f3ef..a10fe63 100644 --- a/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameController.java +++ b/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameController.java @@ -94,23 +94,6 @@ public class BlackJackGameController { return ResponseEntity.ok(blackJackService.doubleDown(game)); } - @PostMapping("/blackjack/{id}/split") - public ResponseEntity split(@PathVariable Long id, @RequestHeader("Authorization") String token) { - Optional optionalUser = userService.getCurrentUser(token); - - if (optionalUser.isEmpty()) { - return ResponseEntity.notFound().build(); - } - - UserEntity user = optionalUser.get(); - BlackJackGameEntity game = blackJackService.getBlackJackGame(id); - if (game == null || !Objects.equals(game.getUserId(), user.getId())) { - return ResponseEntity.notFound().build(); - } - - return ResponseEntity.ok(blackJackService.split(game)); - } - @PostMapping("/blackjack/start") public ResponseEntity createBlackJackGame(@RequestBody @Valid CreateBlackJackGameDto createBlackJackGameDto, @RequestHeader("Authorization") String token) { Optional optionalUser = userService.getCurrentUser(token); diff --git a/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameEntity.java b/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameEntity.java index 4b03b21..4f22c9d 100644 --- a/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameEntity.java +++ b/backend/src/main/java/de/szut/casino/blackjack/BlackJackGameEntity.java @@ -51,15 +51,4 @@ public class BlackJackGameEntity { @JsonManagedReference @SQLRestriction("card_type = 'DEALER'") private List dealerCards = new ArrayList<>(); - - @OneToMany(mappedBy = "game", cascade = CascadeType.ALL, orphanRemoval = true) - @JsonManagedReference - @SQLRestriction("card_type = 'PLAYER_SPLIT'") - private List playerSplitCards = new ArrayList<>(); - - @Column(name = "split_bet") - private BigDecimal splitBet; - - @Column(name = "is_split") - private boolean isSplit; } diff --git a/backend/src/main/java/de/szut/casino/blackjack/BlackJackService.java b/backend/src/main/java/de/szut/casino/blackjack/BlackJackService.java index ebddd1a..2ce7b07 100644 --- a/backend/src/main/java/de/szut/casino/blackjack/BlackJackService.java +++ b/backend/src/main/java/de/szut/casino/blackjack/BlackJackService.java @@ -1,314 +1,245 @@ -package de.szut.casino.blackjack; - -import de.szut.casino.user.UserEntity; -import de.szut.casino.user.UserRepository; -import org.springframework.stereotype.Service; -import org.springframework.transaction.annotation.Transactional; - -import java.math.BigDecimal; -import java.util.List; -import java.util.Optional; -import java.util.Random; - -@Service -public class BlackJackService { - private final BlackJackGameRepository blackJackGameRepository; - private final UserRepository userRepository; - private final Random random = new Random(); - - public BlackJackService(BlackJackGameRepository blackJackGameRepository, UserRepository userRepository) { - this.blackJackGameRepository = blackJackGameRepository; - this.userRepository = userRepository; - } - - public BlackJackGameEntity getBlackJackGame(Long id) { - return blackJackGameRepository.findById(id).orElse(null); - } - - @Transactional - public BlackJackGameEntity createBlackJackGame(UserEntity user, BigDecimal betAmount) { - BlackJackGameEntity game = new BlackJackGameEntity(); - game.setUser(user); - game.setBet(betAmount); - - initializeDeck(game); - dealInitialCards(game); - - game.setState(getState(game)); - deductBetFromBalance(user, betAmount); - - return blackJackGameRepository.save(game); - } - - @Transactional - public BlackJackGameEntity hit(BlackJackGameEntity game) { - if (game.getState() != BlackJackState.IN_PROGRESS) { - return game; - } - - dealCardToPlayer(game); - - updateGameStateAndBalance(game); - - return blackJackGameRepository.save(game); - } - - @Transactional - public BlackJackGameEntity stand(BlackJackGameEntity game) { - if (game.getState() != BlackJackState.IN_PROGRESS) { - return game; - } - - dealCardsToDealerUntilMinimumScore(game); - determineWinnerAndUpdateBalance(game); - - return blackJackGameRepository.save(game); - } - - @Transactional - public BlackJackGameEntity doubleDown(BlackJackGameEntity game) { - if (game.getState() != BlackJackState.IN_PROGRESS || game.getPlayerCards().size() != 2) { - return game; - } - - UserEntity user = getUserWithFreshData(game.getUser()); - BigDecimal additionalBet = game.getBet(); - - if (user.getBalance().compareTo(additionalBet) < 0) { - return game; - } - - deductBetFromBalance(user, additionalBet); - game.setBet(game.getBet().add(additionalBet)); - - dealCardToPlayer(game); - updateGameStateAndBalance(game); - - if (game.getState() == BlackJackState.IN_PROGRESS) { - return stand(game); - } - - return game; - } - - @Transactional - public BlackJackGameEntity split(BlackJackGameEntity game) { - if (game.getState() != BlackJackState.IN_PROGRESS || - game.getPlayerCards().size() != 2 || - game.isSplit() || - !game.getPlayerCards().get(0).getRank().equals(game.getPlayerCards().get(1).getRank())) { - return game; - } - - UserEntity user = getUserWithFreshData(game.getUser()); - BigDecimal splitBet = game.getBet(); - - if (user.getBalance().compareTo(splitBet) < 0) { - return game; - } - - deductBetFromBalance(user, splitBet); - game.setSplitBet(splitBet); - game.setSplit(true); - - CardEntity card = game.getPlayerCards().remove(1); - card.setCardType(CardType.PLAYER_SPLIT); - game.getPlayerSplitCards().add(card); - - dealCardToPlayer(game); - dealCardToSplitHand(game); - - return blackJackGameRepository.save(game); - } - - private BlackJackGameEntity refreshGameState(BlackJackGameEntity game) { - return blackJackGameRepository.findById(game.getId()).orElse(game); - } - - private UserEntity getUserWithFreshData(UserEntity user) { - return userRepository.findById(user.getId()).orElse(user); - } - - private void dealInitialCards(BlackJackGameEntity game) { - for (int i = 0; i < 2; i++) { - dealCardToPlayer(game); - } - - dealCardToDealer(game); - } - - private void dealCardToPlayer(BlackJackGameEntity game) { - CardEntity card = drawCardFromDeck(game); - card.setCardType(CardType.PLAYER); - game.getPlayerCards().add(card); - } - - private void dealCardToDealer(BlackJackGameEntity game) { - CardEntity card = drawCardFromDeck(game); - card.setCardType(CardType.DEALER); - game.getDealerCards().add(card); - } - - private void dealCardsToDealerUntilMinimumScore(BlackJackGameEntity game) { - while (calculateHandValue(game.getDealerCards()) < 17) { - dealCardToDealer(game); - } - } - - private void dealCardToSplitHand(BlackJackGameEntity game) { - CardEntity card = drawCardFromDeck(game); - card.setCardType(CardType.PLAYER_SPLIT); - game.getPlayerSplitCards().add(card); - } - - private void updateGameStateAndBalance(BlackJackGameEntity game) { - if (game.isSplit()) { - int mainHandValue = calculateHandValue(game.getPlayerCards()); - int splitHandValue = calculateHandValue(game.getPlayerSplitCards()); - - if (mainHandValue > 21 && splitHandValue > 21) { - game.setState(BlackJackState.PLAYER_LOST); - updateUserBalance(game, false); - } else if (mainHandValue <= 21 && splitHandValue <= 21) { - game.setState(BlackJackState.IN_PROGRESS); - } else { - game.setState(BlackJackState.IN_PROGRESS); - } - } else { - game.setState(getState(game)); - - if (game.getState() == BlackJackState.PLAYER_WON) { - updateUserBalance(game, true); - } else if (game.getState() == BlackJackState.PLAYER_LOST) { - updateUserBalance(game, false); - } - } - } - - private void determineWinnerAndUpdateBalance(BlackJackGameEntity game) { - int playerValue = calculateHandValue(game.getPlayerCards()); - int dealerValue = calculateHandValue(game.getDealerCards()); - - if (dealerValue > 21 || playerValue > dealerValue) { - game.setState(BlackJackState.PLAYER_WON); - updateUserBalance(game, true); - } else if (playerValue < dealerValue) { - game.setState(BlackJackState.PLAYER_LOST); - updateUserBalance(game, false); - } else { - game.setState(BlackJackState.DRAW); - updateUserBalance(game, false); - } - } - - private void deductBetFromBalance(UserEntity user, BigDecimal betAmount) { - user.setBalance(user.getBalance().subtract(betAmount)); - userRepository.save(user); - } - - @Transactional - private void updateUserBalance(BlackJackGameEntity game, boolean isWin) { - UserEntity user = getUserWithFreshData(game.getUser()); - BigDecimal totalBet = game.getBet(); - BigDecimal balance = user.getBalance(); - - if (game.isSplit()) { - totalBet = totalBet.add(game.getSplitBet()); - - if (isWin) { - int mainHandValue = calculateHandValue(game.getPlayerCards()); - int splitHandValue = calculateHandValue(game.getPlayerSplitCards()); - int dealerValue = calculateHandValue(game.getDealerCards()); - - if (mainHandValue <= 21 && (dealerValue > 21 || mainHandValue > dealerValue)) { - balance = balance.add(game.getBet().multiply(BigDecimal.valueOf(2))); - } else if (mainHandValue == dealerValue) { - balance = balance.add(game.getBet()); - } - - if (splitHandValue <= 21 && (dealerValue > 21 || splitHandValue > dealerValue)) { - balance = balance.add(game.getSplitBet().multiply(BigDecimal.valueOf(2))); - } else if (splitHandValue == dealerValue) { - balance = balance.add(game.getSplitBet()); - } - } else if (game.getState() == BlackJackState.DRAW) { - balance = balance.add(totalBet); - } - } else { - if (isWin) { - balance = balance.add(totalBet.multiply(BigDecimal.valueOf(2))); - } else if (game.getState() == BlackJackState.DRAW) { - balance = balance.add(totalBet); - } - } - - user.setBalance(balance); - userRepository.save(user); - } - - private void initializeDeck(BlackJackGameEntity game) { - for (Suit suit : Suit.values()) { - for (Rank rank : Rank.values()) { - CardEntity card = new CardEntity(); - card.setGame(game); - card.setSuit(suit); - card.setRank(rank); - card.setCardType(CardType.DECK); - game.getDeck().add(card); - } - } - - java.util.Collections.shuffle(game.getDeck(), random); - } - - private CardEntity drawCardFromDeck(BlackJackGameEntity game) { - if (game.getDeck().isEmpty()) { - throw new IllegalStateException("Deck is empty"); - } - - return game.getDeck().removeFirst(); - } - - private BlackJackState getState(BlackJackGameEntity game) { - int playerHandValue = calculateHandValue(game.getPlayerCards()); - - if (playerHandValue == 21) { - CardEntity hole = drawCardFromDeck(game); - hole.setCardType(CardType.DEALER); - game.getDealerCards().add(hole); - - int dealerHandValue = calculateHandValue(game.getDealerCards()); - - if (dealerHandValue == 21) { - return BlackJackState.DRAW; - } else { - BigDecimal blackjackWinnings = game.getBet().multiply(new BigDecimal("1.5")); - UserEntity user = getUserWithFreshData(game.getUser()); - user.setBalance(user.getBalance().add(blackjackWinnings)); - return BlackJackState.PLAYER_BLACKJACK; - } - } else if (playerHandValue > 21) { - return BlackJackState.PLAYER_LOST; - } - - return BlackJackState.IN_PROGRESS; - } - - private int calculateHandValue(List hand) { - int sum = 0; - int aceCount = 0; - for (CardEntity card : hand) { - sum += card.getRank().getValue(); - if (card.getRank() == Rank.ACE) { - aceCount++; - } - } - - while (sum > 21 && aceCount > 0) { - sum -= 10; - aceCount--; - } - - return sum; - } -} +package de.szut.casino.blackjack; + +import de.szut.casino.user.UserEntity; +import de.szut.casino.user.UserRepository; +import org.springframework.stereotype.Service; +import org.springframework.transaction.annotation.Transactional; + +import java.math.BigDecimal; +import java.util.List; +import java.util.Optional; +import java.util.Random; + +@Service +public class BlackJackService { + private final BlackJackGameRepository blackJackGameRepository; + private final UserRepository userRepository; + private final Random random = new Random(); + + public BlackJackService(BlackJackGameRepository blackJackGameRepository, UserRepository userRepository) { + this.blackJackGameRepository = blackJackGameRepository; + this.userRepository = userRepository; + } + + public BlackJackGameEntity getBlackJackGame(Long id) { + return blackJackGameRepository.findById(id).orElse(null); + } + + @Transactional + public BlackJackGameEntity createBlackJackGame(UserEntity user, BigDecimal betAmount) { + BlackJackGameEntity game = new BlackJackGameEntity(); + game.setUser(user); + game.setBet(betAmount); + + initializeDeck(game); + dealInitialCards(game); + + game.setState(getState(game)); + deductBetFromBalance(user, betAmount); + + return blackJackGameRepository.save(game); + } + + @Transactional + public BlackJackGameEntity hit(BlackJackGameEntity game) { + game = refreshGameState(game); + + if (game.getState() != BlackJackState.IN_PROGRESS) { + return game; + } + + dealCardToPlayer(game); + updateGameStateAndBalance(game); + + return blackJackGameRepository.save(game); + } + + @Transactional + public BlackJackGameEntity stand(BlackJackGameEntity game) { + game = refreshGameState(game); + + if (game.getState() != BlackJackState.IN_PROGRESS) { + return game; + } + + dealCardsToDealerUntilMinimumScore(game); + determineWinnerAndUpdateBalance(game); + + return blackJackGameRepository.save(game); + } + + @Transactional + public BlackJackGameEntity doubleDown(BlackJackGameEntity game) { + game = refreshGameState(game); + + if (game.getState() != BlackJackState.IN_PROGRESS || game.getPlayerCards().size() != 2) { + return game; + } + + UserEntity user = getUserWithFreshData(game.getUser()); + BigDecimal additionalBet = game.getBet(); + + if (user.getBalance().compareTo(additionalBet) < 0) { + return game; + } + + deductBetFromBalance(user, additionalBet); + game.setBet(game.getBet().add(additionalBet)); + + dealCardToPlayer(game); + updateGameStateAndBalance(game); + + if (game.getState() == BlackJackState.IN_PROGRESS) { + return stand(game); + } + + return game; + } + + private BlackJackGameEntity refreshGameState(BlackJackGameEntity game) { + return blackJackGameRepository.findById(game.getId()).orElse(game); + } + + private UserEntity getUserWithFreshData(UserEntity user) { + return userRepository.findById(user.getId()).orElse(user); + } + + private void dealInitialCards(BlackJackGameEntity game) { + for (int i = 0; i < 2; i++) { + dealCardToPlayer(game); + } + + dealCardToDealer(game); + } + + private void dealCardToPlayer(BlackJackGameEntity game) { + CardEntity card = drawCardFromDeck(game); + card.setCardType(CardType.PLAYER); + game.getPlayerCards().add(card); + } + + private void dealCardToDealer(BlackJackGameEntity game) { + CardEntity card = drawCardFromDeck(game); + card.setCardType(CardType.DEALER); + game.getDealerCards().add(card); + } + + private void dealCardsToDealerUntilMinimumScore(BlackJackGameEntity game) { + while (calculateHandValue(game.getDealerCards()) < 17) { + dealCardToDealer(game); + } + } + + private void updateGameStateAndBalance(BlackJackGameEntity game) { + game.setState(getState(game)); + + if (game.getState() == BlackJackState.PLAYER_WON) { + updateUserBalance(game, true); + } else if (game.getState() == BlackJackState.PLAYER_LOST) { + updateUserBalance(game, false); + } + } + + private void determineWinnerAndUpdateBalance(BlackJackGameEntity game) { + int playerValue = calculateHandValue(game.getPlayerCards()); + int dealerValue = calculateHandValue(game.getDealerCards()); + + if (dealerValue > 21 || playerValue > dealerValue) { + game.setState(BlackJackState.PLAYER_WON); + updateUserBalance(game, true); + } else if (playerValue < dealerValue) { + game.setState(BlackJackState.PLAYER_LOST); + updateUserBalance(game, false); + } else { + game.setState(BlackJackState.DRAW); + updateUserBalance(game, false); // For draw, player gets their bet back + } + } + + private void deductBetFromBalance(UserEntity user, BigDecimal betAmount) { + user.setBalance(user.getBalance().subtract(betAmount)); + userRepository.save(user); + } + + @Transactional + private void updateUserBalance(BlackJackGameEntity game, boolean isWin) { + UserEntity user = getUserWithFreshData(game.getUser()); + BigDecimal betAmount = game.getBet(); + BigDecimal balance = user.getBalance(); + + if (isWin) { + balance = balance.add(betAmount.multiply(BigDecimal.valueOf(2))); + } else if (game.getState() == BlackJackState.DRAW) { + balance = balance.add(betAmount); + } + + user.setBalance(balance); + userRepository.save(user); + } + + private void initializeDeck(BlackJackGameEntity game) { + for (Suit suit : Suit.values()) { + for (Rank rank : Rank.values()) { + CardEntity card = new CardEntity(); + card.setGame(game); + card.setSuit(suit); + card.setRank(rank); + card.setCardType(CardType.DECK); + game.getDeck().add(card); + } + } + + java.util.Collections.shuffle(game.getDeck(), random); + } + + private CardEntity drawCardFromDeck(BlackJackGameEntity game) { + if (game.getDeck().isEmpty()) { + throw new IllegalStateException("Deck is empty"); + } + + return game.getDeck().removeFirst(); + } + + private BlackJackState getState(BlackJackGameEntity game) { + int playerHandValue = calculateHandValue(game.getPlayerCards()); + + if (playerHandValue == 21) { + CardEntity hole = drawCardFromDeck(game); + hole.setCardType(CardType.DEALER); + game.getDealerCards().add(hole); + + int dealerHandValue = calculateHandValue(game.getDealerCards()); + + if (dealerHandValue == 21) { + return BlackJackState.DRAW; + } else { + BigDecimal blackjackWinnings = game.getBet().multiply(new BigDecimal("1.5")); + UserEntity user = getUserWithFreshData(game.getUser()); + user.setBalance(user.getBalance().add(blackjackWinnings)); + return BlackJackState.PLAYER_BLACKJACK; + } + } else if (playerHandValue > 21) { + return BlackJackState.PLAYER_LOST; + } + + return BlackJackState.IN_PROGRESS; + } + + private int calculateHandValue(List hand) { + int sum = 0; + int aceCount = 0; + for (CardEntity card : hand) { + sum += card.getRank().getValue(); + if (card.getRank() == Rank.ACE) { + aceCount++; + } + } + + while (sum > 21 && aceCount > 0) { + sum -= 10; + aceCount--; + } + + return sum; + } +} diff --git a/backend/src/main/java/de/szut/casino/blackjack/CardEntity.java b/backend/src/main/java/de/szut/casino/blackjack/CardEntity.java index e2c1f0e..3b6903a 100644 --- a/backend/src/main/java/de/szut/casino/blackjack/CardEntity.java +++ b/backend/src/main/java/de/szut/casino/blackjack/CardEntity.java @@ -36,5 +36,5 @@ public class CardEntity { } enum CardType { - DECK, PLAYER, DEALER, PLAYER_SPLIT + DECK, PLAYER, DEALER } diff --git a/frontend/src/app/feature/game/blackjack/blackjack.component.ts b/frontend/src/app/feature/game/blackjack/blackjack.component.ts index 78f3533..d6bfec4 100644 --- a/frontend/src/app/feature/game/blackjack/blackjack.component.ts +++ b/frontend/src/app/feature/game/blackjack/blackjack.component.ts @@ -1,4 +1,4 @@ -import { ChangeDetectionStrategy, Component, inject, signal, OnInit } from '@angular/core'; +import { ChangeDetectionStrategy, Component, inject, signal } from '@angular/core'; import { CommonModule } from '@angular/common'; import { Router } from '@angular/router'; import { PlayingCardComponent } from './components/playing-card/playing-card.component'; @@ -13,7 +13,6 @@ import { GameResultComponent } from '@blackjack/components/game-result/game-resu import { GameState } from '@blackjack/enum/gameState'; import { NavbarComponent } from '@shared/components/navbar/navbar.component'; import { UserService } from '@service/user.service'; -import { timer } from 'rxjs'; @Component({ selector: 'app-blackjack', @@ -31,7 +30,7 @@ import { timer } from 'rxjs'; templateUrl: './blackjack.component.html', changeDetection: ChangeDetectionStrategy.OnPush, }) -export default class BlackjackComponent implements OnInit { +export default class BlackjackComponent { private router = inject(Router); private userService = inject(UserService); private blackjackService = inject(BlackjackService); @@ -48,11 +47,13 @@ export default class BlackjackComponent implements OnInit { isActionInProgress = signal(false); currentAction = signal(''); - ngOnInit(): void { - this.userService.currentUser$.subscribe((user) => { - if (user) { - this.balance.set(user.balance); - } + constructor() { + this.refreshUserBalance(); + } + + private refreshUserBalance(): void { + this.userService.getCurrentUser().subscribe((user) => { + this.balance.set(user?.balance ?? 0); }); } @@ -82,10 +83,8 @@ export default class BlackjackComponent implements OnInit { if (isGameOver) { console.log('Game is over, state:', game.state); this.userService.refreshCurrentUser(); - timer(1500).subscribe(() => { - this.showGameResult.set(true); - console.log('Game result dialog shown after delay'); - }); + this.showGameResult.set(true); + console.log('Game result dialog should be shown now'); } } @@ -181,18 +180,21 @@ export default class BlackjackComponent implements OnInit { onCloseGameResult(): void { console.log('Closing game result dialog'); this.showGameResult.set(false); - this.userService.refreshCurrentUser(); } private handleGameError(error: HttpErrorResponse): void { if (error instanceof HttpErrorResponse) { if (error.status === 400 && error.error?.error === 'Invalid state') { this.gameInProgress.set(false); - this.userService.refreshCurrentUser(); + + this.refreshUserBalance(); } else if (error.status === 500) { console.log('Server error occurred. The game may have been updated in another session.'); + this.gameInProgress.set(false); - this.userService.refreshCurrentUser(); + + this.refreshUserBalance(); + if (this.currentGameId()) { this.refreshGameState(this.currentGameId()!); } diff --git a/frontend/src/app/feature/game/blackjack/components/game-info/game-info.component.ts b/frontend/src/app/feature/game/blackjack/components/game-info/game-info.component.ts index df29e81..fea5453 100644 --- a/frontend/src/app/feature/game/blackjack/components/game-info/game-info.component.ts +++ b/frontend/src/app/feature/game/blackjack/components/game-info/game-info.component.ts @@ -6,11 +6,9 @@ import { OnChanges, Output, SimpleChanges, - signal, } from '@angular/core'; import { CommonModule, CurrencyPipe } from '@angular/common'; -import { FormGroup, ReactiveFormsModule } from '@angular/forms'; -import { BettingService } from '@blackjack/services/betting.service'; +import { FormBuilder, FormGroup, ReactiveFormsModule, Validators } from '@angular/forms'; @Component({ selector: 'app-game-info', @@ -67,12 +65,10 @@ import { BettingService } from '@blackjack/services/betting.service'; type="number" id="bet" formControlName="bet" - class="w-full px-3 py-2 bg-deep-blue-light text-white rounded focus:outline-none focus:ring-2 focus:ring-emerald disabled:opacity-50" + class="w-full px-3 py-2 bg-deep-blue-light text-white rounded focus:outline-none focus:ring-2 focus:ring-emerald" [min]="1" [max]="balance" step="0.01" - [disabled]="gameInProgress || isActionInProgress" - [placeholder]="balance | currency: 'EUR'" /> @if (betForm.get('bet')?.errors?.['required'] && betForm.get('bet')?.touched) { Bitte geben Sie einen Einsatz ein @@ -105,14 +101,7 @@ import { BettingService } from '@blackjack/services/betting.service'; changeDetection: ChangeDetectionStrategy.OnPush, }) export class GameInfoComponent implements OnChanges { - @Input() set balance(value: number) { - this._balance.set(value); - } - get balance() { - return this._balance(); - } - private _balance = signal(0); - + @Input() balance = 0; @Input() currentBet = 0; @Input() gameInProgress = false; @Input() isActionInProgress = false; @@ -120,19 +109,24 @@ export class GameInfoComponent implements OnChanges { betForm: FormGroup; - constructor(private bettingService: BettingService) { - this.betForm = this.bettingService.createBetForm(); + constructor(private fb: FormBuilder) { + this.betForm = this.fb.group({ + bet: ['', [Validators.required, Validators.min(1)]], + }); } ngOnChanges(changes: SimpleChanges): void { if (changes['balance']) { - this.bettingService.updateBetFormValidators(this.betForm, this.balance); + this.betForm + .get('bet') + ?.setValidators([Validators.required, Validators.min(1), Validators.max(this.balance)]); + this.betForm.get('bet')?.updateValueAndValidity(); } } setBetAmount(percentage: number) { - const betAmount = this.bettingService.calculateBetAmount(this.balance, percentage); - if (this.bettingService.isValidBet(betAmount, this.balance)) { + const betAmount = Math.floor(this.balance * percentage * 100) / 100; + if (betAmount >= 1) { this.betForm.patchValue({ bet: betAmount }); } } @@ -140,7 +134,7 @@ export class GameInfoComponent implements OnChanges { onSubmit() { if (this.betForm.valid) { const betAmount = parseFloat(this.betForm.value.bet); - if (this.bettingService.isValidBet(betAmount, this.balance)) { + if (betAmount <= this.balance) { this.newGame.emit(betAmount); this.betForm.reset(); } diff --git a/frontend/src/app/feature/game/blackjack/services/betting.service.ts b/frontend/src/app/feature/game/blackjack/services/betting.service.ts deleted file mode 100644 index 347f19b..0000000 --- a/frontend/src/app/feature/game/blackjack/services/betting.service.ts +++ /dev/null @@ -1,31 +0,0 @@ -import { Injectable } from '@angular/core'; -import { FormBuilder, FormGroup, Validators } from '@angular/forms'; - -@Injectable({ - providedIn: 'root', -}) -export class BettingService { - constructor(private fb: FormBuilder) {} - - createBetForm(): FormGroup { - return this.fb.group({ - bet: ['', [Validators.required, Validators.min(1)]], - }); - } - - updateBetFormValidators(form: FormGroup, balance: number): void { - form.reset(); - form - .get('bet') - ?.setValidators([Validators.required, Validators.min(1), Validators.max(balance)]); - form.get('bet')?.updateValueAndValidity(); - } - - calculateBetAmount(balance: number, percentage: number): number { - return Math.floor(balance * percentage * 100) / 100; - } - - isValidBet(betAmount: number, balance: number): boolean { - return betAmount >= 1 && betAmount <= balance; - } -}