Feat: Added validation enum and validation for multiple classes

src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/engine/Exchange.java

@@ -1,14 +1,15 @@
 package edu.ntnu.idi.idatt2003.g40.mappe.engine;
 
+import edu.ntnu.idi.idatt2003.g40.mappe.model.Player;
 import edu.ntnu.idi.idatt2003.g40.mappe.model.Purchase;
 import edu.ntnu.idi.idatt2003.g40.mappe.model.Sale;
 import edu.ntnu.idi.idatt2003.g40.mappe.model.Share;
-import edu.ntnu.idi.idatt2003.g40.mappe.model.Player;
 import edu.ntnu.idi.idatt2003.g40.mappe.model.Stock;
 import edu.ntnu.idi.idatt2003.g40.mappe.model.Transaction;
 import edu.ntnu.idi.idatt2003.g40.mappe.service.PurchaseCalculator;
 import edu.ntnu.idi.idatt2003.g40.mappe.service.SaleCalculator;
 import edu.ntnu.idi.idatt2003.g40.mappe.service.TransactionCalculator;
+import edu.ntnu.idi.idatt2003.g40.mappe.utils.Validator;
 
 import java.math.BigDecimal;
 import java.util.ArrayList;
@@ -56,13 +57,17 @@ public final class Exchange {
    *
    * @param name name of exchange.
    * @param stocks list of {@link Stock} objects.
+   *
+   * @throws IllegalArgumentException if name or stocks are empty/null.
    * */
-  public Exchange(final String name, final List<Stock> stocks) {
+  public Exchange(final String name, final List<Stock> stocks) throws IllegalArgumentException {
+    if (!Validator.NOT_EMPTY.isValid(name) || stocks == null || stocks.isEmpty()) {
+      throw new IllegalArgumentException("Invalid exchange parameters!");
+    }
     this.name = name;
     this.week = 1;
     this.stockMap = new HashMap<>();
     this.random = new Random();
-
     for (Stock stock : stocks) {
       stockMap.put(stock.getSymbol(), stock);
     }
@@ -103,8 +108,14 @@ public boolean hasStock(final String symbol) {
    * @param symbol the symbol of the stock to get.
    *
    * @return {@link Stock} element gotten.
+   *
+   * @throws IllegalArgumentException if symbol is invalid.
    * */
-  public Stock getStock(final String symbol) {
+  public Stock getStock(final String symbol) throws IllegalArgumentException {
+    if (!Validator.VALID_STOCK_NAME.isValid(symbol)) {
+      throw new IllegalArgumentException(
+              Validator.VALID_STOCK_NAME.getErrorMessage());
+    }
     return stockMap.get(symbol);
   }
 
@@ -136,10 +147,15 @@ public List<Stock> findStocks(final String searchTerm) {
    * @param player the player buying stock.
    *
    * @return Transaction representing the transaction.
+   *
+   * @throws IllegalArgumentException if symbol or player is invalid.
    * */
   public Transaction buy(final String symbol,
                          final BigDecimal quantity,
-                         final Player player) {
+                         final Player player) throws IllegalArgumentException {
+    if (!Validator.VALID_STOCK_NAME.isValid(symbol) || player == null) {
+      throw new IllegalArgumentException("Invalid purchase!");
+    }
     Stock stock = stockMap.get(symbol);
     Share share = new Share(stock, quantity, stock.getSalesPrice());
     TransactionCalculator calculator = new PurchaseCalculator(share);
@@ -154,8 +170,14 @@ public Transaction buy(final String symbol,
    * @param player the player buying stock.
    *
    * @return Transaction representing the transaction.
+   *
+   * @throws IllegalArgumentException if share or player is null.
    * */
-  public Transaction sell(final Share share, final Player player) {
+  public Transaction sell(final Share share, final Player player)
+          throws IllegalArgumentException {
+    if (share == null || player == null) {
+      throw new IllegalArgumentException("Invalid sell!");
+    }
     TransactionCalculator calculator = new SaleCalculator(share);
     player.addMoney(calculator.calculateTotal());
     return new Sale(share, week, calculator);

src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/model/Player.java

@@ -1,8 +1,8 @@
 package edu.ntnu.idi.idatt2003.g40.mappe.model;
 
-import edu.ntnu.idi.idatt2003.g40.mappe.engine.Exchange;
+import edu.ntnu.idi.idatt2003.g40.mappe.controller.PlayerStatusController;
 import edu.ntnu.idi.idatt2003.g40.mappe.engine.TransactionArchive;
-
+import edu.ntnu.idi.idatt2003.g40.mappe.utils.Validator;
 import java.math.BigDecimal;
 
 /**
@@ -11,7 +11,8 @@
  * <p>Each player:</p>
  * <ul>
  *   <li>Has a portfolio</li>
- *   <li>Can buy and sell shares on an {@link Exchange}</li>
+ *   <li>Can buy and sell shares on an
+ *   {@link edu.ntnu.idi.idatt2003.g40.mappe.engine.Exchange}.</li>
  *   <li>Has a set amount of money to use on said exchange.</li>
  *   <li>Has a {@link TransactionArchive}</li>
  * </ul>
@@ -49,8 +50,13 @@ public final class Player {
    *
    * @param name          the name of the player
    * @param startingMoney the starting amount of money
+   *
+   * @throws IllegalArgumentException if name is null.
    */
-  public Player(final String name, final BigDecimal startingMoney) {
+  public Player(final String name, final BigDecimal startingMoney) throws IllegalArgumentException {
+    if (!Validator.NOT_EMPTY.isValid(name)) {
+      throw new IllegalArgumentException("Invalid name!");
+    }
     this.name = name;
     this.startingMoney = startingMoney;
     this.money = this.startingMoney;
@@ -132,4 +138,15 @@ public BigDecimal getNetWorth() {
     netWorth = netWorth.add(portfolio.getNetWorth()).add(money);
     return netWorth;
   }
+
+  /**
+   * Getter method for players' current status.
+   *
+   * <p>Leaves calculations for the {@link PlayerStatusController}</p>
+   *
+   * @return PlayerStatus.
+   * */
+  public PlayerStatus getStatus() {
+    return PlayerStatusController.getStatus(this);
+  }
 }

src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/model/Portfolio.java

@@ -1,6 +1,7 @@
 package edu.ntnu.idi.idatt2003.g40.mappe.model;
 
 import edu.ntnu.idi.idatt2003.g40.mappe.service.SaleCalculator;
+import edu.ntnu.idi.idatt2003.g40.mappe.utils.Validator;
 
 import java.math.BigDecimal;
 import java.util.ArrayList;
@@ -31,22 +32,33 @@ public Portfolio() {
    * Adds a share to the portfolio.
    *
    * @param share the share to add
+   *
    * @return {@code true} if the share was added, {@code false} otherwise
+   *
+   * @throws IllegalArgumentException if share is null.
    */
-  public boolean addShare(final Share share) {
-    Objects.requireNonNull(share, "share cannot be null");
+  public boolean addShare(final Share share) throws IllegalArgumentException {
+    if (share == null) {
+      throw new IllegalArgumentException("Invalid share!");
+    }
     return shares.add(share);
   }
 
   /**
    * Removes a share from the portfolio.
    *
    * @param share the share to remove
+   *
    * @return {@code true} if the share was removed,
    *         {@code false} if it was not present.
+   *
+   * @throws IllegalArgumentException if share is null.
+   *
    */
-  public boolean removeShare(final Share share) {
-    Objects.requireNonNull(share, "share cannot be null");
+  public boolean removeShare(final Share share) throws IllegalArgumentException {
+    if (share == null) {
+      throw new IllegalArgumentException("Invalid share!");
+    }
     return shares.remove(share);
   }
 
@@ -65,9 +77,14 @@ public List<Share> getShares() {
    *
    * @param symbol the stock symbol to match
    * @return a list of shares matching the symbol
+   *
+   * @throws IllegalArgumentException if symbol is invalid.
    */
-  public List<Share> getShares(final String symbol) {
-    Objects.requireNonNull(symbol, "symbol cannot be null");
+  public List<Share> getShares(final String symbol) throws IllegalArgumentException {
+    if (!Validator.VALID_STOCK_NAME.isValid(symbol)) {
+      throw new IllegalArgumentException(
+              Validator.VALID_STOCK_NAME.getErrorMessage());
+    }
     return shares.stream()
             .filter(s -> symbol.equalsIgnoreCase(s.getStock().getSymbol()))
             .toList();
@@ -77,11 +94,16 @@ public List<Share> getShares(final String symbol) {
    * Checks whether the given share exists in the portfolio.
    *
    * @param share the share to check
+   *
    * @return {@code true} if the portfolio
    *          contains the share, otherwise {@code false}
+   *
+   * @throws IllegalArgumentException if share is null.
    */
-  public boolean contains(final Share share) {
-    Objects.requireNonNull(share, "share cannot be null");
+  public boolean contains(final Share share) throws IllegalArgumentException {
+    if (share == null) {
+      throw new IllegalArgumentException("Invalid share!");
+    }
     return shares.contains(share);
   }
 

src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/model/Share.java

@@ -31,10 +31,15 @@ public final class Share {
    * @param stock         the stock that was purchased
    * @param quantity      the quantity purchased
    * @param purchasePrice the price per unit at purchase time
+   *
+   * @throws IllegalArgumentException if stock is null.
    */
   public Share(final Stock stock,
                final BigDecimal quantity,
-               final BigDecimal purchasePrice) {
+               final BigDecimal purchasePrice) throws IllegalArgumentException {
+    if (stock == null) {
+      throw new IllegalArgumentException("Invalid stock!");
+    }
     this.stock = stock;
     this.quantity = quantity;
     this.purchasePrice = purchasePrice;

src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/model/Stock.java

@@ -1,5 +1,6 @@
 package edu.ntnu.idi.idatt2003.g40.mappe.model;
 
+import edu.ntnu.idi.idatt2003.g40.mappe.utils.Validator;
 import java.math.BigDecimal;
 import java.util.ArrayList;
 import java.util.Comparator;
@@ -34,23 +35,23 @@ public final class Stock {
    */
   public Stock(final String symbol,
                final String company,
-               final BigDecimal salesPrice) {
-
-    if (symbol.length() != 4) {
+               final BigDecimal salesPrice) throws IllegalArgumentException {
+    if (!Validator.VALID_STOCK_NAME.isValid(symbol)) {
       throw new IllegalArgumentException(
-              "Stock's symbol must be 4 characters!");
+              Validator.VALID_STOCK_NAME.getErrorMessage());
+    } else {
+      this.symbol = symbol;
+      this.company = company;
+      prices.add(salesPrice);
     }
-    this.symbol = symbol;
-    this.company = company;
-    prices.add(salesPrice);
   }
 
   /**
    * Returns the stock symbol.
    *
    * @return the stock symbol
    */
- public String getSymbol() {
+  public String getSymbol() {
     return symbol;
   }
 
@@ -76,9 +77,16 @@ public BigDecimal getSalesPrice() {
    * Adds a new sales price to the price history.
    *
    * @param price the new sales price
+   *
+   * @throws IllegalArgumentException if price is 0 or null.
    */
-  public void addNewSalesPrice(final BigDecimal price) {
-    prices.add(price);
+  public void addNewSalesPrice(final BigDecimal price)
+          throws IllegalArgumentException {
+    if (price != null && price.intValue() != 0) {
+      prices.add(price);
+    } else {
+      throw new IllegalArgumentException("Invalid price to add to stock: " + getSymbol());
+    }
   }
 
   /**

src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/model/Transaction.java

@@ -34,13 +34,20 @@ public abstract class Transaction {
    * @param share      the share to perform the transaction on
    * @param week       the current week to perform the transaction under
    * @param calculator the {@link TransactionCalculator} to use.
+   *
+   * @throws IllegalArgumentException if any parameter is null.
    */
 
   protected Transaction(final Share share, final int week,
-                        final TransactionCalculator calculator) {
-    this.share = share;
-    this.week = week;
-    this.calculator = calculator;
+                        final TransactionCalculator calculator)
+          throws IllegalArgumentException{
+    if (share == null || calculator == null) {
+      throw new IllegalArgumentException("Invalid stock or calculator!");
+    } else {
+      this.share = share;
+      this.week = week;
+      this.calculator = calculator;
+    }
   }
 
   /**