Fixed TransactionCalculatorFactory to actually follow factory design …

…pattern
IDATT2003-gruppe06 · Apr 17, 2026 · 77700e0 · 77700e0
1 parent 043369c
commit 77700e0
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 37 deletions.
diff --git a/src/main/java/millions/calculators/PurchaseCalculatorFactory.java b/src/main/java/millions/calculators/PurchaseCalculatorFactory.java
diff --git a/src/main/java/millions/calculators/SaleCalculatorFactory.java b/src/main/java/millions/calculators/SaleCalculatorFactory.java
diff --git a/src/main/java/millions/calculators/TransactionCalculatorFactory.java b/src/main/java/millions/calculators/TransactionCalculatorFactory.java
@@ -2,7 +2,15 @@
 
 import millions.Share;
 
-public abstract class TransactionCalculatorFactory {
-  public abstract void setShare(Share share);
-  public abstract TransactionCalculator createCalculator();
+public class TransactionCalculatorFactory {
+
+  private TransactionCalculatorFactory() {}
+
+  public TransactionCalculator createPurchaseCalculator(Share share) {
+    return new PurchaseCalculator(share);
+  }
+
+  public TransactionCalculator createSaleCalculator(Share share) {
+    return new SaleCalculator(share);
+  }
 }