.gitignore

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

Bank-Beispiel/.gitignore

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

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

@@ -7,24 +7,13 @@ import java.util.HashMap;
 public class Bank implements Serializable {
 	private String name;
 	private HashMap<Integer, Konto> konten = new HashMap<>();
-	private int kontozähler;
 	
 	public Bank(String name) {
 		this.name = name;
-		this.kontozähler = -1;
 	}
 	
-	public int addKonto(String name, int auswahl) {
+	public void addKonto(Konto k) {
-		Konto k;
-
-		if (auswahl == 1) 
-			k = new Konto(name, ++kontozähler);
-		else 
-			k = new Girokonto(name, ++kontozähler);
-		
 		konten.put(k.getKontonummer(), k);
-		
-		return k.getKontonummer();
 	}
 	
 	public String getName() {

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

@@ -3,14 +3,13 @@ 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) {
+	public Girokonto(String inhaber) {
-		super(inhaber, kontozähler);
+		super(inhaber);
 	}
 	
 	public boolean überweise(Girokonto ziel, long betrag, String zweck) {
-		if (super.getKontostand() - betrag >= dispo * (-1)) {
+		if (super.getKontostand() - betrag >= 0) {
 			this.auszahlen(betrag, zweck, "Überweisungsausgang", super.getInhaber());
 			ziel.einzahlen(betrag, zweck, "Überweisungseingang", super.getInhaber());
 			
@@ -20,19 +19,6 @@ 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();

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

@@ -1,27 +0,0 @@
-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

@@ -1,17 +1,31 @@
 package de.hs_mannheim.informatik.bank.domain;
 
+import java.io.IOException;
 import java.io.Serializable;
 import java.util.ArrayList;
 
+import de.hs_mannheim.informatik.bank.infrastructure.Persistenz;
+
 public class Konto implements Serializable {
+	private static int kontozähler = 0;
+
+	static { 		// die bislang eleganteste Lösung, die mir eingefallen ist
+		try {
+			if (Persistenz.sindDatenGespeichert())
+				kontozähler = Persistenz.ladeKontozähler();
+		} catch (IOException e) {
+			e.printStackTrace();
+		}
+	}
+
 	private int nummer;
-	protected long stand = 0;
+	private long stand = 0;
 	private String inhaber;
 
-	protected ArrayList<Kontobewegung> kontobewegungen;
+	private ArrayList<Kontobewegung> kontobewegungen;
 
-	public Konto(String inhaber, int kontozähler) {
+	public Konto(String inhaber) {
-		nummer = 1000 + kontozähler;
+		nummer = 1000 + kontozähler++;
 		this.inhaber = inhaber;
 
 		this.kontobewegungen = new ArrayList<>();
@@ -63,14 +77,8 @@ public class Konto implements Serializable {
 		return auflistung;
 	}
 	
-	public long berechneSaldo(int anzahl) {
+	public int getKontozähler() {
-		long saldo = 0;
+		return kontozähler;
-		
-		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

@@ -7,54 +7,24 @@ import org.junit.jupiter.api.Test;
 class KontoTest {
 
 	@Test
-	void testKontoBasics() {
+	void testKonto() {
-		Konto k = new Konto("Müller", 0);
+		Konto k = new Konto("Müller");
 		assertEquals("Müller", k.getInhaber());
 		assertEquals(1000, k.getKontonummer());
 		assertEquals(0, k.getKontostand());		
-	}
 	
-	@Test
+		Konto k2 = new Konto("Mayer");
-	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());
+		assertEquals(1001, k2.getKontonummer());
 		
 		k2.einzahlen(100, "Test", "Einzahlung", "JUnit");
 		assertEquals(100, k2.getKontostand());
-		
+		assertTrue(k2.auszahlen(50, "Test", "Einzahlung", "JUnit"));
-		assertTrue(k2.auszahlen(50, "Test", "Auszahlung", "JUnit"));
 		assertEquals(50, k2.getKontostand());
 
-		assertTrue(k2.auszahlen(50, "Test", "Auszahlung", "JUnit"));
+		assertFalse(k2.auszahlen(500, "Test", "Einzahlung", "JUnit"));
-		assertEquals(0, k2.getKontostand());
+		assertEquals(50, 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"));
 	}
 
 }

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

@@ -19,10 +19,6 @@ 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

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

@@ -11,18 +11,26 @@ public class Banksystem {
 	private Bank bank;
 
 	public Banksystem(String bankname) throws Exception {
-		if (Persistenz.sindDatenGespeichert(bankname))
+		if (Persistenz.sindDatenGespeichert())
-			this.bank = (Bank) Persistenz.ladeBankDaten(bankname);
+			this.bank = (Bank) Persistenz.ladeBankDaten();
 		else
 			this.bank = new Bank(bankname);
 	}
 
 	public int kontoAnlegen(String name, int auswahl) throws Exception {
-		int kontonummer = bank.addKonto(name, auswahl);
+		Konto k;
 
-		Persistenz.speichereBankDaten(this.bank, bank.getName());
+		if (auswahl == 1) 
+			k = new Konto(name);
+		else 
+			k = new Girokonto(name);
 
-		return kontonummer;
+		bank.addKonto(k);
+		
+		Persistenz.speichereBankDaten(this.bank);
+		Persistenz.speichereKontozähler(k.getKontozähler());
+
+		return k.getKontonummer();
 	}
 
 	public String[] getKontenliste() {
@@ -45,7 +53,7 @@ public class Banksystem {
 		Konto konto = bank.findeKonto(kontonummer);
 		konto.einzahlen(betrag, "Einzahlung am Schalter", "Einzahlung", konto.getInhaber());
 
-		Persistenz.speichereBankDaten(this.bank, bank.getName());
+		Persistenz.speichereBankDaten(this.bank);
 		
 		return konto.getKontostand();
 	}
@@ -53,11 +61,9 @@ 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);
 
-		Persistenz.speichereBankDaten(this.bank, bank.getName());
+		return konto.auszahlen(betrag, "Auszahlung am Schalter", "Auszahlung", konto.getInhaber());
-		
-		return erg;
 	}
 
 	public String[] erstelleKontoauszug(int kontonummer) {
@@ -66,15 +72,12 @@ public class Banksystem {
 		return konto.getKontobewegungen();
 	}
 
-	public boolean überweisungBeauftragen(int startkonto, int zielkonto, long betrag, String verwendungszweck) throws Exception {
+	public boolean überweisungBeauftragen(int startkonto, int zielkonto, long betrag, String verwendungszweck) {
 		Konto start = bank.findeKonto(startkonto);
 		Konto ziel = bank.findeKonto(zielkonto);
 
 		if (start instanceof Girokonto && ziel instanceof Girokonto) {
-			boolean erfolg = ((Girokonto)start).überweise((Girokonto)ziel, betrag, verwendungszweck);
+			return ((Girokonto)start).überweise((Girokonto)ziel, betrag, verwendungszweck);
-			Persistenz.speichereBankDaten(this.bank, bank.getName());
-			
-			return erfolg;
 		}
 
 		return false;
@@ -86,10 +89,4 @@ public class Banksystem {
 		return konto.getKontostand();
 	}
 
-	public long saldoBestimmen(int kontonummer, int anzahl) {
-		Konto konto = bank.findeKonto(kontonummer);
-		
-		return konto.berechneSaldo(anzahl);
-	}
-
 }

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

@@ -1,70 +0,0 @@
-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 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
-	@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", 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();
-	}
-
-}

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/infrastructure/Persistenz.java

@@ -2,29 +2,46 @@ package de.hs_mannheim.informatik.bank.infrastructure;
 
 import java.io.File;
 import java.io.FileInputStream;
+import java.io.FileNotFoundException;
 import java.io.FileOutputStream;
+import java.io.IOException;
 import java.io.ObjectInputStream;
 import java.io.ObjectOutputStream;
 
 public class Persistenz {
-	private static final String BANK_DATEI = "-bank-data.ser";
+	private static final String BANK_DATEI = "bank.ser";
+	private static final String ZÄHLER_DATEI = "zähler.ser";
 	
-	public static boolean sindDatenGespeichert(String name) {
+	public static boolean sindDatenGespeichert() {
-		return new File(name + BANK_DATEI).exists();
+		return new File(BANK_DATEI).exists();
 	}
 
-	public static void speichereBankDaten(Object bank, String name) throws Exception {
+	public static void speichereBankDaten(Object bank) throws Exception {
-		ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(name + BANK_DATEI));
+		ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(BANK_DATEI));
 		oos.writeObject(bank);
 		oos.close();
 	}
 	
-	public static Object ladeBankDaten(String name) throws Exception {
+	public static void speichereKontozähler(int kontozähler) throws IOException {
-		ObjectInputStream ois = new ObjectInputStream(new FileInputStream(name + BANK_DATEI));
+		ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(ZÄHLER_DATEI));
+		oos.writeInt(kontozähler);
+		oos.close();
+	}
+	
+	public static Object ladeBankDaten() throws Exception {
+		ObjectInputStream ois = new ObjectInputStream(new FileInputStream(BANK_DATEI));
 		Object bank = ois.readObject();
 		ois.close();
 		
 		return bank;
 	}
 	
+	public static int ladeKontozähler() throws FileNotFoundException, IOException {
+		ObjectInputStream ois = new ObjectInputStream(new FileInputStream(ZÄHLER_DATEI));
+		int zähler = ois.readInt();
+		ois.close();
+		
+		return zähler; 
+	}
+
 }

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

@@ -27,7 +27,6 @@ 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();
@@ -46,7 +45,6 @@ public class UI {
 					case 4: geldAuszahlen(); break;
 					case 5: kontoauszugDrucken(); break;
 					case 6: überweisungBeauftragen(); break;
-					case 7: saldoAbfragen(); break;
 					case 9: break mainloop;
 					}
 
@@ -116,6 +114,8 @@ 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);
@@ -128,7 +128,7 @@ public class UI {
 			System.out.println("Noch keine Kontobewegungen.");
 	}
 
-	private void überweisungBeauftragen() throws Exception {
+	private void überweisungBeauftragen() {
 		System.out.print("Bitte die Kontonummer des Ausgangskontos der Überweisung eingeben: ");
 		int startkonto = Integer.parseInt(sc.nextLine());
 
@@ -146,15 +146,4 @@ 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));
-	}
-
 }