Update Purchase class with exceptions

solvena · May 24, 2026 · 428fde3 · 428fde3
1 parent 3bb7de3
commit 428fde3
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/src/main/java/Model/Purchase.java b/src/main/java/Model/Purchase.java
@@ -8,10 +8,19 @@ public Purchase(Share share, int week) {
 
     @Override
     public void commit(Player player) {
+        if (player == null) {
+            throw new IllegalArgumentException("Player cannot be null");
+        }
+        if (isCommitted()) {
+            throw new IllegalStateException("Purchase has already been committed");
+        }
+
         BigDecimal price = this.getCalculator().calculateTotal();
 
-        if (isCommitted() || (player.getMoney().compareTo(price) < 0)) {
-            return;
+        if (player.getMoney().compareTo(price) < 0) {
+            throw new IllegalStateException(
+                "Insufficient funds: required " + price + ", available " + player.getMoney()
+            );
         }
 
         player.withdrawMoney(price);