first test check second NullPointException dont now why

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

@@ -1,15 +1,13 @@
 package de.hs_mannheim.informatik.bank;
 
 import de.hs_mannheim.informatik.bank.facade.Banksystem;
-import de.hs_mannheim.informatik.bank.gui.GUI;
 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);
+		UI ui = new UI(bs);
-		GUI gui = new GUI(bs);
 	}
 
 }

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

@@ -1,7 +1,6 @@
 package de.hs_mannheim.informatik.bank.domain;
 
 import java.io.Serializable;
-import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashMap;
 
@@ -10,6 +9,8 @@ import org.hamcrest.core.IsInstanceOf;
 import de.hs_mannheim.informatik.bank.facade.Banksystem;
 import de.hs_mannheim.informatik.bank.ui.UI;
 
+
+
 public class Bank implements Serializable {
 	private String name;
 	private HashMap<Integer, Konto> konten = new HashMap<>();
@@ -22,6 +23,8 @@ public class Bank implements Serializable {
 		this.kontozähler = -1;
 	}
 	
+
+	
 	public void kaufeAktien(int kontonummer, int idAktien) {
 		Konto kd = konten.get(kontonummer);
 		Aktien ak = wertpapiere.get(idAktien);
@@ -33,30 +36,21 @@ public class Bank implements Serializable {
 		} 
 	}
 	
-	public int addKonto(String name, int auswahl, int id) throws KundeNullException {
+	public int addKonto(String name, int auswahl , int id) {
 		Konto k;
 
 		if (auswahl == 1) {
 			k = new Konto(name, ++kontozähler);
 			Kunde ku = kunden.get(id);
-			if (ku == null) {
+			k.setKunde(ku);}
-				throw new KundeNullException();
+		else if(auswahl == 2) {
-			}
-			k.setKunde(ku.getId());
-		} else if (auswahl == 2) {
 			k = new Girokonto(name, ++kontozähler);
 			Kunde ku = kunden.get(id);
-			if (ku == null) {
+			k.setKunde(ku);
-				throw new KundeNullException();
-			}
-			k.setKunde(ku.getId());
 		} else {
 			k = new Depot(name, ++kontozähler);
 			Kunde ku = kunden.get(id);
-			if (ku == null) {
+			k.setKunde(ku);
-				throw new KundeNullException();
-			}
-			k.setKunde(ku.getId());
 		}
 		konten.put(k.getKontonummer(), k);
 		
@@ -83,8 +77,7 @@ public class Bank implements Serializable {
 	public void kontoZuKunde(int kontonummer, int id) {
 		Kunde ku = kunden.get(id);
 		Konto k = konten.get(kontonummer);
-		k.setKunde(ku.getId());
+		k.setKunde(ku);
-		ku.kundeHatKonto(k.getKontonummer());
 	}
 
 	public Collection<Kunde> getKundenListe() {
@@ -96,24 +89,10 @@ public class Bank implements Serializable {
 		wertpapiere.put(ak.getId(), ak);
 	}
 
+
+
 	public Collection<Aktien> getAktienListe() {
 		return wertpapiere.values();
 	}
 	
-	public Kunde findeKunde(int k) {
-		Kunde kunde = kunden.get(k);
-		return kunde;
-	}
-
-	public ArrayList<Konto> getKontenFromKunde(int kunde) {
-		Kunde k = kunden.get(kunde);
-		ArrayList<Konto> kontenKunde = new ArrayList<>();
-		ArrayList<Integer> kontonKunde = k.getKontennummern();
-		for (int i : kontonKunde) {
-			Konto ku = konten.get(i);
-			kontenKunde.add(ku);
-		}
-		return kontenKunde;
-	}
-
 }

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

@@ -35,7 +35,4 @@ public class Depot extends Girokonto implements Serializable {
 			
 		return false;
 	}
-	public String toString() {
-		return "Depot-"+super.helpString();
-	}
 }

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

@@ -34,10 +34,6 @@ public class Girokonto extends Konto implements Serializable {
 		return false;
 	}
 	
-	
-	public String helpString() {
-		return super.toString();
-	}
 	@Override
 	public String toString() {
 		return "Giro-" + super.toString();

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

@@ -7,12 +7,13 @@ public class Konto implements Serializable {
 	private int nummer;
 	protected long stand = 0;
 	private String inhaber;
-	protected int kundenid;
+	protected Kunde kunde;
 	protected ArrayList<Kontobewegung> kontobewegungen;
 
 	public Konto(String inhaber, int kontozähler) {
 		nummer = 1000 + kontozähler;
 		this.inhaber = inhaber;
+
 		this.kontobewegungen = new ArrayList<>();
 	}
 
@@ -39,16 +40,16 @@ public class Konto implements Serializable {
 		kontobewegungen.add(new Kontobewegung(betrag, zweck, art, auftraggeber));
 	}
 
-	public boolean auszahlen(long betrag, String zweck, String art, String auftraggeber) throws NotEnoughMoneyException {
+	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;
-		} else {
-			throw new NotEnoughMoneyException();
 		}
+
+		return false;
 	}
 
 	public String[] getKontobewegungen() {
@@ -72,8 +73,8 @@ public class Konto implements Serializable {
 		return saldo;
 	}
 	
-	public void setKunde(int kid) {
+	public void setKunde(Kunde k) {
-		this.kundenid = kid;
+		this.kunde = k;
 	}
 
 }

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

@@ -2,7 +2,6 @@ package de.hs_mannheim.informatik.bank.domain;
 
 import static org.junit.jupiter.api.Assertions.*;
 
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 class KontoTest {
@@ -26,7 +25,6 @@ class KontoTest {
 		testKontoEinUndAuszahlungUndSaldo(k, k2);
 	}
 	
-	
 	private void testKontoEinUndAuszahlungUndSaldo(Konto k, Konto k2) {
 		assertEquals("Mayer", k2.getInhaber());
 		assertNotEquals(k.getKontonummer(), k2.getKontonummer());

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

@@ -10,7 +10,7 @@ public class Kunde implements Serializable{
 	private String name;
 	private int alter;
 	private String wohnort;
-	private ArrayList<Integer> kontennummern;
+	private ArrayList<Konto> konten;
 
 	public Kunde(String vorname, String name, int alter, String wohnort) {
 		this.vorname = vorname;
@@ -18,24 +18,40 @@ public class Kunde implements Serializable{
 		this.alter = alter;
 		this.wohnort = wohnort;
 		this.id++;
-		kontennummern = new ArrayList<>();
 	}
 
-	public void kundeHatKonto(int k) {
+	public String getVorname() {
-		kontennummern.add(k);
+		return vorname;
+	}
+
+	public String getName() {
+		return name;
+	}
+
+	public int getAlter() {
+		return alter;
+	}
+
+	public String getWohnort() {
+		return wohnort;
+	}
+
+	public void kundeHatKonte(Konto k) {
+		konten.add(k);
 	}
 
 	public int getId() {
 		return this.id;
 	}
 
-	public ArrayList<Integer> getKontennummern() {
+	public ArrayList<Konto> getKonten() {
-		return kontennummern;
+		return konten;
 	}
 
 	@Override
 	public String toString() {
-		return "Vorname: " + this.vorname + " Name: " + this.name +" Alter: " + this.alter + " Wohnort: "+ this.wohnort;
+		return "Vorname: " + this.vorname + " Name: " + this.name + " Alter: " + this.alter + " Wohnort: "
+				+ this.wohnort;
 	}
 
 }

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

@@ -1,17 +0,0 @@
-package de.hs_mannheim.informatik.bank.domain;
-
-public class KundeNullException extends Exception {
-	public KundeNullException() {}
-	
-	public KundeNullException(String message) {
-		super(message);
-	}
-	
-	public KundeNullException(Throwable cause) {
-		super(cause);
-	}
-	
-	public KundeNullException(String message, Throwable cause) {
-		super(message, cause);
-	}
-}

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

@@ -0,0 +1,38 @@
+package de.hs_mannheim.informatik.bank.domain;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import java.util.ArrayList;
+
+import org.junit.jupiter.api.Test;
+
+class KundeTest {
+
+	@Test
+	void testKontoBasic() {
+		Kunde k = new Kunde("TestVor","TestNam",20,"TestHeim");
+		assertEquals(1, k.getId());
+		assertEquals("TestVor", k.getVorname());
+		assertEquals("TestNam", k.getName());
+		assertEquals(20, k.getAlter());
+		assertEquals("TestHeim", k.getWohnort());
+
+	}
+	
+	@Test
+	void testKundeHatKonto() {
+		Kunde ku = new Kunde("TestVor","TestNam",20,"TestHeim");
+		Konto ko = new Konto("TestKonto",1);
+		Konto gi = new Girokonto("TestGiro",2);
+		Konto de = new Depot("Testdep",3);
+		ku.kundeHatKonte(ko);
+		ku.kundeHatKonte(gi);
+		ku.kundeHatKonte(de);
+		ArrayList<Konto> konten = new ArrayList<>();
+		konten.add(ko);
+		konten.add(gi);
+		konten.add(de);
+		assertTrue(konten.equals(ku.getKonten()));
+	}
+
+}

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

@@ -1,20 +0,0 @@
-package de.hs_mannheim.informatik.bank.domain;
-
-public class NotEnoughMoneyException extends IllegalArgumentException {
-	
-	public NotEnoughMoneyException() {
-	}
-
-	public NotEnoughMoneyException(String message) {
-		super(message);
-	}
-
-	public NotEnoughMoneyException(Throwable cause) {
-		super(cause);
-	}
-
-	public NotEnoughMoneyException(String message, Throwable cause) {
-		super(message, cause);
-	}
-
-}

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

@@ -1,7 +1,6 @@
 package de.hs_mannheim.informatik.bank.facade;
 
 import java.io.InvalidClassException;
-import java.util.ArrayList;
 import java.util.Collection;
 
 import de.hs_mannheim.informatik.bank.domain.Aktien;
@@ -104,10 +103,6 @@ public class Banksystem {
 		bank.kundeAnlegen( vorname,  name,  alter,  wh);
 		Persistenz.speichereBankDaten(this.bank, bank.getName());
 	}
-	public Collection<Kunde> getKunden(){
-		return bank.getKundenListe();
-	}
-	
 
 	public String[] getKundenListe() {
 		Collection<Kunde> kunden = bank.getKundenListe();
@@ -146,17 +141,4 @@ public class Banksystem {
 		bank.kaufeAktien(kontonummer,aktienId);
 		Persistenz.speichereBankDaten(this.bank, bank.getName());
 	}
-
-	public String[] getKundenKontos(int kunde) {
-		ArrayList<Konto> konten = bank.getKontenFromKunde(kunde);
-		String[] liste = new String[konten.size()];
-		
-		int i = 0;
-		for (Konto k : konten) {
-			liste[i++] = k.toString(); 
-		}
-
-		return liste;
-		
-	}
 }

Bank-Beispiel/src/de/hs_mannheim/informatik/bank/gui/GUI.java

@@ -1,125 +0,0 @@
-package de.hs_mannheim.informatik.bank.gui;
-
-import java.awt.Color;
-
-import javax.swing.DefaultListModel;
-import javax.swing.JButton;
-import javax.swing.JFrame;
-import javax.swing.JLabel;
-import javax.swing.JList;
-import javax.swing.JPanel;
-import javax.swing.JTextField;
-import javax.swing.plaf.synth.ColorType;
-
-import org.w3c.dom.css.RGBColor;
-
-import de.hs_mannheim.informatik.bank.facade.Banksystem;
-
-public class GUI {
-
-	private Banksystem bs;
-
-	public GUI(Banksystem bs) {
-		this.bs = bs;
-		hauptmenü();
-
-	}
-
-	private void hauptmenü() {
-		JFrame frame = new JFrame();
-		frame.setSize(500, 500);
-		frame.setTitle(bs.getBankname());
-		frame.setLayout(null);
-		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
-
-		JButton getKunde = new JButton("Alle Konten");
-		getKunde.setBounds(100, 50, 300, 50);
-		getKunde.addActionListener(e -> {
-			JFrame allKontos = new JFrame();
-			JPanel alleKontenPanel = new JPanel();
-			allKontos.setTitle(bs.getBankname());
-			allKontos.setSize(500, 500);
-			allKontos.add(alleKontenPanel);
-			String[] konten = bs.getKontenliste();
-			DefaultListModel<String> model = new DefaultListModel<>();
-			if (konten.length > 0) {
-				JLabel labelU = new JLabel("Alle Konten");
-				labelU.setBounds(50, 10, 100, 50);
-				// alleKontenPanel.add(labelU);
-				for (String s : konten) {
-					model.addElement(s);
-				}
-				JList<String> kontenJ = new JList<>(konten);
-				kontenJ.setBackground(new Color(119, 215, 239));
-				// labelU.setBounds(50,50,300,200);
-				alleKontenPanel.add(kontenJ);
-			} else {
-
-			}
-			allKontos.setVisible(true);
-		});
-		frame.add(getKunde);
-
-		JButton kontoAnlegen = new JButton("Konto anlegen");
-		kontoAnlegen.setBounds(100, 120, 300, 50);
-		kontoAnlegen.addActionListener(e -> {
-
-			JFrame kontosAnlegen = new JFrame();
-			JPanel kontenAnlegenPanel = new JPanel();
-			kontosAnlegen.setTitle(bs.getBankname());
-			kontosAnlegen.setSize(500, 500);
-			kontosAnlegen.add(kontenAnlegenPanel);
-			kontenAnlegenPanel.setLayout(null);
-			
-			JLabel l_name = new JLabel("Name");
-			l_name.setBounds(50,50,100,50);
-			kontenAnlegenPanel.add(l_name);
-			
-			JTextField t_name = new JTextField(20);
-			t_name.setBounds(200,50,100,30);
-			kontenAnlegenPanel.add(t_name);
-			
-			JLabel l_konto = new JLabel("Konto");
-			l_konto.setBounds(50,150,100,50);
-			kontenAnlegenPanel.add(l_konto);
-			
-			JTextField t_konto = new JTextField(20);
-			t_konto.setBounds(200,150,100,30);
-			kontenAnlegenPanel.add(t_konto);
-			
-			JLabel l_kundenNr = new JLabel("Kunden Nummer");
-			l_kundenNr.setBounds(50,250,100,50);
-			kontenAnlegenPanel.add(l_kundenNr);
-			
-			JTextField t_kundenNr = new JTextField(20);
-			t_kundenNr.setBounds(200,250,100,30);
-			kontenAnlegenPanel.add(t_kundenNr);
-			
-			JButton kundeAnlegen = new JButton("Konto anlegen");
-			kundeAnlegen.setBounds(50,400,350,50);
-			kundeAnlegen.addActionListener(a -> {
-				try {
-					int kontonummer = bs.kontoAnlegen(t_name.getText(), Integer.parseInt(t_konto.getText()),
-							Integer.parseInt(t_kundenNr.getText()));
-					JLabel ko = new JLabel();
-					ko.setBounds(50,350,10,50);
-					ko.setText(Integer.toString(kontonummer));
-					kontenAnlegenPanel.add(ko);
-				} catch (NumberFormatException e1) {
-					// TODO Auto-generated catch block
-					e1.printStackTrace();
-				} catch (Exception e1) {
-					// TODO Auto-generated catch block
-					e1.printStackTrace();
-				}
-			});
-			kontenAnlegenPanel.add(kundeAnlegen);
-				
-			kontosAnlegen.setVisible(true);
-		});
-		frame.add(kontoAnlegen);
-
-		frame.setVisible(true);
-	}
-
-}

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

@@ -1,10 +1,7 @@
 package de.hs_mannheim.informatik.bank.ui;
 
-import java.util.Collection;
 import java.util.Scanner;
 
-import de.hs_mannheim.informatik.bank.domain.Kunde;
-import de.hs_mannheim.informatik.bank.domain.KundeNullException;
 import de.hs_mannheim.informatik.bank.facade.Banksystem;
 
 public class UI {
@@ -36,7 +33,6 @@ public class UI {
 			System.out.println("11 -> Aktie anlegen");
 			System.out.println("12 -> Alle Aktien anzeigen");
 			System.out.println("13 -> Aktien kaufen");
-			System.out.println("14 -> Kunden Konten anzeigen");
 			System.out.println("99 -> Beenden");
 			System.out.println();
 
@@ -85,9 +81,6 @@ public class UI {
 				case 13:
 					aktienKaufen();
 					break;
-				case 14:
-					kundenKonten();
-					break;
 				case 99:
 					break mainloop;
 				}
@@ -102,17 +95,6 @@ public class UI {
 
 	} // hauptmenü
 
-	private void kundenKonten() {
-		System.out.println("Geben sie ihre KundenID ein: ");
-		int k = returnUserInput();
-		String[] list = bs.getKundenKontos(k);
-		if(list.length >0) {
-			for(String s : list) {
-				System.out.println(s);
-			}
-		}
-	}
-
 	private void aktienKaufen() throws Exception {
 		System.out.println("Geben sie die Kontonummer ein");
 		int kontonummer = returnUserInput();
@@ -187,28 +169,17 @@ public class UI {
 	}
 
 	private void kontoAnlegen() throws Exception {
-		
 		System.out.println("Bitte den Namen des Kontoinhabers angeben: ");
 		String name = sc.nextLine();
 
 		System.out.println("Möchten Sie ein Sparkonto (1), Girokonto (2) oder ein Depot (3) anlegen?");
 		int auswahl = returnUserInput();
 
-		System.out.println("Geben sie ihre Kundennummer ein:");
+		System.out.println("Geben sie die Kundennummer ein(id)");
 		int Kundennummer = returnUserInput();
-		try {
+
 		int kontonummer = bs.kontoAnlegen(name, auswahl, Kundennummer);
 		System.out.println("Konto mit der Nummer " + kontonummer + " neu angelegt.");
-		}catch(KundeNullException e) {
-			System.err.print("Der Kunde konnte nicht gefunden werden!");
-		}
-			
-		
-
-		
-		
-		
-		
 	}
 
 	private void geldEinzahlen() throws Exception {