diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/Main.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/Main.java index 0d9178d..16a794f 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/Main.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/Main.java @@ -7,7 +7,7 @@ public class Main { public static void main(String[] args) throws Exception { Banksystem bs = new Banksystem("Spaßkasse Mannheim"); - UI ui = new UI(bs); // Test-Push + UI ui = new UI(bs); } } \ No newline at end of file diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java index 715adee..d8f12a9 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java @@ -3,15 +3,14 @@ package de.hs_mannheim.informatik.bank.domain; import java.io.Serializable; public class Girokonto extends Konto implements Serializable { - - private long dispo = 0; + private long dispo = 100000; public Girokonto(String inhaber, int kontozähler) { super(inhaber, kontozähler); } public boolean überweise(Girokonto ziel, long betrag, String zweck) { - if (super.getKontostand() - betrag >= dispo) { + if (super.getKontostand() - betrag >= dispo * (-1)) { this.auszahlen(betrag, zweck, "Überweisungsausgang", super.getInhaber()); ziel.einzahlen(betrag, zweck, "Überweisungseingang", super.getInhaber()); @@ -21,12 +20,17 @@ public class Girokonto extends Konto implements Serializable { return false; } - public void setDispo(long neuerDispo) { - dispo = neuerDispo * -1; - } - - public long getDispo() { - return dispo; + @Override + public boolean auszahlen(long betrag, String zweck, String art, String auftraggeber) { + if (stand - betrag >= dispo * (-1)) { + stand -= betrag; + + kontobewegungen.add(new Kontobewegung(betrag * -1, zweck, art, auftraggeber)); + + return true; + } + + return false; } @Override diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java index fc0e31c..5490892 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java @@ -1,19 +1,27 @@ package de.hs_mannheim.informatik.bank.domain; -import static org.junit.jupiter.api.Assertions.*; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertTrue; import org.junit.jupiter.api.Test; class GirokontoTest { @Test - void testDispo() { - Girokonto konto = new Girokonto("Max Mustermann", 0); - konto.setDispo(1000); - konto.auszahlen(1001, "Test", "Test", konto.getInhaber()); - assertEquals(0, konto.getKontostand()); - konto.auszahlen(1000, "Test", "Test", konto.getInhaber()); - assertEquals(-1000, konto.getKontostand()); + void testÜberziehung() { + Konto k = new Girokonto("Müller", 0); + k.einzahlen(10000, "Test", "Einzahlung", "JUnit"); + + assertTrue(k.auszahlen(20000, "Test", "Einzahlung", "JUnit")); + assertEquals(-10000, k.getKontostand()); + + assertTrue(k.auszahlen(40000, "Test", "Einzahlung", "JUnit")); + assertTrue(k.auszahlen(50000, "Test", "Einzahlung", "JUnit")); + assertEquals(-100000, k.getKontostand()); + + assertFalse(k.auszahlen(40000, "Test", "Einzahlung", "JUnit")); + assertFalse(k.auszahlen(1, "Test", "Einzahlung", "JUnit")); } } diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java index 5aed933..e787df5 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java @@ -5,10 +5,10 @@ import java.util.ArrayList; public class Konto implements Serializable { private int nummer; - private long stand = 0; + protected long stand = 0; private String inhaber; - private ArrayList kontobewegungen; + protected ArrayList kontobewegungen; public Konto(String inhaber, int kontozähler) { nummer = 1000 + kontozähler; @@ -41,10 +41,7 @@ public class Konto implements Serializable { } public boolean auszahlen(long betrag, String zweck, String art, String auftraggeber) { - long dispo = 0; - if (this instanceof Girokonto) - dispo = ((Girokonto) this).getDispo(); - if (stand - betrag >= dispo) { + if (stand - betrag >= 0) { stand -= betrag; kontobewegungen.add(new Kontobewegung(betrag * -1, zweck, art, auftraggeber)); @@ -65,12 +62,14 @@ public class Konto implements Serializable { return auflistung; } - - public long saldoBerechnen(int anzahlKontobew) { + + public long berechneSaldo(int anzahl) { long saldo = 0; - for (int i = 0; i < anzahlKontobew; i++) { - saldo += kontobewegungen.get(i).getBetrag(); + + for (int i = 0; i < anzahl; i++) { + saldo += kontobewegungen.get(i).getBetrag() ; } + return saldo; } diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/KontoTest.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/KontoTest.java index 1ed93b6..e20060a 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/KontoTest.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/KontoTest.java @@ -15,10 +15,17 @@ class KontoTest { } @Test - void testKontoEinUndAuszahlung() { + void testKontoEinUndAuszahlungUndSaldo() { Konto k = new Konto("Müller", 0); Konto k2 = new Konto("Mayer", 1); + testKontoEinUndAuszahlungUndSaldo(k, k2); + k = new Girokonto("Müller", 0); + k2 = new Girokonto("Mayer", 1); + testKontoEinUndAuszahlungUndSaldo(k, k2); + } + + private void testKontoEinUndAuszahlungUndSaldo(Konto k, Konto k2) { assertEquals("Mayer", k2.getInhaber()); assertNotEquals(k.getKontonummer(), k2.getKontonummer()); @@ -28,18 +35,26 @@ class KontoTest { assertTrue(k2.auszahlen(50, "Test", "Auszahlung", "JUnit")); assertEquals(50, k2.getKontostand()); - assertFalse(k2.auszahlen(500, "Test", "Auszahlung", "JUnit")); - assertEquals(50, k2.getKontostand()); + assertTrue(k2.auszahlen(50, "Test", "Auszahlung", "JUnit")); + assertEquals(0, k2.getKontostand()); + + k2.einzahlen(100, "Test", "Einzahlung", "JUnit"); + k2.einzahlen(100, "Test", "Einzahlung", "JUnit"); + k2.einzahlen(1, "Test", "Einzahlung", "JUnit"); + + assertEquals(100, k2.berechneSaldo(1)); + assertEquals(100, k2.berechneSaldo(4)); + assertEquals(k2.getKontostand(), k2.berechneSaldo(6)); } @Test - void testSaldoBerechnen() { - Konto k = new Konto("Max Mustermann", 0); - k.einzahlen(1000, null, "Einzahlung", null); - k.einzahlen(2000, null, "Einzahlung", null); - k.einzahlen(3000, null, "Einzahlung", null); - k.auszahlen(250, null, "Auszahlung", null); - assertEquals(1000+2000+3000-250, k.saldoBerechnen(4)); + void testKeineÜberziehungFürSparkonten() { + Konto k = new Konto("Müller", 0); + k.einzahlen(100, "Test", "Einzahlung", "JUnit"); + assertFalse(k.auszahlen(500, "Test", "Auszahlung", "JUnit")); + + k.auszahlen(50, "Test", "Auszahlung", "JUnit"); + assertFalse(k.auszahlen(100, "Test", "Auszahlung", "JUnit")); } } diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java index 0923e6f..22f1401 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java @@ -18,21 +18,15 @@ public class Kontobewegung implements Serializable { this.datum = new Date(); } + + public long getBetrag() { + return betrag; + } @Override public String toString() { return "Kontobewegung [betrag=" + betrag + ", datum=" + datum + ", betreff=" + betreff + ", art=" + art + ", auftraggeber=" + auftraggeber + "]"; } - - public long getBetrag() { - return betrag; - } - - public String getArt() { - return art; - } - - } diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java index 18f2f91..5ff5144 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java @@ -50,16 +50,14 @@ public class Banksystem { return konto.getKontostand(); } - public boolean geldAuszahlen(int kontonummer, long betrag) throws Exception { // Fehler es wird vor der eigentlichen Auszahlung gespeichert + public boolean geldAuszahlen(int kontonummer, long betrag) throws Exception { Konto konto = bank.findeKonto(kontonummer); - if (konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber())) { - Persistenz.speichereBankDaten(this.bank, bank.getName()); - return true; - } - else { - Persistenz.speichereBankDaten(this.bank, bank.getName()); - return false; - } + + boolean erg = konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber()); + + Persistenz.speichereBankDaten(this.bank, bank.getName()); + + return erg; } public String[] erstelleKontoauszug(int kontonummer) { @@ -68,12 +66,15 @@ public class Banksystem { return konto.getKontobewegungen(); } - public boolean überweisungBeauftragen(int startkonto, int zielkonto, long betrag, String verwendungszweck) { + public boolean überweisungBeauftragen(int startkonto, int zielkonto, long betrag, String verwendungszweck) throws Exception { Konto start = bank.findeKonto(startkonto); Konto ziel = bank.findeKonto(zielkonto); if (start instanceof Girokonto && ziel instanceof Girokonto) { - return ((Girokonto)start).überweise((Girokonto)ziel, betrag, verwendungszweck); + boolean erfolg = ((Girokonto)start).überweise((Girokonto)ziel, betrag, verwendungszweck); + Persistenz.speichereBankDaten(this.bank, bank.getName()); + + return erfolg; } return false; @@ -84,9 +85,11 @@ public class Banksystem { return konto.getKontostand(); } - - public long saldoAnzeigen(int kontonummer, int anzahlKontobew) { - return bank.findeKonto(kontonummer).saldoBerechnen(anzahlKontobew); + + public long saldoBestimmen(int kontonummer, int anzahl) { + Konto konto = bank.findeKonto(kontonummer); + + return konto.berechneSaldo(anzahl); } } diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java index c450a26..b063dce 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java @@ -2,46 +2,69 @@ package de.hs_mannheim.informatik.bank.facade; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertTrue; import java.io.File; -import java.io.FileInputStream; -import java.io.ObjectInputStream; +import org.junit.jupiter.api.AfterAll; +import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.MethodOrderer.OrderAnnotation; +import org.junit.jupiter.api.Order; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.TestMethodOrder; -import de.hs_mannheim.informatik.bank.domain.Bank; -import de.hs_mannheim.informatik.bank.infrastructure.Persistenz; - +@TestMethodOrder(OrderAnnotation.class) class SystemTest { + private static Banksystem bs; + + @BeforeAll + static void initBanksystem() throws Exception { + bs = new Banksystem("Testsystem"); + } @Test - void smokeTest() throws Exception { - Banksystem bs = new Banksystem("Testsystem"); - + @Order(1) + void smokeTest() { assertNotNull(bs); assertEquals(0, bs.getKontenliste().length); assertEquals("Testsystem", bs.getBankname()); - new File("Testsystem-bank-data.ser").delete(); } @Test + @Order(2) void einzahlenTest() throws Exception { - new File("Testsystem-bank-data.ser").delete(); - Banksystem bs = new Banksystem("Testsystem"); - bs.kontoAnlegen("Max", 0); - bs.geldEinzahlen(1000, 250); - assertEquals(250, bs.getKontostand(1000)); - ObjectInputStream ois = new ObjectInputStream(new FileInputStream("Testsystem-bank-data.ser")); - Bank newBs = (Bank) ois.readObject(); - assertEquals(250, newBs.findeKonto(1000).getKontostand()); - bs.geldAuszahlen(1000, 250); - assertEquals(0, bs.getKontostand(1000)); - ObjectInputStream ois2 = new ObjectInputStream(new FileInputStream("Testsystem-bank-data.ser")); - newBs = (Bank) ois2.readObject(); - assertEquals(0, newBs.findeKonto(1000).getKontostand()); - ois.close(); - ois2.close(); - new File("Testsystem-bank-data.ser").delete(); + int knr = bs.kontoAnlegen("Test1", 1); + + assertEquals(1000, bs.geldEinzahlen(knr, 1000)); + + bs.geldEinzahlen(knr, 1); + assertEquals(1001, bs.getKontostand(knr)); + + assertEquals(1001, bs.geldEinzahlen(knr, 0)); + } + + @Test + @Order(3) + void persistenzTest() throws Exception { + int knr = bs.kontoAnlegen("Test2", 2); + int knr2 = bs.kontoAnlegen("Test3", 2); + + bs.geldEinzahlen(knr, 1000); + bs.geldAuszahlen(knr, 500); + assertTrue(bs.überweisungBeauftragen(knr, knr2, 100, "Überweisungstest.")); + + assertEquals(400, bs.getKontostand(knr)); + + bs = null; + + Banksystem bs2 = new Banksystem("Testsystem"); + assertEquals(400, bs2.getKontostand(knr)); + } + + @AfterAll + static void cleanup() { + File file = new File("/Users/oliver/git/Bank-System/Bank-Beispiel/Testsystem-bank-data.ser"); + file.delete(); } } diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java index 24ec413..3f10d79 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java @@ -27,7 +27,7 @@ public class UI { System.out.println("4 -> Geld auszahlen"); System.out.println("5 -> Kontoauszug drucken"); System.out.println("6 -> Überweisung beauftragen"); - System.out.println("7 -> Saldo anzeigen"); + System.out.println("7 -> Saldo abfragen"); System.out.println("9 -> Beenden"); System.out.println(); @@ -46,7 +46,7 @@ public class UI { case 4: geldAuszahlen(); break; case 5: kontoauszugDrucken(); break; case 6: überweisungBeauftragen(); break; - case 7: saldoAnzeigen(); break; + case 7: saldoAbfragen(); break; case 9: break mainloop; } @@ -116,8 +116,6 @@ public class UI { System.out.print("Bitte die gewünschte Kontonummer für den Auszug eingeben: "); int kontonummer = Integer.parseInt(sc.nextLine()); - System.out.println(); - // in echt auf einem Drucker System.out.println("Auszug für Konto " + kontonummer); String[] kontobewegungen = bs.erstelleKontoauszug(kontonummer); @@ -130,7 +128,7 @@ public class UI { System.out.println("Noch keine Kontobewegungen."); } - private void überweisungBeauftragen() { + private void überweisungBeauftragen() throws Exception { System.out.print("Bitte die Kontonummer des Ausgangskontos der Überweisung eingeben: "); int startkonto = Integer.parseInt(sc.nextLine()); @@ -148,16 +146,15 @@ public class UI { System.out.println("Überweisung" + ( (!erfolgreich) ? " nicht" : "") + " erfolgreich ausgeführt."); } - private void saldoAnzeigen() { - System.out.print("Bitte die Kontonummer des Kontos angeben, dessen Saldo berechnet werden soll: "); - int kontoNr = Integer.parseInt(sc.nextLine()); + private void saldoAbfragen() { + System.out.print("Bitte die Kontonummer des gewünschten Kontos eingeben: "); + int konto = Integer.parseInt(sc.nextLine()); + + System.out.print("Bitte die Anzahl der Kontobewegungen für den Saldo eingeben: "); + int anzahl = Integer.parseInt(sc.nextLine()); - System.out.print("Bitte die Anzahl an Kontobewegungen angeben: "); - int anzahlKontobew = Integer.parseInt(sc.nextLine()); - - long saldo = bs.saldoAnzeigen(kontoNr, anzahlKontobew); - - System.out.println("Berechneter Saldo: " + saldo); + long saldo = bs.saldoBestimmen(konto, anzahl); + System.out.printf("Der Saldo nach %d Kontobewegungen beträgt %.2f Euro.%n", anzahl, (saldo/100d)); } }