From 852b7ae9d4048404abdebf28678c88d30313eec4 Mon Sep 17 00:00:00 2001 From: Oliver Hummel Date: Wed, 26 Oct 2022 17:07:15 +0200 Subject: [PATCH 1/4] =?UTF-8?q?Stand=20nach=20der=20=C3=9Cbungsbesprechung?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit (mit einigen weiteren Tests, aber ohne Gewähr!) --- .gitignore | 1 + Bank-Beispiel/.gitignore | 2 + .../informatik/bank/GirokontoTest.java | 30 +++++++++++ .../informatik/bank/domain/Girokonto.java | 16 +++++- .../informatik/bank/domain/Konto.java | 14 ++++- .../informatik/bank/domain/KontoTest.java | 31 +++++++++-- .../informatik/bank/domain/Kontobewegung.java | 4 ++ .../informatik/bank/facade/Banksystem.java | 12 ++++- .../informatik/bank/facade/SystemTest.java | 54 +++++++++++++++++-- .../de/hs_mannheim/informatik/bank/ui/UI.java | 15 +++++- 10 files changed, 165 insertions(+), 14 deletions(-) create mode 100644 .gitignore create mode 100644 Bank-Beispiel/src/de/hs_mannheim/informatik/bank/GirokontoTest.java diff --git a/.gitignore b/.gitignore new file mode 100644 index 0000000..9bb88d3 --- /dev/null +++ b/.gitignore @@ -0,0 +1 @@ +/.DS_Store diff --git a/Bank-Beispiel/.gitignore b/Bank-Beispiel/.gitignore index c8f10d9..c77e98d 100644 --- a/Bank-Beispiel/.gitignore +++ b/Bank-Beispiel/.gitignore @@ -181,3 +181,5 @@ replay_pid* /bank.ser /zähler.ser +/.DS_Store +/Spaßkasse Mannheim-bank-data.ser diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/GirokontoTest.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/GirokontoTest.java new file mode 100644 index 0000000..396edf1 --- /dev/null +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/GirokontoTest.java @@ -0,0 +1,30 @@ +package de.hs_mannheim.informatik.bank; + +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; + +import de.hs_mannheim.informatik.bank.domain.Girokonto; +import de.hs_mannheim.informatik.bank.domain.Konto; + +class GirokontoTest { + + @Test + 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/Girokonto.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java index d9b1d77..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,13 +3,14 @@ package de.hs_mannheim.informatik.bank.domain; import java.io.Serializable; public class Girokonto extends Konto implements Serializable { + 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 >= 0) { + if (super.getKontostand() - betrag >= dispo * (-1)) { this.auszahlen(betrag, zweck, "Überweisungsausgang", super.getInhaber()); ziel.einzahlen(betrag, zweck, "Überweisungseingang", super.getInhaber()); @@ -19,6 +20,19 @@ public class Girokonto extends Konto implements Serializable { return false; } + @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 public String toString() { return "Giro-" + super.toString(); 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 31220e1..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; @@ -63,4 +63,14 @@ public class Konto implements Serializable { return auflistung; } + public long berechneSaldo(int anzahl) { + long saldo = 0; + + 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 04e96ae..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,8 +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 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 d6b15dd..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,6 +18,10 @@ public class Kontobewegung implements Serializable { this.datum = new Date(); } + + public long getBetrag() { + return betrag; + } @Override public String toString() { 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 5d72301..6c0bfc2 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 @@ -53,9 +53,11 @@ public class Banksystem { public boolean geldAuszahlen(int kontonummer, long betrag) throws Exception { Konto konto = bank.findeKonto(kontonummer); + boolean erg = konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber()); + Persistenz.speichereBankDaten(this.bank, bank.getName()); - - return konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber()); + + return erg; } public String[] erstelleKontoauszug(int kontonummer) { @@ -81,4 +83,10 @@ public class Banksystem { return konto.getKontostand(); } + 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 0ce5674..90d65e0 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 @@ -3,17 +3,63 @@ package de.hs_mannheim.informatik.bank.facade; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; -import org.junit.jupiter.api.Test; +import java.io.File; +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; + +@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()); } + + @Test + @Order(2) + void einzahlenTest() throws Exception { + 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", 1); + bs.geldEinzahlen(knr, 1000); + + bs.geldAuszahlen(knr, 500); + + bs = null; + + Banksystem bs2 = new Banksystem("Testsystem"); + assertEquals(500, 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 001e321..c3bfe9b 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,6 +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 abfragen"); System.out.println("9 -> Beenden"); System.out.println(); @@ -45,6 +46,7 @@ public class UI { case 4: geldAuszahlen(); break; case 5: kontoauszugDrucken(); break; case 6: überweisungBeauftragen(); break; + case 7: saldoAbfragen(); break; case 9: break mainloop; } @@ -114,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); @@ -145,5 +145,16 @@ public class UI { System.out.println("Überweisung" + ( (!erfolgreich) ? " nicht" : "") + " erfolgreich ausgeführt."); } + + 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()); + + long saldo = bs.saldoBestimmen(konto, anzahl); + System.out.printf("Der Saldo nach %d Kontobewegungen beträgt %.2f Euro.%n", anzahl, (saldo/100d)); + } } From c7600e3325743e14fb94f88ef8b7bfdcd8e728cb Mon Sep 17 00:00:00 2001 From: Oliver Hummel Date: Wed, 26 Oct 2022 17:08:53 +0200 Subject: [PATCH 2/4] =?UTF-8?q?=E2=80=9EBank-Beispiel/.gitignore=E2=80=9C?= =?UTF-8?q?=20=C3=A4ndern?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Bank-Beispiel/.gitignore | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Bank-Beispiel/.gitignore b/Bank-Beispiel/.gitignore index c77e98d..d77bfa2 100644 --- a/Bank-Beispiel/.gitignore +++ b/Bank-Beispiel/.gitignore @@ -179,7 +179,5 @@ replay_pid* # Built Visual Studio Code Extensions *.vsix -/bank.ser -/zähler.ser +*.ser /.DS_Store -/Spaßkasse Mannheim-bank-data.ser From 2a1714df2f603a7fd724f36bb7047d8d99b99735 Mon Sep 17 00:00:00 2001 From: Oliver Hummel Date: Fri, 4 Nov 2022 10:12:10 +0100 Subject: [PATCH 3/4] Testklasse an die richtige Stelle geschoben. --- .../informatik/bank/{ => domain}/GirokontoTest.java | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) rename Bank-Beispiel/src/de/hs_mannheim/informatik/bank/{ => domain}/GirokontoTest.java (84%) diff --git a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/GirokontoTest.java b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java similarity index 84% rename from Bank-Beispiel/src/de/hs_mannheim/informatik/bank/GirokontoTest.java rename to Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java index 396edf1..5490892 100644 --- a/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/GirokontoTest.java +++ b/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java @@ -1,4 +1,4 @@ -package de.hs_mannheim.informatik.bank; +package de.hs_mannheim.informatik.bank.domain; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertFalse; @@ -6,9 +6,6 @@ import static org.junit.jupiter.api.Assertions.assertTrue; import org.junit.jupiter.api.Test; -import de.hs_mannheim.informatik.bank.domain.Girokonto; -import de.hs_mannheim.informatik.bank.domain.Konto; - class GirokontoTest { @Test From 44541d72555426bcbc80dea5326d653069c560c4 Mon Sep 17 00:00:00 2001 From: Oliver Hummel Date: Fri, 4 Nov 2022 10:19:24 +0100 Subject: [PATCH 4/4] =?UTF-8?q?Persistenzfehler=20behoben=20und=20Testfall?= =?UTF-8?q?=20daf=C3=BCr=20erstellt.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../informatik/bank/facade/Banksystem.java | 7 +++++-- .../informatik/bank/facade/SystemTest.java | 11 ++++++++--- .../src/de/hs_mannheim/informatik/bank/ui/UI.java | 2 +- 3 files changed, 14 insertions(+), 6 deletions(-) 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 6c0bfc2..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 @@ -66,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; 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 90d65e0..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,6 +2,7 @@ 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; @@ -45,15 +46,19 @@ class SystemTest { @Test @Order(3) void persistenzTest() throws Exception { - int knr = bs.kontoAnlegen("Test2", 1); - bs.geldEinzahlen(knr, 1000); + 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(500, bs2.getKontostand(knr)); + assertEquals(400, bs2.getKontostand(knr)); } @AfterAll 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 c3bfe9b..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 @@ -128,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());