diff --git a/Bank-System-main/bank-system/.gitignore b/Bank-System-main/bank-system/.gitignore new file mode 100644 index 0000000..9bb88d3 --- /dev/null +++ b/Bank-System-main/bank-system/.gitignore @@ -0,0 +1 @@ +/.DS_Store diff --git a/Bank-System-main/bank-system/Bank-Beispiel/.gitignore b/Bank-System-main/bank-system/Bank-Beispiel/.gitignore index c8f10d9..d77bfa2 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/.gitignore +++ b/Bank-System-main/bank-system/Bank-Beispiel/.gitignore @@ -179,5 +179,5 @@ replay_pid* # Built Visual Studio Code Extensions *.vsix -/bank.ser -/zähler.ser +*.ser +/.DS_Store diff --git a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java index 19af250..d8f12a9 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java +++ b/Bank-System-main/bank-system/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 dispotBetrag = 200000; + 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 >= (dispotBetrag * -1)) { + if (super.getKontostand() - betrag >= dispo * (-1)) { this.auszahlen(betrag, zweck, "Überweisungsausgang", super.getInhaber()); ziel.einzahlen(betrag, zweck, "Überweisungseingang", super.getInhaber()); @@ -19,8 +20,9 @@ public class Girokonto extends Konto implements Serializable { return false; } + @Override public boolean auszahlen(long betrag, String zweck, String art, String auftraggeber) { - if (stand - betrag >= (dispotBetrag * -1)) { + if (stand - betrag >= dispo * (-1)) { stand -= betrag; kontobewegungen.add(new Kontobewegung(betrag * -1, zweck, art, auftraggeber)); diff --git a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java index aac51ab..5490892 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java +++ b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java @@ -1,47 +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 testKontoBasics() { + void testÜberziehung() { Konto k = new Girokonto("Müller", 0); - assertEquals("Müller", k.getInhaber()); - assertEquals(1000, k.getKontonummer()); - assertEquals(0, k.getKontostand()); - } - - @Test - void testKontoEinUndAuszahlung() { - Konto k = new Girokonto("Müller", 0); - Konto k2 = new Girokonto("Mayer", 1); + k.einzahlen(10000, "Test", "Einzahlung", "JUnit"); - assertEquals("Mayer", k2.getInhaber()); - assertNotEquals(k.getKontonummer(), k2.getKontonummer()); + assertTrue(k.auszahlen(20000, "Test", "Einzahlung", "JUnit")); + assertEquals(-10000, k.getKontostand()); - k2.einzahlen(10000, "Test", "Einzahlung", "JUnit"); - assertEquals(10000, k2.getKontostand()); - - assertTrue(k2.auszahlen(100000, "Test", "Auszahlung", "JUnit")); - assertEquals(-90000, k2.getKontostand()); - - assertFalse(k2.auszahlen(120000, "Test", "Auszahlung", "JUnit")); - assertNotEquals(-210000, k2.getKontostand()); - } - - @Test - void überweiseTest() { - Konto k = new Girokonto("Müller", 0); - Konto k2 = new Girokonto("Mayer", 1); - - boolean test1 = ((Girokonto)k).überweise((Girokonto) k2, 100000, "Test"); - assertTrue(test1); + assertTrue(k.auszahlen(40000, "Test", "Einzahlung", "JUnit")); + assertTrue(k.auszahlen(50000, "Test", "Einzahlung", "JUnit")); assertEquals(-100000, k.getKontostand()); - boolean test2 = ((Girokonto)k).überweise((Girokonto) k2, 110000, "Test"); - assertFalse(test2); + assertFalse(k.auszahlen(40000, "Test", "Einzahlung", "JUnit")); + assertFalse(k.auszahlen(1, "Test", "Einzahlung", "JUnit")); } + } diff --git a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java index ae602f8..e787df5 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java +++ b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java @@ -23,7 +23,7 @@ public class Konto implements Serializable { @Override public String toString() { - return "Konto [nummer=" + nummer + ", inhaber=" + inhaber + ", stand=" + stand +"]"; + return "Konto [nummer=" + nummer + ", inhaber=" + inhaber + "]"; } public String getInhaber() { @@ -62,8 +62,15 @@ public class Konto implements Serializable { return auflistung; } - - public long getKontobewegungBetrag(int kontoauszüg) { - return kontobewegungen.get(kontoauszüg).getKontobewegungBetrag(); + + 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-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/KontoTest.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/KontoTest.java index 603e249..e20060a 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/KontoTest.java +++ b/Bank-System-main/bank-system/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,30 +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")); } -// @Test -// void testToString() { -// Konto k = new Konto("Müller", 0); -// -// assertEquals("Müller", k.getInhaber()); -// assertNotNull(k.toString()); -// } - -// @Test -// void testGetKontobewegung() { -// Konto k = new Konto("Müller", 0); -// Konto k2 = new Konto("Mayer", 1); -// -// assertEquals("Müller", k.getInhaber()); -// assertNotNull(k.getKontobewegungen()); -// -// assertEquals("Mayer", k2.getInhaber()); -// k2.einzahlen(100, "Test", "Einzahlung", "JUnit"); -// assertEquals(100, k2.getKontostand()); -// assertNotNull(k2.getKontobewegungen()); -// } - } diff --git a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java index 40179ed..22f1401 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java +++ b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java @@ -19,7 +19,7 @@ public class Kontobewegung implements Serializable { this.datum = new Date(); } - public long getKontobewegungBetrag() { + public long getBetrag() { return betrag; } diff --git a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java index 860b19a..5ff5144 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java +++ b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java @@ -53,23 +53,11 @@ public class Banksystem { public boolean geldAuszahlen(int kontonummer, long betrag) throws Exception { Konto konto = bank.findeKonto(kontonummer); - if(konto instanceof Girokonto) { - boolean auszahlBetrag = ((Girokonto)konto).auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber()); - - Persistenz.speichereBankDaten(this.bank, bank.getName()); - return auszahlBetrag; - - } else { - boolean auszahlBetrag = konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber()); - - Persistenz.speichereBankDaten(this.bank, bank.getName()); - return auszahlBetrag; - } + boolean erg = konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber()); + + Persistenz.speichereBankDaten(this.bank, bank.getName()); -// boolean auszahlBetrag = konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber()); -// -// Persistenz.speichereBankDaten(this.bank, bank.getName()); -// return auszahlBetrag; + return erg; } public String[] erstelleKontoauszug(int kontonummer) { @@ -78,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; @@ -95,14 +86,10 @@ public class Banksystem { return konto.getKontostand(); } - public long saldoRechnung(int kontonummer, int kontoauszüge) { + public long saldoBestimmen(int kontonummer, int anzahl) { Konto konto = bank.findeKonto(kontonummer); - long saldo = 0; - for (int i = 0; i < kontoauszüge; i++) { - saldo += konto.getKontobewegungBetrag(i); - } - return saldo; + return konto.berechneSaldo(anzahl); } } diff --git a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java index 0082881..b063dce 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java +++ b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java @@ -1,88 +1,70 @@ package de.hs_mannheim.informatik.bank.facade; import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertFalse; import static org.junit.jupiter.api.Assertions.assertNotNull; -import static org.junit.jupiter.api.Assertions.assertNull; import static org.junit.jupiter.api.Assertions.assertTrue; 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()); - - dateiLöschen(); } @Test + @Order(2) void einzahlenTest() throws Exception { - Banksystem bs = new Banksystem("Testsystem"); - int kontonummer = bs.kontoAnlegen("Müller", 2); - - assertEquals(500, bs.geldEinzahlen(kontonummer, 500)); + int knr = bs.kontoAnlegen("Test1", 1); - assertTrue(dateiExistiert()); - dateiLöschen(); - assertFalse(dateiExistiert()); - - } - - @Test - void auszahlenTest() throws Exception { - Banksystem bs = new Banksystem("Testsystem"); - int kontonummer = bs.kontoAnlegen("Müller", 2); - int kontonummer2 = bs.kontoAnlegen("Müller", 1); - + assertEquals(1000, bs.geldEinzahlen(knr, 1000)); - assertEquals(500, bs.geldEinzahlen(kontonummer, 500)); - assertEquals(true, bs.geldAuszahlen(kontonummer, 250)); - assertEquals(1000, bs.geldEinzahlen(kontonummer2, 1000)); - assertEquals(true, bs.geldAuszahlen(kontonummer2, 500)); + bs.geldEinzahlen(knr, 1); + assertEquals(1001, bs.getKontostand(knr)); - assertTrue(dateiExistiert()); - dateiLöschen(); - assertFalse(dateiExistiert()); + assertEquals(1001, bs.geldEinzahlen(knr, 0)); } @Test - void saldoBetragTest() throws Exception { - Banksystem bs = new Banksystem("Testsystem"); - int kontonummer = bs.kontoAnlegen("Müller", 1); - - bs.geldEinzahlen(kontonummer, 500); - bs.geldEinzahlen(kontonummer, 1000); - bs.geldAuszahlen(kontonummer, 250); + @Order(3) + void persistenzTest() throws Exception { + int knr = bs.kontoAnlegen("Test2", 2); + int knr2 = bs.kontoAnlegen("Test3", 2); - assertEquals(1250, bs.saldoRechnung(kontonummer, 3)); + bs.geldEinzahlen(knr, 1000); + bs.geldAuszahlen(knr, 500); + assertTrue(bs.überweisungBeauftragen(knr, knr2, 100, "Überweisungstest.")); - dateiLöschen(); + assertEquals(400, bs.getKontostand(knr)); + + bs = null; + + Banksystem bs2 = new Banksystem("Testsystem"); + assertEquals(400, bs2.getKontostand(knr)); } - @Test - boolean dateiExistiert() throws Exception { - File f = new File("Testsystem-bank-data.ser"); + @AfterAll + static void cleanup() { + File file = new File("/Users/oliver/git/Bank-System/Bank-Beispiel/Testsystem-bank-data.ser"); + file.delete(); + } - if(f.exists()) { - return true; - } - return false; - } - - @Test - void dateiLöschen() throws Exception { - File f = new File("Testsystem-bank-data.ser"); - - if(f.exists()) { - f.delete(); - } - } } diff --git a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java b/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java index 73ada34..3f10d79 100644 --- a/Bank-System-main/bank-system/Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java +++ b/Bank-System-main/bank-system/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 errechnen"); + System.out.println("7 -> Saldo abfragen"); System.out.println("9 -> Beenden"); System.out.println(); @@ -39,12 +39,14 @@ public class UI { try { switch(input) { case 1: kontenAnzeigen(); break; - case 2: kontoAnlegen(); break; + case 2: + kontoAnlegen(); + break; case 3: geldEinzahlen(); break; case 4: geldAuszahlen(); break; case 5: kontoauszugDrucken(); break; case 6: überweisungBeauftragen(); break; - case 7: saldoRechnung(); break; + case 7: saldoAbfragen(); break; case 9: break mainloop; } @@ -58,7 +60,6 @@ public class UI { } // hauptmenü - private void kontenAnzeigen() { String[] konten = bs.getKontenliste(); if (konten.length > 0) { @@ -115,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); @@ -129,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()); @@ -147,20 +146,15 @@ public class UI { System.out.println("Überweisung" + ( (!erfolgreich) ? " nicht" : "") + " erfolgreich ausgeführt."); } - private void saldoRechnung() { - System.out.print("Bitte die gewünschte Kontonummer für den Auszug eingeben: "); - int kontonummer = 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.println(); + System.out.print("Bitte die Anzahl der Kontobewegungen für den Saldo eingeben: "); + int anzahl = Integer.parseInt(sc.nextLine()); - System.out.print("Bitte die gewünschte Anzahl der Kontoauszüge angeben: "); - int kontoauszüge = Integer.parseInt(sc.nextLine()); - - System.out.println(); - - System.out.println("Auszug für Konto " + kontonummer); - long saldo = bs.saldoRechnung(kontonummer, kontoauszüge); - System.out.println("Hier ist Ihr Saldo von " + kontoauszüge + " Kontoauszügen: " + saldo); + long saldo = bs.saldoBestimmen(konto, anzahl); + System.out.printf("Der Saldo nach %d Kontobewegungen beträgt %.2f Euro.%n", anzahl, (saldo/100d)); } }