Persistenzfehler behoben und Testfall dafür erstellt.

(mit einigen weiteren Tests, aber ohne Gewähr!)

.gitignore

@@ -0,0 +1 @@
+/.DS_Store

Bank-Beispiel/.gitignore

@@ -179,5 +179,5 @@ replay_pid*
 # Built Visual Studio Code Extensions
 *.vsix
 
-/bank.ser
-/zähler.ser
+*.ser
+/.DS_Store

Bank-Beispiel/.project

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <projectDescription>
-	<name>Bank-Beispiel-Forked</name>
+	<name>Bank-Beispiel</name>
 	<comment></comment>
 	<projects>
 	</projects>

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/Main.java

@@ -5,7 +5,6 @@ import de.hs_mannheim.informatik.bank.ui.UI;
 
 public class Main {
 
-	
 	public static void main(String[] args) throws Exception {
 		Banksystem bs = new Banksystem("Spaßkasse Mannheim");
 		UI ui = new UI(bs);

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Girokonto.java

@@ -1,31 +1,31 @@
 package de.hs_mannheim.informatik.bank.domain;
 
 import java.io.Serializable;
-import java.util.ArrayList;
 
 public class Girokonto extends Konto implements Serializable {
-	private final long dispo_summe_giro = -(100 * 100);
+	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 * (-1)) {
+			this.auszahlen(betrag, zweck, "Überweisungsausgang", super.getInhaber());
+			ziel.einzahlen(betrag, zweck, "Überweisungseingang", super.getInhaber());
 			
-	public boolean auszahlen(long betrag, String zweck, String art, String auftraggeber) {
-
-		if (super.stand - betrag >= dispo_summe_giro) {
-			super.stand -= betrag;
-			super.kontobewegungen.add(new Kontobewegung(betrag * -1, zweck, art, auftraggeber));
 			return true;
 		}
 			
 		return false;
 	}
 	
-	public boolean überweise(Girokonto ziel, long betrag, String zweck) {
-		if (super.getKontostand() - betrag >= dispo_summe_giro) {
-			this.auszahlen(betrag, zweck, "Überweisungsausgang", super.getInhaber());
-			ziel.einzahlen(betrag, zweck, "Überweisungseingang", super.getInhaber());
+	@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;
 		}

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/GirokontoTest.java

@@ -0,0 +1,27 @@
+package de.hs_mannheim.informatik.bank.domain;
+
+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 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"));
+	}
+
+}

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Konto.java

@@ -23,17 +23,9 @@ public class Konto implements Serializable {
 
 	@Override
 	public String toString() {
-		return "Konto [nummer=" + nummer + ", inhaber=" + inhaber + " Stand=" + stand / 100 + "]";
+		return "Konto [nummer=" + nummer + ", inhaber=" + inhaber + "]";
 	}
 
-	public long getSaldo(int bewegung) {
-		int summe = 0;
-		for (int i = 0; i <= bewegung - 1; i++) {
-			summe += kontobewegungen.get(i).getBetrag();
-		}
-		
-		return summe;
-	}
 	public String getInhaber() {
 		return inhaber;
 	}
@@ -49,10 +41,11 @@ public class Konto implements Serializable {
 	}
 
 	public boolean auszahlen(long betrag, String zweck, String art, String auftraggeber) {
-
 		if (stand - betrag >= 0) {
 			stand -= betrag;
+
 			kontobewegungen.add(new Kontobewegung(betrag * -1, zweck, art, auftraggeber));
+
 			return true;
 		}
 
@@ -70,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;
+	}
+
 }

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,37 +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 dispoTest() {
-		Girokonto ausgang = new Girokonto("Ausgang", 0);
-		Girokonto ziel = new Girokonto("Ziel", 1);
-		//Überweisung
-		ausgang.überweise(ziel, 100, "Überweisung");
-		assertEquals(-100, ausgang.getKontostand());
-		assertEquals(100, ziel.getKontostand());
-		
-		//Abheben
-		ausgang.einzahlen(100, "Einzahlung", "ka", "ka");
-		assertEquals(0, ausgang.getKontostand());
-		ausgang.auszahlen(-100, "Auszahlung", "ka", "ka");
-
-	}
-
-	@Test
-	void saldoTest() {
-		Konto giro = new Konto("Ausgang", 0);
-
-		//Überweisung
-		giro.einzahlen(1000, "", "", "");
-		giro.auszahlen(180, "", "", "");
-		assertEquals(820, giro.getSaldo(2));
-		assertEquals(1000, giro.getSaldo(1));
-		
+	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"));
 	}
 
 }

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/domain/Kontobewegung.java

@@ -15,6 +15,7 @@ public class Kontobewegung implements Serializable {
 		this.betreff = betreff;
 		this.art = art;
 		this.auftraggeber = auftraggeber;
+		
 		this.datum = new Date();
 	}
 	

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/Banksystem.java

@@ -1,12 +1,10 @@
 package de.hs_mannheim.informatik.bank.facade;
 
-import java.util.ArrayList;
 import java.util.Collection;
 
 import de.hs_mannheim.informatik.bank.domain.Bank;
 import de.hs_mannheim.informatik.bank.domain.Girokonto;
 import de.hs_mannheim.informatik.bank.domain.Konto;
-import de.hs_mannheim.informatik.bank.domain.Kontobewegung;
 import de.hs_mannheim.informatik.bank.infrastructure.Persistenz;
 
 public class Banksystem {
@@ -54,9 +52,12 @@ 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) {
@@ -65,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;
@@ -82,10 +86,10 @@ public class Banksystem {
 		return konto.getKontostand();
 	}
 
-	public long getSaldo(int kontonummer, int bewegung) {
+	public long saldoBestimmen(int kontonummer, int anzahl) {
+		Konto konto = bank.findeKonto(kontonummer);
 		
-		long summe = bank.findeKonto(kontonummer).getSaldo(bewegung);
-		return summe;
+		return konto.berechneSaldo(anzahl);
 	}
 
 }

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/facade/SystemTest.java

@@ -1,57 +1,70 @@
 package de.hs_mannheim.informatik.bank.facade;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.*;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+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;
 
-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());
 	}
 	
 	@Test
-	void geldEinzahlen() throws Exception {
-		Banksystem bs = new Banksystem("UnitBank");
-		bs.kontoAnlegen("TestKonto", 1);
-		assertEquals(0l, bs.getKontostand(1000));
-		bs.geldEinzahlen(1000, 200l);
-		assertEquals(200l, bs.getKontostand(1000));
-		File x = new File(bs.getBankname() + "-bank-data.ser");
-		assertTrue(x.delete());
+	@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
-	void geldAuszahlen() throws Exception {
-		Banksystem bs = new Banksystem("UnitBank");
-		bs.kontoAnlegen("TestKonto", 1);
-		bs.geldEinzahlen(1000, 200l);
-		bs.geldAuszahlen(1000, 100l);
-		assertEquals(100l, bs.getKontostand(1000));
-		File x = new File(bs.getBankname() + "-bank-data.ser");
-		assertTrue(x.delete());
+	@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));
 	}
 	
-	@Test
-	void persitenzTest() throws Exception {
-		Bank n = new Bank("Unit");
-		Banksystem bs = new Banksystem("UnitBank");
-		File x = new File(bs.getBankname() + "-bank-data.ser");
-		bs.kontoAnlegen("TestKonto", 1);
-		bs.geldEinzahlen(1000, 200l);
-		Persistenz.speichereBankDaten(n,"n");
-		assertTrue(Persistenz.sindDatenGespeichert("n"));
-		assertTrue(x.delete());
+	@AfterAll
+	static void cleanup() {
+		File file = new File("/Users/oliver/git/Bank-System/Bank-Beispiel/Testsystem-bank-data.ser");
+		file.delete();
 	}
+
 }

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/ui/UI.java

@@ -16,55 +16,45 @@ public class UI {
 	private void hauptmenü() {
 		System.out.println("Willkommen bei der " + bs.getBankname() + "!");
 
-		mainloop: while (true) {
-			System.out.println();
-			System.out.println("--------");
-			System.out.println("Hauptmenü");
-			System.out.println("1 -> Konten anzeigen");
-			System.out.println("2 -> Konto anlegen");
-			System.out.println("3 -> Geld einzahlen");
-			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("9 -> Beenden");
-			System.out.println();
+		mainloop: 
+			while (true) {
+				System.out.println();
+				System.out.println("--------");
+				System.out.println("Hauptmenü");
+				System.out.println("1 -> Konten anzeigen");
+				System.out.println("2 -> Konto anlegen");
+				System.out.println("3 -> Geld einzahlen");
+				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.print("> ");
-			int input = Integer.parseInt(sc.nextLine());
-			System.out.println();
+				System.out.println("9 -> Beenden");
+				System.out.println();
 
-			try {
-				switch (input) {
-				case 1:
-					kontenAnzeigen();
-					break;
-				case 2:
-					kontoAnlegen();
-					break;
-				case 3:
-					geldEinzahlen();
-					break;
-				case 4:
-					geldAuszahlen();
-					break;
-				case 5:
-					kontoauszugDrucken();
-					break;
-				case 6:
-					überweisungBeauftragen();
-					break;
-				case 7:
-					saldoAnzeigen(); break;
-				case 9:
-					break mainloop;
+				System.out.print("> ");
+				int input = Integer.parseInt(sc.nextLine());
+				System.out.println();
+
+				try {
+					switch(input) {
+					case 1: kontenAnzeigen(); break;
+					case 2: 
+						kontoAnlegen();
+						break;
+					case 3: geldEinzahlen(); break;
+					case 4: geldAuszahlen(); break;
+					case 5: kontoauszugDrucken(); break;
+					case 6: überweisungBeauftragen(); break;
+					case 7: saldoAbfragen(); break;
+					case 9: break mainloop;
+					}
+
+				} catch (Exception e) {
+					System.err.println(e.getLocalizedMessage());
 				} 
-
-			} catch (Exception e) {
-				System.err.println(e.getLocalizedMessage());
+				System.out.println();
 			}
-			System.out.println();
-		}
 
 		System.out.println("Auf Wiedersehen!");
 
@@ -103,7 +93,7 @@ public class UI {
 		System.out.print("Bitte den gewünschten Betrag eingeben: ");
 		double betrag = Double.parseDouble(sc.nextLine());
 
-		long neuerKontostand = bs.geldEinzahlen(kontonummer, (long) betrag * 100);
+		long neuerKontostand = bs.geldEinzahlen(kontonummer, (long)betrag * 100);
 
 		System.out.printf("Einzahlung erfolgreich, neuer Kontostand = %.2f Euro", (neuerKontostand / 100.0));
 	}
@@ -116,9 +106,9 @@ public class UI {
 		System.out.print("Bitte den gewünschten Betrag eingeben: ");
 		double betrag = Double.parseDouble(sc.nextLine());
 
-		boolean erfolgreich = bs.geldAuszahlen(kontonummer, (long) betrag * 100);
+		boolean erfolgreich = bs.geldAuszahlen(kontonummer, (long)betrag * 100);
 
-		System.out.printf("Auszahlung" + ((!erfolgreich) ? " nicht" : "") + " erfolgreich. ");
+		System.out.printf("Auszahlung" + ((!erfolgreich)? " nicht" : "" )+ " erfolgreich. ");
 		System.out.printf("Neuer Kontostand = %.2f Euro.", (bs.getKontostand(kontonummer) / 100.0));	
 	}
 
@@ -126,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);
@@ -140,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());
 
@@ -153,18 +141,20 @@ public class UI {
 		System.out.print("Bitte den Verwendungszweck eingeben: ");
 		String verwendungszweck = sc.nextLine();
 
-		boolean erfolgreich = bs.überweisungBeauftragen(startkonto, zielkonto, (long) (betrag * 100), verwendungszweck);
+		boolean erfolgreich = bs.überweisungBeauftragen(startkonto, zielkonto, (long)(betrag * 100), verwendungszweck);
 
-		System.out.println("Überweisung" + ((!erfolgreich) ? " nicht" : "") + " erfolgreich ausgeführt.");
+		System.out.println("Überweisung" + ( (!erfolgreich) ? " nicht" : "") + " erfolgreich ausgeführt.");
 	}
 	
-	private void saldoAnzeigen() {
-		System.out.print("Bitte die gewünschte Kontonummer 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.print("Bitte den gewünschten Kontobewegungen eingeben: ");
-		int bewegung = 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.println("Saldo bis " + bewegung + ": " + bs .getSaldo(kontonummer, bewegung)/100 + " Euro");
+		long saldo = bs.saldoBestimmen(konto, anzahl);
+		System.out.printf("Der Saldo nach %d Kontobewegungen beträgt %.2f Euro.%n", anzahl, (saldo/100d));
 	}
+
 }