Persistenzfehler behoben und Testfall dafür erstellt.

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

.gitignore

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

.project

@@ -1,11 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<projectDescription>
-	<name>Bank-System</name>
-	<comment></comment>
-	<projects>
-	</projects>
-	<buildSpec>
-	</buildSpec>
-	<natures>
-	</natures>
-</projectDescription>

Bank-Beispiel/.gitignore

@@ -179,3 +179,5 @@ replay_pid*
 # Built Visual Studio Code Extensions
 *.vsix
 
+*.ser
+/.DS_Store

Bank-Beispiel/.project

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

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

@@ -7,7 +7,6 @@ public class Main {
 
 	public static void main(String[] args) throws Exception {
 		Banksystem bs = new Banksystem("Spaßkasse Mannheim");
-		System.out.println("Hallo");
 		UI ui = new UI(bs);
 	}
 

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

@@ -1,15 +1,10 @@
 package de.hs_mannheim.informatik.bank.domain;
 
 import java.io.Serializable;
-import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashMap;
-import java.util.List;
-
-
 
 public class Bank implements Serializable {
-
 	private String name;
 	private HashMap<Integer, Konto> konten = new HashMap<>();
 	private int kontozähler;
@@ -19,9 +14,6 @@ public class Bank implements Serializable {
 		this.kontozähler = -1;
 	}
 	
-	public void addKonto(Konto k) {
-		konten.put(k.getKontonummer(), k);
-	}
 	public int addKonto(String name, int auswahl) {
 		Konto k;
 
@@ -35,73 +27,16 @@ public class Bank implements Serializable {
 		return k.getKontonummer();
 	}
 	
-
-
-	public long zahleEin(int kontonummer, long betrag) throws Exception {
-		Konto konto = findeKonto(kontonummer);
-		konto.zahleEin(betrag, null);
-		
-		return konto.getKontostand();
-	}
-	
-	public long zahleAus(Integer kontonummer, long einzahlung) {
-		Konto konto = findeKonto(kontonummer);
-		
-//		if(konto == null)
-//			return Long.MIN_VALUE;
-		
-		return konto.zahleAus(einzahlung,konto.getName());
-	}
-	public long zahleAus2(Integer kontonummer, long einzahlung) {
-		Konto konto = konten.get(kontonummer);
-		
-		if(konto == null)
-			return Long.MIN_VALUE;
-		
-		return konto.zahleAus2(einzahlung,konto.getName());
-	}
-	
-	
 	public String getName() {
 		return name;
 	}
 	
-	public long getKontostand(Integer kontonummer) {
-		Konto konto = konten.get(kontonummer);
-		
-		if(konto == null)
-			return Long.MIN_VALUE;
-		
-		return konto.getKontostand();
-	}
-	
-	// Damit nicht auf nicht existierende Konten eingezahlt werden kann
-	public boolean existiertKonto(Integer kontonummer) {
-		return konten.containsKey(kontonummer);
-	}
-	public Konto existiertKonto2(Integer kontonummer) {
-		return konten.get(kontonummer);
-	}
-	
 	public Collection<Konto> getKontenliste() {
 		return konten.values();
 	}
 
-	public List<Kontoauszug> getAuszuge(Integer kontonummer) {
-		Konto konto = konten.get(kontonummer);
-		
-		if(konto == null)
-			return new ArrayList<>();
-		
-		return konto.getAuszuge();
-	}
-	
-	public long getSaldo(int kontonummer, int anzahl) {
-		Konto konto = konten.get(kontonummer);
-		return konto.getSaldo(anzahl);
-	}
-
 	public Konto findeKonto(int kontonummer) {
 		return konten.get(kontonummer);
 	}
+	
 }

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

@@ -3,28 +3,34 @@ 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) {
-			this.zahleAus2(betrag, this.getInhaber());
-			ziel.zahleEin(betrag, this.getInhaber());
+		if (super.getKontostand() - betrag >= dispo * (-1)) {
+			this.auszahlen(betrag, zweck, "Überweisungsausgang", super.getInhaber());
+			ziel.einzahlen(betrag, zweck, "Überweisungseingang", super.getInhaber());
 			
 			return true;
 		}
 			
 		return false;
 	}
-	public long kreditEinfügen(Girokonto start, long betrag) {
-		if (super.getKontostand() - betrag <= 0 && super.getKontostand()-betrag >=-500) {
-			this.zahleAus2(betrag,this.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;
 		}
-		return this.getKontostand();
 
+		return false;
 	}
 	
 	@Override

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

@@ -1,59 +1,27 @@
 package de.hs_mannheim.informatik.bank.domain;
 
-import static org.junit.Assert.assertTrue;
-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 überweise() {
-		Girokonto g = new Girokonto("hans", 0);
-		assertEquals("hans", g.getInhaber());
+	void testÜberziehung() {
+		Konto k = new Girokonto("Müller", 0);
+		k.einzahlen(10000, "Test", "Einzahlung", "JUnit");
 		
-		Girokonto g2 = new Girokonto("Peter", 1);
-		assertNotEquals("Peter", g.getInhaber());
-		
-		g2.zahleEin(500, "hanso");
-
-		Girokonto g3 = new Girokonto("Felix" ,2);
-	
-		assertEquals(g2.getKontostand(), 500);
-		assertEquals("Peter", g2.getInhaber());
-		
-		
-		assertTrue(g2.überweise(g3, 500, "Testen"));
-		assertEquals(g3.getKontostand(), 500);
+		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"));
 	}
 
-		// oos = legt neue datei ein, wenn die datei da ist dann öffnet es die datei, da ist ne varibale drin und das wird in input stream reingeschrieben und ausgegebn
-	@Test
-	void kreditEInfügen() {
-		Girokonto g4 = new Girokonto("Tim", 1);
-		
-		long betrag = 400;
-		g4.kreditEinfügen(g4, betrag);
-		System.out.println(g4.getKontostand());
-		assertEquals(g4.getKontostand(), -400);
-	}
-	
-		
-	@Test 
-	void testÜberweisen() {
-		
-		
-		Girokonto g5 = new Girokonto("hans", 0);
-		Girokonto g6 = new Girokonto("Peter", 1);
-		g5.zahleEin(500, "Peter");
-		g5.überweise(g6, 500,"testen" );
-		System.out.println(g6.getKontostand());
-		assertEquals(500, g6.getKontostand());
-		
-	}
-	
-	
-
 }

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

@@ -2,81 +2,75 @@ package de.hs_mannheim.informatik.bank.domain;
 
 import java.io.Serializable;
 import java.util.ArrayList;
-import java.util.List;
 
 public class Konto implements Serializable {
-
-	private static int kontozähler = 0; 
-	
 	private int nummer;
-	private long stand = 0; 
+	protected long stand = 0;
 	private String inhaber;
 
-	private ArrayList<Kontoauszug> auszuge = new ArrayList<>();
+	protected ArrayList<Kontobewegung> kontobewegungen;
 
 	public Konto(String inhaber, int kontozähler) {
-		nummer = 1000 + kontozähler++;
+		nummer = 1000 + kontozähler;
 		this.inhaber = inhaber;
 
-	}
-	public long getSaldo(int z) {
-		long saldo = 0;
-		for ( int x = 0 ; x < z; x++){
-			saldo += auszuge.get(z).getBetrag();	
-		}
-		return saldo;
+		this.kontobewegungen = new ArrayList<>();
 	}
 
 	public int getKontonummer() {
 		return nummer;
 	}
 
-	public long getKontostand() {
-		return stand;
+	@Override
+	public String toString() {
+		return "Konto [nummer=" + nummer + ", inhaber=" + inhaber + "]";
 	}
+
 	public String getInhaber() {
 		return inhaber;
 	}
 
-	public long zahleEin(long einzahlung, String auftraggeber) {
-		this.stand += einzahlung;
-		
-		// inhaber muss theoretisch ermittelt werden mit scanner
-		auszuge.add(new Kontoauszug(stand, einzahlung, auftraggeber));
+	public long getKontostand() {
 		return stand;
 	}
-	public long zahleAus(long einzahlung, String auftraggeber) {
-		if (stand - einzahlung >= 0) {
-			stand -= einzahlung;
-			auszuge.add(new Kontoauszug(stand,einzahlung, auftraggeber));
-			return stand;
-		}
-		System.out.println("Sie möchten zu viel abheben. Aktueller Stand: " + stand);
-		return stand;
-	}
-	public long zahleAus2(long einzahlung, String auftraggeber) {
 
-			stand -= einzahlung;
-			System.out.println(stand);
-			auszuge.add(new Kontoauszug(stand,einzahlung, auftraggeber));
-			return stand;
-	}
-	public List<Kontoauszug> getAuszuge() {
-		return auszuge;
+	public void einzahlen(long betrag, String zweck, String art, String auftraggeber) {
+		stand += betrag;
+
+		kontobewegungen.add(new Kontobewegung(betrag, zweck, art, auftraggeber));
 	}
 
-	@Override
-	public String toString() {
-		return "Konto [nummer=" + nummer + ", inhaber=" + inhaber + "]" + " Konotstand= " + stand; 
+	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;
 		}
 
-	public int getKontozähler() {
-		return kontozähler;
+		return false;
 	}
 
-	public String getName() {
-		return inhaber;
+	public String[] getKontobewegungen() {
+		String[] auflistung = new String[kontobewegungen.size()];
+
+		int i = 0;
+		for (Kontobewegung kb : kontobewegungen) {
+			auflistung[i++] = kb.toString();
 		}
 
+		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

@@ -7,36 +7,54 @@ import org.junit.jupiter.api.Test;
 class KontoTest {
 
 	@Test
-	void testKonto() {
+	void testKontoBasics() {
 		Konto k = new Konto("Müller", 0);
 		assertEquals("Müller", k.getInhaber());
 		assertEquals(1000, k.getKontonummer());
 		assertEquals(0, k.getKontostand());	
-
-		Konto k2 = new Konto("Mayer", 1);
-		assertEquals("Mayer", k2.getInhaber());
-		assertNotEquals(k.getKontonummer(), k2.getKontonummer());
-		assertEquals(1001, k2.getKontonummer());
-
-		k2.zahleEin(100, k2.getName());
-		assertEquals(100, k2.getKontostand());
-		// assertTrue(k2.zahleAus(50, "Test", "Einzahlung", "JUnit"));
-		assertNotEquals(50, k2.getKontostand());
-
-		// assertFalse(k2.zahleAus(500, "Test", "Einzahlung", "JUnit"));
-		assertNotEquals(50, k2.getKontostand());
 	}
 	
 	@Test
-	void testSaldo() {
-		Konto k = new Konto("Meier",1);
-		Konto k2 = new Konto("franz",2);
-		k.zahleEin(100, "Meier");
-		k.zahleEin(100, "Meier");
-		k.zahleEin(100, "Meier");
-		assertEquals(k.getSaldo(1), 100);
-		assertEquals(k.getSaldo(2),200);
+	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());
+		
+		k2.einzahlen(100, "Test", "Einzahlung", "JUnit");
+		assertEquals(100, k2.getKontostand());
+		
+		assertTrue(k2.auszahlen(50, "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"));
 	}
 
 }

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

@@ -1,29 +0,0 @@
-package de.hs_mannheim.informatik.bank.domain;
-
-import java.io.Serializable;
-import java.util.Date;
-
-public class Kontoauszug implements Serializable {
-
-	
-	private long kontostand, betrag;
-	private Date date = new Date();
-	private String auftraggeber;
-	
-	public Kontoauszug(long kontostand, long betrag, String auftraggeber) {
-		this.kontostand = kontostand;
-		this.betrag = betrag;
-		this.auftraggeber = auftraggeber;
-	}
-	
-	@Override
-	public String toString() {
-		return "Datum: " + date.toString() + " | Kontostand: " + kontostand + " | Betrag: "+ betrag + " | " 
-				+ "Auftraggeber: " + auftraggeber;
-	}
-
-	public long getBetrag() {
-		return betrag;
-		
-	}
-}

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

@@ -0,0 +1,32 @@
+package de.hs_mannheim.informatik.bank.domain;
+
+import java.io.Serializable;
+import java.util.Date;
+
+public class Kontobewegung implements Serializable {
+	private long betrag;
+	private Date datum;
+	private String betreff;
+	private String art;
+	private String auftraggeber;
+	
+	public Kontobewegung(long betrag, String betreff, String art, String auftraggeber) {
+		this.betrag = betrag;
+		this.betreff = betreff;
+		this.art = art;
+		this.auftraggeber = auftraggeber;
+		
+		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 + "]";
+	}
+	
+}

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

@@ -1,21 +1,18 @@
 package de.hs_mannheim.informatik.bank.facade;
 
-import java.io.Serializable;
 import java.util.Collection;
-import java.util.List;
 
 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.Kontoauszug;
-import persistence.BankSerializer;
+import de.hs_mannheim.informatik.bank.infrastructure.Persistenz;
 
-public class Banksystem implements Serializable {
+public class Banksystem {
 	private Bank bank;
 
 	public Banksystem(String bankname) throws Exception {
-		if (BankSerializer.sindDatenGespeichert(bankname))
-			this.bank = (Bank) BankSerializer.ladeBankDaten(bankname);
+		if (Persistenz.sindDatenGespeichert(bankname))
+			this.bank = (Bank) Persistenz.ladeBankDaten(bankname);
 		else
 			this.bank = new Bank(bankname);
 	}
@@ -23,7 +20,7 @@ public class Banksystem implements Serializable {
 	public int kontoAnlegen(String name, int auswahl) throws Exception {
 		int kontonummer = bank.addKonto(name, auswahl);
 		
-		BankSerializer.speichereBankDaten(this.bank, bank.getName());
+		Persistenz.speichereBankDaten(this.bank, bank.getName());
 
 		return kontonummer;
 	}
@@ -40,67 +37,59 @@ public class Banksystem implements Serializable {
 		return liste;
 	}
 
-	
-	public long zahleEin(Integer kontonummer, long einzahlung) throws Exception {
-		Konto konto = bank.findeKonto(kontonummer);
-		BankSerializer.speichereBankDaten(this.bank, bank.getName());
-		return bank.zahleEin(kontonummer, einzahlung);
-		
-	}
-	public long zahleAus(Integer kontonummer, long einzahlung) throws Exception {
-		BankSerializer.speichereBankDaten(this.bank, bank.getName());
-		return bank.zahleAus(kontonummer, einzahlung);
-	}
-	public long zahleAus2(Integer kontonummer, long einzahlung) throws Exception {
-		BankSerializer.speichereBankDaten(this.bank, bank.getName());
-		return bank.zahleAus2(kontonummer, einzahlung);
-	}
-	
-	public long getKontostand(Integer kontonummer) {
-		return bank.getKontostand(kontonummer);
-	}
-	
-	// Damit nicht auf nicht existierende Konten eingezahlt werden kann
-	public boolean existiertKonto(Integer kontonummer) {
-		return bank.existiertKonto(kontonummer);
-	}
-	
 	public String getBankname() {
 		return bank.getName();
 	}
 
-	public boolean überweisungBeauftragen(int startkonto, int zielkonto, long betrag, String verwendungszweck) {
-		Konto start = bank.existiertKonto2(startkonto);
-		Konto ziel = bank.existiertKonto2(zielkonto);
+	public long geldEinzahlen(int kontonummer, long betrag) throws Exception {
+		Konto konto = bank.findeKonto(kontonummer);
+		konto.einzahlen(betrag, "Einzahlung am Schalter", "Einzahlung", konto.getInhaber());
+
+		Persistenz.speichereBankDaten(this.bank, bank.getName());
+		
+		return konto.getKontostand();
+	}
+
+	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 erg;
+	}
+
+	public String[] erstelleKontoauszug(int kontonummer) {
+		Konto konto = bank.findeKonto(kontonummer);
+
+		return konto.getKontobewegungen();
+	}
+
+	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;
 	}
 
+	public long getKontostand(int kontonummer) {
+		Konto konto = bank.findeKonto(kontonummer);
 
-	public List<Kontoauszug> getAuszuge(Integer kontonummer) {
-		return bank.getAuszuge(kontonummer);
+		return konto.getKontostand();
 	}
 
-	public boolean instanceOfGirokonto(int kontonummer) {
-		Konto start = bank.existiertKonto2(kontonummer);
-		if(start instanceof Girokonto  ) {
-			return true;
-			
-		}
-		return false;
-		
-	}
-	public long getSaldo(int kontonummer, int anzahl) {
-		
-		return bank.getSaldo(kontonummer, anzahl);
+	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,57 +1,70 @@
 package de.hs_mannheim.informatik.bank.facade;
 
-import static org.junit.Assert.assertTrue;
-import static org.junit.jupiter.api.Assertions.*;
+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;
 
-import de.hs_mannheim.informatik.bank.domain.Konto;
-
+@TestMethodOrder(OrderAnnotation.class)
 class SystemTest {
+	private static Banksystem bs;
+	
+	@BeforeAll
+	static void initBanksystem() throws Exception  {
+		bs = new Banksystem("Testsystem");
+	}
 
 	@Test
-	void test() throws Exception {
-			Banksystem bs = new Banksystem("Testsystem");
-			
+	@Order(1)
+	void smokeTest() {
 		assertNotNull(bs);
 		assertEquals(0, bs.getKontenliste().length);
 		assertEquals("Testsystem", bs.getBankname());
-			
-			assertEquals(0,bs.getKontenliste().length );
-			
 	}
+	
 	@Test
-	void testKontoAnlegen() throws Exception {
-		Banksystem bs = new Banksystem("Testbank");
-		String name = "Peter";
-		int auswahl = 1;
-		bs.kontoAnlegen(name, auswahl);
-		assertNotEquals(0, bs.getKontenliste());
+	@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 testZahelEin() throws Exception {
-		Banksystem bs = new Banksystem("Testbank");
-		//int kontonummer = 1000;
+	@Order(3)
+	void persistenzTest() throws Exception {
+		int knr = bs.kontoAnlegen("Test2", 2);
+		int knr2 = bs.kontoAnlegen("Test3", 2);
 		
-		bs.kontoAnlegen("hans", 1);
-		long einzahlung = 500;
-//		Konto k = new Konto ("Hans",1);
-		bs.zahleEin(1001, einzahlung);
-		assertEquals( 500, bs.getKontostand(1000));
+		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 zahleAusTest () throws Exception {
-		Banksystem bs = new Banksystem("Testbank");
-//		Konto k = new Konto("Hans", 1);
-		bs.kontoAnlegen("hans", 1);
 	
-		int kontonummer = 1001;
-		long einzahlung = 500;
-		bs.zahleEin(1000, einzahlung);
-		bs.zahleAus(1000, 400);
-		assertEquals(100, bs.getKontostand(1000));
+	@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

@@ -0,0 +1,30 @@
+package de.hs_mannheim.informatik.bank.infrastructure;
+
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.FileOutputStream;
+import java.io.ObjectInputStream;
+import java.io.ObjectOutputStream;
+
+public class Persistenz {
+	private static final String BANK_DATEI = "-bank-data.ser";
+	
+	public static boolean sindDatenGespeichert(String name) {
+		return new File(name + BANK_DATEI).exists();
+	}
+
+	public static void speichereBankDaten(Object bank, String name) throws Exception {
+		ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(name + BANK_DATEI));
+		oos.writeObject(bank);
+		oos.close();
+	}
+	
+	public static Object ladeBankDaten(String name) throws Exception {
+		ObjectInputStream ois = new ObjectInputStream(new FileInputStream(name + BANK_DATEI));
+		Object bank = ois.readObject();
+		ois.close();
+		
+		return bank;
+	}
+
+}

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

@@ -1,77 +1,59 @@
 package de.hs_mannheim.informatik.bank.ui;
 
-import java.util.List;
 import java.util.Scanner;
 
-import de.hs_mannheim.informatik.bank.domain.Konto;
-import de.hs_mannheim.informatik.bank.domain.Kontoauszug;
 import de.hs_mannheim.informatik.bank.facade.Banksystem;
 
 public class UI {
 	private Banksystem bs;
 	Scanner sc = new Scanner(System.in);
 
-	public UI(Banksystem bs) throws Exception {
+	public UI(Banksystem bs) {
 		this.bs = bs;
 		hauptmenü();
 	}
 
-	private void hauptmenü() throws Exception {
+	private void hauptmenü() {
 		System.out.println("Willkommen bei der " + bs.getBankname() + "!");
-		mainloop: //
+
+		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 -> Auf Konto einzahlen");
-			System.out.println("4 -> Von Konto abheben");
-			System.out.println("5 -> Kontostand anzeigen");
-			System.out.println("6 -> Kontoauszug anzeigen");
-			System.out.println("7 -> Überweisung beauftragen");
-			System.out.println("8 -> Saldo");
+				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.println("9 -> Beenden");
 				System.out.println();
 
 				System.out.print("> ");
-			String input = sc.nextLine();
-			int inputNumber = 0;
-
-			if (!input.equalsIgnoreCase(""))
-				inputNumber = Integer.parseInt(input);
-
+				int input = Integer.parseInt(sc.nextLine());
 				System.out.println();
 
-			switch (inputNumber) {
-			case 1:
-				kontenAnzeigen();
-				break;
+				try {
+					switch(input) {
+					case 1: kontenAnzeigen(); break;
 					case 2: 
 						kontoAnlegen();
 						break;
-			case 3:
-				kontoEinzahlen();
-				break;
-			case 4:
-				kontoAbheben();
-				break;
-			case 5:
-				kontostandAnzeigen();
-				break;
-			case 6:
-				kontoauszugAnzeigen();
-				break;
-			case 7 : überweisungBeauftragen();break;
-			
-			case 8 : SaldoAnzeigen(); break;
-			
-			case 9:
-				break mainloop;
-			default:
-				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());
+				} 
+				System.out.println();
 			}
 
 		System.out.println("Auf Wiedersehen!");
@@ -92,107 +74,61 @@ public class UI {
 
 	private void kontoAnlegen() throws Exception {
 		System.out.println("Bitte den Namen des Kontoinhabers angeben: ");
-		String name = sc.nextLine(); // warum kein null check / empty check
+		String name = sc.nextLine();
 
 		System.out.println("Möchten Sie ein Sparkonto (1) oder ein Girokonto (2) anlegen?");
-		Integer auswahl = Integer.parseInt(sc.nextLine()); // warum kein null check / empty check
+		int auswahl = Integer.parseInt(sc.nextLine());
 
 		int kontonummer = bs.kontoAnlegen(name, auswahl);
-		System.out.print("Konto mit der Nummer " + kontonummer + " neu angelegt.");
-		System.out.println();
+		System.out.println("Konto mit der Nummer " + kontonummer + " neu angelegt.");
 	}
 
-	private void kontoEinzahlen() throws Exception {
-		// Konto ermitteln
-		System.out.print("Bitte die Kontonummer angeben: ");
-		Integer kontonummer = Integer.parseInt(sc.nextLine()); // warum kein null check / empty check
+	private void geldEinzahlen() throws Exception {
+		System.out.println("Geld einzahlen");
+		System.out.print("Bitte die gewünschte Kontonummer eingeben: ");
+		int kontonummer = Integer.parseInt(sc.nextLine());
 
-		if (!bs.existiertKonto(kontonummer)) {
-			System.out.print("Kein Konto mit dieser Kontonummer existiert.");
-			return;
-			// hier kann man auch wieder kontoEinzahlen aufrufen, für eine erneute
-			// Eingabemöglichkeit, dann muss aber vorher geprüft werden, ob Konten
-			// existieren.
+		// optional prüfen, ob Konto existiert
+
+		System.out.print("Bitte den gewünschten Betrag eingeben: ");
+		double betrag = Double.parseDouble(sc.nextLine());
+
+		long neuerKontostand = bs.geldEinzahlen(kontonummer, (long)betrag * 100);
+
+		System.out.printf("Einzahlung erfolgreich, neuer Kontostand = %.2f Euro", (neuerKontostand / 100.0));
 	}
 
-		// Betrag ermitteln
-		System.out.print("Bitte den einzuzahlenden Betrag angeben: ");
-		long betrag = Long.parseLong(sc.nextLine());
-		if (betrag <= 0) {
-			System.out.println("Du kannst mit diesem Betrag nicht einzahlen, bitte versuche nochmal");
-			return;
-		}
-		// Einzahlen
-		bs.zahleEin(kontonummer, betrag);
+	private void geldAuszahlen() throws Exception {
+		System.out.println("Geld auszahlen");
+		System.out.print("Bitte die gewünschte Kontonummer eingeben: ");
+		int kontonummer = Integer.parseInt(sc.nextLine());
 
-		System.out.print("Konto mit der Nummer " + kontonummer + " hat einen Kontostand von " + betrag );
-		System.out.println();
+		System.out.print("Bitte den gewünschten Betrag eingeben: ");
+		double betrag = Double.parseDouble(sc.nextLine());
+
+		boolean erfolgreich = bs.geldAuszahlen(kontonummer, (long)betrag * 100);
+
+		System.out.printf("Auszahlung" + ((!erfolgreich)? " nicht" : "" )+ " erfolgreich. ");
+		System.out.printf("Neuer Kontostand = %.2f Euro.", (bs.getKontostand(kontonummer) / 100.0));	
 	}
 
-	private void kontoAbheben() throws Exception {
-		// Konto ermitteln
-		System.out.println("Bitte die Kontonummer angeben: ");
-		Integer kontonummer = Integer.parseInt(sc.nextLine());
+	private void kontoauszugDrucken() {
+		System.out.print("Bitte die gewünschte Kontonummer für den Auszug eingeben: ");
+		int kontonummer = Integer.parseInt(sc.nextLine());
 
-		if (!bs.existiertKonto(kontonummer)) {
-			System.out.println("Kein Konto mit dieser Kontonummer existiert.");
-			return;
+		// in echt auf einem Drucker
+		System.out.println("Auszug für Konto " + kontonummer);
+		String[] kontobewegungen = bs.erstelleKontoauszug(kontonummer);
+
+		if (kontobewegungen.length > 0)
+			for (String kb : kontobewegungen) {
+				System.out.println(kb);
 			}
-		// Betrag ermitteln
-		System.out.println("Bitte den abzuhebenden Betrag angeben: ");
-		long betrag = Long.parseLong(sc.nextLine());
-		if (betrag <= 0) {
-			System.out.println("Du kannst mit diesem Betrag nicht auszahlen, bitte versuche nochmal");
-			return;
+		else
+			System.out.println("Noch keine Kontobewegungen.");
 	}
 
-		// Abheben
-		if(bs.instanceOfGirokonto(kontonummer)) {
-			bs.zahleAus2(kontonummer, betrag);
-			System.out.println("Sie haben ihr konto Überzogen "+(bs.getKontostand(1000))+" Euro");
-			return;
-			
-		}
-		bs.zahleAus(kontonummer, betrag);
-		System.out.println("Konto mit der Nummer " + kontonummer + " hat einen Kontostand von " + bs.getKontostand(kontonummer) );
-	}
-
-	private void kontoauszugAnzeigen() {
-		// Konto ermitteln
-		System.out.println("Bitte die Kontonummer angeben: ");
-		Integer kontonummer = sc.nextInt();
-
-		if (!bs.existiertKonto(kontonummer)) {
-			System.out.println("Kein Konto mit dieser Kontonummer existiert.");
-			return;
-		}
-
-		List<Kontoauszug> auszuge = bs.getAuszuge(kontonummer);
-
-		for (Kontoauszug auszug : auszuge)
-			System.out.println(auszug);
-	}
-
-	private void kontostandAnzeigen() {
-		// Konto ermitteln
-		System.out.println("Bitte die Kontonummer angeben: ");
-		Integer kontonummer = sc.nextInt();
-
-		System.out.println(
-				"Konto mit der Nummer " + kontonummer + " hat einen Kontostand von " + bs.getKontostand(kontonummer));
-		
-		
-	}
-	private void SaldoAnzeigen() {
-		System.out.println("Bitte die Kontonummer angeben: ");
-		Integer kontonummer = sc.nextInt();
-		System.out.println("Welche Anzahl wollen Sie sehen?");
-		Integer anzahl = sc.nextInt();
-		System.out.println(bs.getSaldo(kontonummer, anzahl));
-	}
-	
-	
-	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());
 
@@ -210,4 +146,15 @@ 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));
+	}
+
 }

Bank-Beispiel/src/persistence/BankSerializer.java

@@ -1,49 +0,0 @@
-package persistence;
-
-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;
-
-import de.hs_mannheim.informatik.bank.domain.Bank;
-
-public class BankSerializer {
-
-	private static final String BANK_DATEI = "bank.ser";
-	private static final String ZÄHLER_DATEI = "zähler.ser";
-	
-	public static boolean sindDatenGespeichert(String bankname) {
-		return new File(BANK_DATEI).exists();
-	}
-
-	public static void speichereBankDaten(Object bank, String name) throws Exception {
-		ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(BANK_DATEI));
-		oos.writeObject(bank);
-		oos.close();
-	}
-	
-	public static void speichereKontozähler(int kontozähler) throws IOException {
-		ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(ZÄHLER_DATEI));
-		oos.writeInt(kontozähler);
-		oos.close();
-	}
-	
-	public static Object ladeBankDaten(String name) 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/persistence/BankSerializer2.java

@@ -1,44 +0,0 @@
-package persistence;
-
-import java.io.File;
-import java.io.FileInputStream;
-import java.io.FileOutputStream;
-import java.io.IOException;
-import java.io.InputStream;
-import java.io.OutputStream;
-
-public class BankSerializer2 {
-	
-	public static void main(String[] args) {
-
-		File inFile = new File("C:/Users/mehta/OneDrive/Masaüstü/Input");
-		File outFile = new File("C:/Users/mehta/OneDrive/Masaüstü/Input");
-
-		BankSerializer2.inFileStream(inFile, outFile);
-
-	}
-
-	public static void inFileStream(File in, File out) {
-		InputStream inStream = null;
-		OutputStream outStream = null;
-
-		try {
-			inStream = new FileInputStream(in);
-			outStream = new FileOutputStream(out);
-
-			int input;
-			while ((input = inStream.read()) != -1) {
-				outStream.write(input);
-			}
-		} catch (IOException e) {
-			e.printStackTrace();
-		} finally {
-			try {
-				inStream.close();
-				outStream.close();
-			} catch (IOException e) {
-				e.printStackTrace();
-			}
-		}
-	}
-}