Fix: changed for loops to lambda & code refactor
parent
b837bddaa5
commit
6a3fffdc33
|
@ -1,6 +1,7 @@
|
|||
package de.hs_mannheim.informatik.bank;
|
||||
|
||||
import de.hs_mannheim.informatik.bank.facade.Banksystem;
|
||||
import de.hs_mannheim.informatik.bank.gui.AuswahlmenüFrame;
|
||||
import de.hs_mannheim.informatik.bank.gui.HauptmenüFrame;
|
||||
|
||||
|
||||
|
@ -12,6 +13,8 @@ public class Main {
|
|||
|
||||
HauptmenüFrame hmf = new HauptmenüFrame(bs);
|
||||
|
||||
AuswahlmenüFrame amf = new AuswahlmenüFrame(bs);
|
||||
|
||||
}
|
||||
|
||||
}
|
|
@ -2,7 +2,6 @@ package de.hs_mannheim.informatik.bank.domain;
|
|||
|
||||
import java.io.Serializable;
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
|
||||
public class Konto implements Serializable, Comparable<Konto>{
|
||||
|
||||
|
@ -40,9 +39,13 @@ public class Konto implements Serializable, Comparable<Konto>{
|
|||
|
||||
public String[] getKontobewegungen() {
|
||||
|
||||
String[] bewegungen = kontobewegungen.stream().map(b -> b.toString()).toArray(String[]::new);
|
||||
String[] auflistung = new String[kontobewegungen.size()];
|
||||
|
||||
return bewegungen;
|
||||
kontobewegungen.stream().forEach(kb -> {
|
||||
auflistung[kontobewegungen.indexOf(kb)] = kb.toString();
|
||||
});
|
||||
|
||||
return auflistung;
|
||||
|
||||
}
|
||||
|
||||
|
|
|
@ -4,7 +4,8 @@ import java.io.IOException;
|
|||
import java.util.Collection;
|
||||
import java.util.HashMap;
|
||||
import java.util.List;
|
||||
import java.util.TreeSet;
|
||||
|
||||
|
||||
|
||||
import de.hs_mannheim.informatik.bank.domain.Bank;
|
||||
import de.hs_mannheim.informatik.bank.domain.Depot;
|
||||
|
@ -64,21 +65,13 @@ public class Banksystem {
|
|||
|
||||
Collection<Konto> konten = bank.getKundenKonto(currentKunde);
|
||||
|
||||
//TreeSet<Konto> kontenSet = new TreeSet<Konto>(konten);
|
||||
|
||||
List<Konto> kontenList = (List<Konto>) konten;
|
||||
|
||||
String[] liste = new String[konten.size()];
|
||||
|
||||
// kontenSet.stream().forEach(
|
||||
// konto -> {
|
||||
// liste[kontenSet.headSet(konto).size()] = konto.toString(); // treeset methode
|
||||
// }
|
||||
// );
|
||||
|
||||
kontenList.stream().forEach(
|
||||
konto -> {
|
||||
liste[kontenList.indexOf(konto)] = konto.toString(); // list methode
|
||||
liste[kontenList.indexOf(konto)] = konto.toString();
|
||||
}
|
||||
);
|
||||
|
||||
|
@ -200,6 +193,7 @@ public class Banksystem {
|
|||
|
||||
}
|
||||
|
||||
|
||||
public String getBankname() {
|
||||
return bank.getName();
|
||||
}
|
||||
|
|
|
@ -18,6 +18,7 @@ public class AuswahlmenüFrame implements ActionListener {
|
|||
private JButton button3;
|
||||
private JButton button4;
|
||||
|
||||
|
||||
private Banksystem bs;
|
||||
|
||||
public AuswahlmenüFrame(Banksystem bs) {
|
||||
|
@ -50,6 +51,7 @@ public class AuswahlmenüFrame implements ActionListener {
|
|||
button3.addActionListener(this);
|
||||
panel.add(button3);
|
||||
|
||||
|
||||
this.button4 = new JButton("Hauptmenü");
|
||||
button4.setBounds(40, 170, 300, 25);
|
||||
button4.addActionListener(this);
|
||||
|
@ -88,6 +90,7 @@ public class AuswahlmenüFrame implements ActionListener {
|
|||
|
||||
}
|
||||
|
||||
|
||||
else {
|
||||
|
||||
frame.dispose();
|
||||
|
|
|
@ -2,6 +2,7 @@ package de.hs_mannheim.informatik.bank.gui;
|
|||
|
||||
import java.awt.event.ActionEvent;
|
||||
import java.awt.event.ActionListener;
|
||||
import java.util.Arrays;
|
||||
|
||||
import javax.swing.JButton;
|
||||
import javax.swing.JFrame;
|
||||
|
@ -63,10 +64,14 @@ public class KontoListingFrame implements ActionListener {
|
|||
|
||||
String[] konten = bs.getKontenlisteForKunde();
|
||||
|
||||
for (String konto : konten) {
|
||||
output.append(konto + "\n");
|
||||
if (konten.length > 0) {
|
||||
|
||||
Arrays.stream(konten).forEach(output::append);
|
||||
|
||||
}
|
||||
|
||||
output.append("Keine Konten vorhanden.");
|
||||
|
||||
} else if (e.getSource() == button2) {
|
||||
|
||||
frame.dispose();
|
||||
|
|
Loading…
Reference in New Issue