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
+*.ser
-/zähler.ser
+/.DS_Store

Bank-Beispiel/.project

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

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

@@ -1,14 +1,11 @@
 package de.hs_mannheim.informatik.bank;
 
-import de.hs_mannheim.informatik.bank.domain.Aktien;
 import de.hs_mannheim.informatik.bank.facade.Banksystem;
 import de.hs_mannheim.informatik.bank.ui.UI;
 
 public class Main {
 
 	public static void main(String[] args) throws Exception {
-		//hier die Firmen
-		
 		Banksystem bs = new Banksystem("Spaßkasse Mannheim");
 		UI ui = new UI(bs);
 	}

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

@@ -1,56 +1,30 @@
 package de.hs_mannheim.informatik.bank.domain;
 
 import java.io.Serializable;
-import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashMap;
 
 public class Bank implements Serializable {
 	private String name;
 	private HashMap<Integer, Konto> konten = new HashMap<>();
-	private HashMap<Integer, Kunde> kunden = new HashMap<>();
-	private HashMap<Integer, Depot> depots = new HashMap<>();
 	private int kontozähler;
-	private int kundenzähler;
 	
 	public Bank(String name) {
 		this.name = name;
 		this.kontozähler = -1;
 	}
 	
-	public int addKonto(Kunde kunde, int auswahl) {
+	public int addKonto(String name, int auswahl) {
 		Konto k;
-		Depot d;
 
-		switch (auswahl) {
+		if (auswahl == 1) 
-		case 1:
+			k = new Konto(name, ++kontozähler);
-			k = new Konto(++kontozähler, kunde);
-			konten.put(k.getKontonummer(), k);
-			kunde.addKonto(k);
-			return k.getKontonummer();
-			//break;
-		case 2:
-			k = new Girokonto(++kontozähler, kunde);
-			konten.put(k.getKontonummer(), k);
-			kunde.addKonto(k);
-			return k.getKontonummer();
-			//break;
-		case 3:
-			d = new Depot(++kontozähler, kunde);
-			depots.put(d.getKontonummer(), d);
-			kunde.addDepot(d);
-			return d.getKontonummer();
-			//break;
-		}
-		/*if (auswahl == 1) 
-			k = new Konto(++kontozähler, kunde);
 		else 
-			k = new Girokonto(++kontozähler, kunde);*/
+			k = new Girokonto(name, ++kontozähler);
 		
-		/*konten.put(k.getKontonummer(), k);
+		konten.put(k.getKontonummer(), k);
-		kunde.addKonto(k);*/
 		
-		return -1;
+		return k.getKontonummer();
 	}
 	
 	public String getName() {
@@ -60,36 +34,9 @@ public class Bank implements Serializable {
 	public Collection<Konto> getKontenliste() {
 		return konten.values();
 	}
-	public Collection<Kunde> getKundenliste() {
-		return kunden.values();
-	}
-	public Collection<Depot> getDepotliste(){
-		return depots.values();
-	}
-
 
 	public Konto findeKonto(int kontonummer) {
 		return konten.get(kontonummer);
 	}
 	
-	public int getKundenzähler() {
-		return kundenzähler;
-	}
-
-	public int addKunde(String vorname, String nachname, String wohnort, int alter) {
-		Kunde k = new Kunde(vorname, nachname,wohnort,alter,++kundenzähler);
-		kunden.put(k.getKundennummer(), k);
-		return k.getKundennummer();
-		
-	}
-
-	public Kunde findeKunde(int kundennummer) {
-		return kunden.get(kundennummer);
-		
-	}
-	public Depot findeDepot(int kontonummer) {
-		return depots.get(kontonummer);
-	}
-	
-
 }

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

@@ -5,8 +5,8 @@ import java.io.Serializable;
 public class Girokonto extends Konto implements Serializable {
 	private long dispo = 100000;
 
-	public Girokonto(int kontozähler, Kunde kunde) {
+	public Girokonto(String inhaber, int kontozähler) {
-		super(kontozähler, kunde);
+		super(inhaber, kontozähler);
 	}
 	
 	public boolean überweise(Girokonto ziel, long betrag, String zweck) {

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

@@ -7,14 +7,12 @@ public class Konto implements Serializable {
 	private int nummer;
 	protected long stand = 0;
 	private String inhaber;
-	private Kunde kunde;
 
 	protected ArrayList<Kontobewegung> kontobewegungen;
 
-	public Konto(int kontozähler, Kunde kunde) {
+	public Konto(String inhaber, int kontozähler) {
 		nummer = 1000 + kontozähler;
-
+		this.inhaber = inhaber;
-		this.kunde = kunde;
 
 		this.kontobewegungen = new ArrayList<>();
 	}
@@ -25,11 +23,7 @@ public class Konto implements Serializable {
 
 	@Override
 	public String toString() {
-		return "Konto [nummer=" + nummer + ", kunde=" + kunde.toString() + "]";
+		return "Konto [nummer=" + nummer + ", inhaber=" + inhaber + "]";
-	}
-
-	public Kunde getKunde() {
-		return kunde;
 	}
 
 	public String getInhaber() {
@@ -58,21 +52,6 @@ public class Konto implements Serializable {
 		return false;
 	}
 
-	/*
-	 * public boolean auszahlen(long betrag, String zweck, String art, String
-	 * auftraggeber) { try { long checkStand = stand - betrag; boolean kleinerNull =
-	 * (checkStand <= 0);
-	 * 
-	 * if (kleinerNull == false) { kontobewegungen.add(new Kontobewegung(betrag *
-	 * -1, zweck, art, auftraggeber)); }
-	 * 
-	 * } catch (Exception e) { System.out.println("nope");
-	 * 
-	 * return false;
-	 * 
-	 * } return true; }
-	 */
-
 	public String[] getKontobewegungen() {
 		String[] auflistung = new String[kontobewegungen.size()];
 

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

@@ -3,10 +3,8 @@ package de.hs_mannheim.informatik.bank.facade;
 import java.util.Collection;
 
 import de.hs_mannheim.informatik.bank.domain.Bank;
-import de.hs_mannheim.informatik.bank.domain.Depot;
 import de.hs_mannheim.informatik.bank.domain.Girokonto;
 import de.hs_mannheim.informatik.bank.domain.Konto;
-import de.hs_mannheim.informatik.bank.domain.Kunde;
 import de.hs_mannheim.informatik.bank.infrastructure.Persistenz;
 
 public class Banksystem {
@@ -19,12 +17,8 @@ public class Banksystem {
 			this.bank = new Bank(bankname);
 	}
 
-	public int kontoAnlegen(Kunde kunde, int auswahl) throws Exception {
+	public int kontoAnlegen(String name, int auswahl) throws Exception {
-		int kontonummer = bank.addKonto(kunde, auswahl);
+		int kontonummer = bank.addKonto(name, auswahl);
-		//if (kontonummer == -1) {
-		//	throw new Exception("Ungültige Auswahl");
-
-		//}
 		
 		Persistenz.speichereBankDaten(this.bank, bank.getName());
 
@@ -43,29 +37,6 @@ public class Banksystem {
 		return liste;
 	}
 
-	public String[] getKundenliste() {
-		Collection<Kunde> kunden = bank.getKundenliste();
-		String[] liste = new String[kunden.size()];
-
-		int i = 0;
-		for (Kunde k : kunden) {
-			liste[i++] = k.toString();
-		}
-
-		return liste;
-	}
-	public String[] getDepotliste() {
-		Collection<Depot> depots = bank.getDepotliste();
-		String[] liste = new String[depots.size()];
-
-		int i = 0;
-		for (Depot d : depots) {
-			liste[i++] = d.toString();
-		}
-
-		return liste;
-	}
-
 	public String getBankname() {
 		return bank.getName();
 	}
@@ -95,8 +66,7 @@ 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 {
-			throws Exception {
 		Konto start = bank.findeKonto(startkonto);
 		Konto ziel = bank.findeKonto(zielkonto);
 
@@ -122,50 +92,4 @@ public class Banksystem {
 		return konto.berechneSaldo(anzahl);
 	}
 
-	public int kundeAnlegen(String vorname, String nachname, String wohnort, int alter) {
-		int kundennummer = bank.addKunde(vorname, nachname, wohnort, alter);
-		return kundennummer;
-
-	}
-
-	public Kunde findeKunde(int kundennummer) {
-		Kunde k = bank.findeKunde(kundennummer);
-		return k;
-
-	}
-
-	public boolean aktienKaufen(int kontonummer, int auswahl, int anzahl) {
-		Depot depot = bank.findeDepot(kontonummer);
-		boolean erfolgreich = depot.aktienKaufen(auswahl,anzahl);
-		
-		return erfolgreich;
-		
-	}
-
-	public long geldEinzahlenDepot(int kontonummer, long betrag) {
-		Depot depot = bank.findeDepot(kontonummer);
-		depot.geldEinzahlen(betrag);
-		return depot.getKontostand();
-	}
-
-	public boolean geldAuszahlenDepot(int kontonummer, long betrag) {
-		Depot depot = bank.findeDepot(kontonummer);
-		boolean erfolgreich = depot.geldAuszahlen(betrag);
-		
-		return erfolgreich;
-	}
-
-	public String[] aktienAnzeigen(int kontonummer) {
-		Depot depot = bank.findeDepot(kontonummer);
-		String [] aktien = depot.getAktien();
-		
-		return aktien;
-		
-	}
-
-	public boolean aktienVerkaufen(int kontonummer, int auswahl, int anzahl) {
-		Depot depot = bank.findeDepot(kontonummer);
-		return depot.aktienVerkaufen(auswahl, anzahl);
-	}
-
 }

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

@@ -2,7 +2,6 @@ package de.hs_mannheim.informatik.bank.ui;
 
 import java.util.Scanner;
 
-import de.hs_mannheim.informatik.bank.domain.Kunde;
 import de.hs_mannheim.informatik.bank.facade.Banksystem;
 
 public class UI {
@@ -17,24 +16,20 @@ public class UI {
 	private void hauptmenü() {
 		System.out.println("Willkommen bei der " + bs.getBankname() + "!");
 
-		mainloop: while (true) {
+		mainloop: 
+			while (true) {
 				System.out.println();
 				System.out.println("--------");
 				System.out.println("Hauptmenü");
 				System.out.println("1 -> Konten anzeigen");
-			System.out.println("2 -> Konto oder Depot anlegen");
+				System.out.println("2 -> Konto anlegen");
-			System.out.println("3 -> Geld einzahlen (Giro- oder Sparkonto)");
+				System.out.println("3 -> Geld einzahlen");
-			System.out.println("4 -> Geld auszahlen (Giro- oder Sparkonto)");
+				System.out.println("4 -> Geld auszahlen");
-			System.out.println("5 -> Geld auf ein Depot einzahlen ");
+				System.out.println("5 -> Kontoauszug drucken");
-			System.out.println("6 -> Geld von einem Depot auszahlen");
+				System.out.println("6 -> Überweisung beauftragen");
-			System.out.println("7 -> Kontoauszug drucken");
+				System.out.println("7 -> Saldo abfragen");
-			System.out.println("8 -> Überweisung beauftragen");
+
-			System.out.println("9 -> Saldo abfragen");
+				System.out.println("9 -> Beenden");
-			System.out.println("10 -> Aktien kaufen");
-			System.out.println("11 -> Aktien verkaufen");
-			System.out.println("12 -> Aktien von einem Depot anzeigen");
-			System.out.println("13 -> Kunde anlegen");
-			System.out.println("14 -> Beenden");
 				System.out.println();
 
 				System.out.print("> ");
@@ -43,47 +38,16 @@ public class UI {
 
 				try {
 					switch(input) {
-				case 1:
+					case 1: kontenAnzeigen(); break;
-					kontenAnzeigen();
-					break;
 					case 2: 
 						kontoAnlegen();
 						break;
-				case 3:
+					case 3: geldEinzahlen(); break;
-					geldEinzahlen();
+					case 4: geldAuszahlen(); break;
-					break;
+					case 5: kontoauszugDrucken(); break;
-				case 4:
+					case 6: überweisungBeauftragen(); break;
-					geldAuszahlen();
+					case 7: saldoAbfragen(); break;
-					break;
+					case 9: break mainloop;
-				case 5:
-					geldEinzahlennDepot();
-					break;
-				case 6:
-					geldAuszahlenDepot();
-					break;
-				case 7:
-					kontoauszugDrucken();
-					break;
-				case 8:
-					überweisungBeauftragen();
-					break;
-				case 9:
-					saldoAbfragen();
-					break;
-				case 10:
-					aktienKaufen();
-					break;
-				case 11:
-					aktienVerkaufen();
-					break;
-				case 12:
-					aktienAnzeigen();
-					break;
-				case 13:
-					kundeAnlegen();
-					break;
-				case 14:
-					break mainloop;
 					}
 
 				} catch (Exception e) {
@@ -106,40 +70,18 @@ public class UI {
 		} else {
 			System.out.println("Bisher keine Konten angelegt.");
 		}
-		String[] depots = bs.getDepotliste();
-		if (depots.length > 0) {
-			System.out.println("Folgende Depots sind aktuell verfügbar:");
-			for (String s : depots) {
-				System.out.println(s);
-			}
-		} else {
-			System.out.println("Bisher keine Depots angelegt.");
-		}
 	}
 
 	private void kontoAnlegen() throws Exception {
-		String[] kunden = bs.getKundenliste();
+		System.out.println("Bitte den Namen des Kontoinhabers angeben: ");
-		if (kunden.length == 0) {
+		String name = sc.nextLine();
-			System.out.println("Es gibt noch keine Kunden, bitte Kunde anlegen.");
-			kundeAnlegen();
-		}
 
-		// System.out.println("Bitte den Namen des Kontoinhabers angeben: ");
+		System.out.println("Möchten Sie ein Sparkonto (1) oder ein Girokonto (2) anlegen?");
-		// String name = sc.nextLine();
-
-		System.out.println("Möchten Sie ein Sparkonto (1), ein Girokonto (2) oder ein Wertpapier-Depot (3) anlegen?");
 		int auswahl = Integer.parseInt(sc.nextLine());
-		System.out.println("Bitte geben Sie eine Kundennummer ein: ");
+
-		int kundennummer = Integer.parseInt(sc.nextLine());
+		int kontonummer = bs.kontoAnlegen(name, auswahl);
-		Kunde k = bs.findeKunde(kundennummer);
-		if (k == null) {
-			System.out.println("Kunde existiert nicht, bitte neu versuchen.");
-			kontoAnlegen();
-		} else {
-			int kontonummer = bs.kontoAnlegen(k, auswahl);
 		System.out.println("Konto mit der Nummer " + kontonummer + " neu angelegt.");
 	}
-	}
 
 	private void geldEinzahlen() throws Exception {
 		System.out.println("Geld einzahlen");
@@ -215,105 +157,4 @@ public class UI {
 		System.out.printf("Der Saldo nach %d Kontobewegungen beträgt %.2f Euro.%n", anzahl, (saldo/100d));
 	}
 
-	private void kundeAnlegen() {
-		System.out.println("Vorname");
-		String vorname = sc.nextLine();
-		System.out.println("Nachname");
-		String nachname = sc.nextLine();
-		System.out.println("Wohnort");
-		String wohnort = sc.nextLine();
-		System.out.println("Alter");
-		int alter = Integer.parseInt(sc.nextLine());
-		int kundennummer = bs.kundeAnlegen(vorname, nachname, wohnort, alter);
-		System.out.println("Der Kunde wurde angelegt mit der KN: " + kundennummer);
-
-	}
-
-	private void aktienKaufen() {
-		System.out.println("Für welches Depot wollen Sie Aktien kaufen? Bitte Kontonummer angeben: ");
-		int kontonummer = Integer.parseInt(sc.nextLine());
-		System.out.println(
-				"Für welche Firma wollen Sie Aktien ewerben? Zur Auswahl stehen (1) John Deere, (2) Hochschule Mannheim oder (3) Bundeswehr. ");
-		int auswahl = Integer.parseInt(sc.nextLine());
-		System.out.println("Wie viele Aktien wollen Sie kaufen? ");
-		int anzahl = Integer.parseInt(sc.nextLine());
-
-		boolean erfolgreich = bs.aktienKaufen(kontonummer, auswahl, anzahl);
-		if (erfolgreich == true)
-			System.out.println("Aktienkauf war erfolgreich. ");
-		else
-			System.out.println("Aktienkauf war nicht erfolgreich. ");
-
-	}
-
-	private void geldEinzahlennDepot() {
-		System.out.println("Bitte Kontonummer vom Depot eingeben: ");
-		int kontonummer = Integer.parseInt(sc.nextLine());
-		System.out.println("Wie viel möchten Sie einzahlen? ");
-		long betrag = Long.parseLong(sc.nextLine());
-
-		long neuerStand = bs.geldEinzahlenDepot(kontonummer, betrag);
-		System.out.println("Der neue Stand auf dem Depot beträgt: " + neuerStand);
-
-	}
-
-	private void geldAuszahlenDepot() {
-		System.out.println("Bitte Kontonummer vom Depot eingeben: ");
-		int kontonummer = Integer.parseInt(sc.nextLine());
-		System.out.println("Wie viel möchten Sie auszahlen? ");
-		long betrag = Long.parseLong(sc.nextLine());
-
-		boolean erfolgreich = bs.geldAuszahlenDepot(kontonummer, betrag);
-
-	}
-
-	private void aktienVerkaufen() {
-		System.out.println("Von welchem Depot wollen Sie Aktien verkaufen? Bitte Kontonummer eingeben: ");
-		int kontonummer = Integer.parseInt(sc.nextLine());
-
-		System.out.println("Folgende Aktien sind auf dem Konto: ");
-		String[] aktien = bs.aktienAnzeigen(kontonummer);
-
-		for (int i = 0; i <= aktien.length - 1; i++) {
-			System.out.print(aktien[i] + " ");
-		}
-
-		System.out.println(
-				"Welche Aktien wollen Sie verkaufen? Zur Auswahl stehen (1) John Deere, (2) Hochschule Mannheim oder (3) Bundeswehr.");
-		int auswahl = Integer.parseInt(sc.nextLine());
-
-		System.out.println("Wie viele wollen Sie verkaufen? ");
-		int anzahl = Integer.parseInt(sc.nextLine());
-		boolean erfolgreich = bs.aktienVerkaufen(kontonummer, auswahl, anzahl);
-		if (erfolgreich == true) {
-			System.out.println("Der Verkauf war erfolgreich. ");
-		} else {
-			System.out.println("Der Verkauf war leider nicht erfolgreich. ");
-		}
-
-	}
-
-	private void aktienAnzeigen() {
-		System.out.println("Von welchem Depot möchten Sie die Aktien sehen? Bitte Kontonummer angeben: ");
-		int kontonummer = Integer.parseInt(sc.nextLine());
-		String[] aktien = bs.aktienAnzeigen(kontonummer);
-		int johnDeereCounter = 0;
-		int hsmaCounter = 0;
-		int bundeswehrCounter = 0;
-
-		for (int i = 0; i <= aktien.length - 1; i++) {
-			if (aktien[i].equals("JohnDeere")) {
-				johnDeereCounter++;
-			} else if (aktien[i].equals("HSMA")) {
-				hsmaCounter++;
-			} else if (aktien[i].equals("Bundeswehr")) {
-				bundeswehrCounter++;
-			}
-		}
-		System.out.println("JohnDeere = " + johnDeereCounter);
-		System.out.println("HSMA = " + hsmaCounter);
-		System.out.println("Bundeswehr = " + bundeswehrCounter);
-
-	}
-
 }