From b9d84e797b5659065c025b2a9f298d7a2d28c2fe Mon Sep 17 00:00:00 2001 From: Jim Martens Date: Mon, 1 May 2017 17:25:59 +0200 Subject: [PATCH] Improved coding - changed while to foreach loop where possible - made methods private where possible - removed unnecessary boxing Signed-off-by: Jim Martens --- 03/src/de/dis2013/Main.java | 2 +- 03/src/de/dis2013/core/ImmoService.java | 176 ++++++++---------- 03/src/de/dis2013/editor/MaklerEditor.java | 6 +- 03/src/de/dis2013/editor/PersonEditor.java | 6 +- 03/src/de/dis2013/editor/VertragsEditor.java | 43 +++-- .../dis2013/menu/AppartmentSelectionMenu.java | 6 +- .../de/dis2013/menu/HouseSelectionMenu.java | 6 +- .../de/dis2013/menu/MaklerSelectionMenu.java | 4 +- 03/src/de/dis2013/menu/Menu.java | 6 +- .../de/dis2013/menu/PersonSelectionMenu.java | 6 +- 10 files changed, 114 insertions(+), 147 deletions(-) diff --git a/03/src/de/dis2013/Main.java b/03/src/de/dis2013/Main.java index 0768d1e..1728e8f 100644 --- a/03/src/de/dis2013/Main.java +++ b/03/src/de/dis2013/Main.java @@ -25,7 +25,7 @@ public class Main { /** * Zeigt das Hauptmenü */ - public static void showMainMenu() { + private static void showMainMenu() { //Menüoptionen final int MENU_MAKLER = 0; final int MENU_PERSON= 1; diff --git a/03/src/de/dis2013/core/ImmoService.java b/03/src/de/dis2013/core/ImmoService.java index e674dcf..7f69d64 100644 --- a/03/src/de/dis2013/core/ImmoService.java +++ b/03/src/de/dis2013/core/ImmoService.java @@ -56,14 +56,12 @@ public class ImmoService { * @return Makler mit der ID oder null */ public Makler getMaklerById(int id) { - Iterator it = makler.iterator(); - - while(it.hasNext()) { - Makler m = it.next(); - - if(m.getId() == id) - return m; - } + + for (Makler m : makler) { + if (m.getId() == id) { + return m; + } + } return null; } @@ -74,14 +72,12 @@ public class ImmoService { * @return Makler mit der ID oder null */ public Makler getMaklerByLogin(String login) { - Iterator it = makler.iterator(); - - while(it.hasNext()) { - Makler m = it.next(); - - if(m.getLogin().equals(login)) - return m; - } + + for (Makler m : makler) { + if (m.getLogin().equals(login)) { + return m; + } + } return null; } @@ -99,14 +95,12 @@ public class ImmoService { * @return Person mit der ID oder null */ public Person getPersonById(int id) { - Iterator it = personen.iterator(); - - while(it.hasNext()) { - Person p = it.next(); - - if(p.getId() == id) - return p; - } + + for (Person p : personen) { + if (p.getId() == id) { + return p; + } + } return null; } @@ -181,14 +175,12 @@ public class ImmoService { */ public Set getAllHaeuserForMakler(Makler m) { Set ret = new HashSet(); - Iterator it = haeuser.iterator(); - - while(it.hasNext()) { - Haus h = it.next(); - - if(h.getVerwalter().equals(m)) - ret.add(h); - } + + for (Haus h : haeuser) { + if (h.getVerwalter().equals(m)) { + ret.add(h); + } + } return ret; } @@ -199,13 +191,11 @@ public class ImmoService { * @return Das Haus oder null, falls nicht gefunden */ public Haus getHausById(int id) { - Iterator it = haeuser.iterator(); - while(it.hasNext()) { - Haus h = it.next(); - - if(h.getId() == id) + for (Haus h : haeuser) { + if (h.getId() == id) { return h; + } } return null; @@ -234,14 +224,12 @@ public class ImmoService { */ public Set getAllWohnungenForMakler(Makler m) { Set ret = new HashSet(); - Iterator it = wohnungen.iterator(); - - while(it.hasNext()) { - Wohnung w = it.next(); - - if(w.getVerwalter().equals(m)) - ret.add(w); - } + + for (Wohnung w : wohnungen) { + if (w.getVerwalter().equals(m)) { + ret.add(w); + } + } return ret; } @@ -252,14 +240,12 @@ public class ImmoService { * @return Die Wohnung oder null, falls nicht gefunden */ public Wohnung getWohnungById(int id) { - Iterator it = wohnungen.iterator(); - - while(it.hasNext()) { - Wohnung w = it.next(); - - if(w.getId() == id) - return w; - } + + for (Wohnung w : wohnungen) { + if (w.getId() == id) { + return w; + } + } return null; } @@ -296,14 +282,12 @@ public class ImmoService { */ public Set getAllMietvertraegeForMakler(Makler m) { Set ret = new HashSet(); - Iterator it = mietvertraege.iterator(); - - while(it.hasNext()) { - Mietvertrag v = it.next(); - - if(v.getWohnung().getVerwalter().equals(m)) - ret.add(v); - } + + for (Mietvertrag v : mietvertraege) { + if (v.getWohnung().getVerwalter().equals(m)) { + ret.add(v); + } + } return ret; } @@ -315,14 +299,12 @@ public class ImmoService { */ public Set getAllKaufvertraegeForMakler(Makler m) { Set ret = new HashSet(); - Iterator it = kaufvertraege.iterator(); - - while(it.hasNext()) { - Kaufvertrag k = it.next(); - - if(k.getHaus().getVerwalter().equals(m)) - ret.add(k); - } + + for (Kaufvertrag k : kaufvertraege) { + if (k.getHaus().getVerwalter().equals(m)) { + ret.add(k); + } + } return ret; } @@ -333,14 +315,12 @@ public class ImmoService { * @return Der Mietvertrag oder null, falls nicht gefunden */ public Mietvertrag getMietvertragById(int id) { - Iterator it = mietvertraege.iterator(); - - while(it.hasNext()) { - Mietvertrag m = it.next(); - - if(m.getId() == id) - return m; - } + + for (Mietvertrag m : mietvertraege) { + if (m.getId() == id) { + return m; + } + } return null; } @@ -352,14 +332,12 @@ public class ImmoService { */ public Set getMietvertragByVerwalter(Makler m) { Set ret = new HashSet(); - Iterator it = mietvertraege.iterator(); - - while(it.hasNext()) { - Mietvertrag mv = it.next(); - - if(mv.getWohnung().getVerwalter().getId() == m.getId()) - ret.add(mv); - } + + for (Mietvertrag mv : mietvertraege) { + if (mv.getWohnung().getVerwalter().getId() == m.getId()) { + ret.add(mv); + } + } return ret; } @@ -371,14 +349,12 @@ public class ImmoService { */ public Set getKaufvertragByVerwalter(Makler m) { Set ret = new HashSet(); - Iterator it = kaufvertraege.iterator(); - - while(it.hasNext()) { - Kaufvertrag k = it.next(); - - if(k.getHaus().getVerwalter().getId() == m.getId()) - ret.add(k); - } + + for (Kaufvertrag k : kaufvertraege) { + if (k.getHaus().getVerwalter().getId() == m.getId()) { + ret.add(k); + } + } return ret; } @@ -389,14 +365,12 @@ public class ImmoService { * @return Der Kaufvertrag oder null, falls nicht gefunden */ public Kaufvertrag getKaufvertragById(int id) { - Iterator it = kaufvertraege.iterator(); - - while(it.hasNext()) { - Kaufvertrag k = it.next(); - - if(k.getId() == id) - return k; - } + + for (Kaufvertrag k : kaufvertraege) { + if (k.getId() == id) { + return k; + } + } return null; } diff --git a/03/src/de/dis2013/editor/MaklerEditor.java b/03/src/de/dis2013/editor/MaklerEditor.java index 2b52719..235245b 100644 --- a/03/src/de/dis2013/editor/MaklerEditor.java +++ b/03/src/de/dis2013/editor/MaklerEditor.java @@ -58,7 +58,7 @@ public class MaklerEditor { * Legt einen neuen Makler an, nachdem der Benutzer * die entprechenden Daten eingegeben hat. */ - public void newMakler() { + private void newMakler() { Makler m = new Makler(); m.setName(FormUtil.readString("Name")); @@ -73,7 +73,7 @@ public class MaklerEditor { /** * Berarbeitet einen Makler, nachdem der Benutzer ihn ausgewählt hat */ - public void editMakler() { + private void editMakler() { //Menü zum selektieren des Maklers Menu maklerSelectionMenu = new MaklerSelectionMenu("Makler editieren", service.getAllMakler()); int id = maklerSelectionMenu.show(); @@ -108,7 +108,7 @@ public class MaklerEditor { * Löscht einen Makler, nachdem der Benutzer * ihn ausgewählt hat. */ - public void deleteMakler() { + private void deleteMakler() { //Menü zum selektieren des Maklers Menu maklerSelectionMenu = new MaklerSelectionMenu("Makler löschen", service.getAllMakler()); int id = maklerSelectionMenu.show(); diff --git a/03/src/de/dis2013/editor/PersonEditor.java b/03/src/de/dis2013/editor/PersonEditor.java index 4b17b11..108b307 100644 --- a/03/src/de/dis2013/editor/PersonEditor.java +++ b/03/src/de/dis2013/editor/PersonEditor.java @@ -58,7 +58,7 @@ public class PersonEditor { * Legt eine neue Person an, nachdem der Benutzer * die entprechenden Daten eingegeben hat. */ - public void newPerson() { + private void newPerson() { Person p = new Person(); p.setVorname(FormUtil.readString("Vorname")); @@ -72,7 +72,7 @@ public class PersonEditor { /** * Editiert eine Person, nachdem der Benutzer sie ausgewählt hat */ - public void editPerson() { + private void editPerson() { //Personenauswahlmenü Menu personSelectionMenu = new PersonSelectionMenu("Person bearbeiten", service.getAllPersons()); int id = personSelectionMenu.show(); @@ -102,7 +102,7 @@ public class PersonEditor { * Löscht eine Person, nachdem der Benutzer * die entprechende ID eingegeben hat. */ - public void deletePerson() { + private void deletePerson() { //Auswahl der Person Menu personSelectionMenu = new PersonSelectionMenu("Person bearbeiten", service.getAllPersons()); int id = personSelectionMenu.show(); diff --git a/03/src/de/dis2013/editor/VertragsEditor.java b/03/src/de/dis2013/editor/VertragsEditor.java index 0d305e7..aecf3a8 100644 --- a/03/src/de/dis2013/editor/VertragsEditor.java +++ b/03/src/de/dis2013/editor/VertragsEditor.java @@ -70,19 +70,22 @@ public class VertragsEditor { } } - public void zeigeVertraege() { + private void zeigeVertraege() { //Mietverträge anzeigen System.out.println("Mietverträge\n-----------------"); Set mvs = service.getAllMietvertraegeForMakler(verwalter); - Iterator itmv = mvs.iterator(); - while(itmv.hasNext()) { - Mietvertrag mv = itmv.next(); - System.out.println("Mietvertrag "+mv.getVertragsnummer()+"\n"+ - "\tGeschlossen am "+Helper.dateToString(mv.getDatum())+" in "+mv.getOrt()+"\n"+ - "\tMieter: "+mv.getVertragspartner().getVorname()+" "+mv.getVertragspartner().getNachname()+"\n"+ - "\tWohnung: "+mv.getWohnung().getStrasse()+" "+mv.getWohnung().getHausnummer()+", "+mv.getWohnung().getPlz()+" "+mv.getWohnung().getOrt()+"\n"+ - "\tMietbeginn: "+Helper.dateToString(mv.getMietbeginn())+", Dauer: "+mv.getDauer()+" Monate\n"+ - "\tMietpreis: "+mv.getWohnung().getMietpreis()+" Euro, Nebenkosten: "+mv.getNebenkosten()+" Euro\n"); + for (Mietvertrag mv : mvs) { + System.out.println("Mietvertrag " + mv.getVertragsnummer() + "\n" + + "\tGeschlossen am " + Helper.dateToString(mv.getDatum()) + " in " + mv.getOrt() + "\n" + + "\tMieter: " + mv.getVertragspartner().getVorname() + " " + + mv.getVertragspartner().getNachname() + "\n" + + "\tWohnung: " + mv.getWohnung().getStrasse() + " " + + mv.getWohnung().getHausnummer() + ", " + mv.getWohnung().getPlz() + " " + + mv.getWohnung().getOrt() + "\n" + + "\tMietbeginn: " + Helper.dateToString(mv.getMietbeginn()) + ", Dauer: " + + mv.getDauer() + " Monate\n" + + "\tMietpreis: " + mv.getWohnung().getMietpreis() + " Euro, Nebenkosten: " + + mv.getNebenkosten() + " Euro\n"); } System.out.println(""); @@ -90,15 +93,15 @@ public class VertragsEditor { //Kaufverträge anzeigen System.out.println("Kaufverträge\n-----------------"); Set kvs = service.getAllKaufvertraegeForMakler(verwalter); - Iterator itkv = kvs.iterator(); - while(itkv.hasNext()) { - Kaufvertrag kv = itkv.next(); - System.out.println("Kaufvertrag "+kv.getVertragsnummer()+"\n"+ - "\tGeschlossen am "+Helper.dateToString(kv.getDatum())+" in "+kv.getOrt()+"\n"+ - "\tMieter: "+kv.getVertragspartner().getVorname()+" "+kv.getVertragspartner().getNachname()+"\n"+ - "\tHaus: "+kv.getHaus().getStrasse()+" "+kv.getHaus().getHausnummer()+", "+kv.getHaus().getPlz()+" "+kv.getHaus().getOrt()+"\n"+ - "\tKaufpreis: "+kv.getHaus().getKaufpreis()+" Euro\n"+ - "\tRaten: "+kv.getAnzahlRaten()+", Ratenzins: "+kv.getRatenzins()+"%\n"); + for (Kaufvertrag kv : kvs) { + System.out.println("Kaufvertrag " + kv.getVertragsnummer() + "\n" + + "\tGeschlossen am " + Helper.dateToString(kv.getDatum()) + " in " + kv.getOrt() + "\n" + + "\tMieter: " + kv.getVertragspartner().getVorname() + " " + + kv.getVertragspartner().getNachname() + "\n" + + "\tHaus: " + kv.getHaus().getStrasse() + " " + kv.getHaus().getHausnummer() + + ", " + kv.getHaus().getPlz() + " " + kv.getHaus().getOrt() + "\n" + + "\tKaufpreis: " + kv.getHaus().getKaufpreis() + " Euro\n" + + "\tRaten: " + kv.getAnzahlRaten() + ", Ratenzins: " + kv.getRatenzins() + "%\n"); } } @@ -106,7 +109,7 @@ public class VertragsEditor { /** * Menü zum anlegen eines neuen Mietvertrags */ - public void newMietvertrag() { + private void newMietvertrag() { //Alle Wohnungen des Maklers finden Set wohnungen = service.getAllWohnungenForMakler(verwalter); diff --git a/03/src/de/dis2013/menu/AppartmentSelectionMenu.java b/03/src/de/dis2013/menu/AppartmentSelectionMenu.java index aad950d..4d71518 100644 --- a/03/src/de/dis2013/menu/AppartmentSelectionMenu.java +++ b/03/src/de/dis2013/menu/AppartmentSelectionMenu.java @@ -14,10 +14,8 @@ public class AppartmentSelectionMenu extends Menu { public AppartmentSelectionMenu(String title, Set wohnungen) { super(title); - Iterator it = wohnungen.iterator(); - while(it.hasNext()) { - Wohnung w = it.next(); - addEntry(w.getStrasse()+" "+w.getHausnummer()+", "+w.getPlz()+" "+w.getOrt(), w.getId()); + for (Wohnung w : wohnungen) { + addEntry(w.getStrasse() + " " + w.getHausnummer() + ", " + w.getPlz() + " " + w.getOrt(), w.getId()); } addEntry("Zurück", BACK); } diff --git a/03/src/de/dis2013/menu/HouseSelectionMenu.java b/03/src/de/dis2013/menu/HouseSelectionMenu.java index 611498f..3ba4971 100644 --- a/03/src/de/dis2013/menu/HouseSelectionMenu.java +++ b/03/src/de/dis2013/menu/HouseSelectionMenu.java @@ -14,10 +14,8 @@ public class HouseSelectionMenu extends Menu { public HouseSelectionMenu(String title, Set haeuser) { super(title); - Iterator it = haeuser.iterator(); - while(it.hasNext()) { - Haus h = it.next(); - addEntry(h.getStrasse()+" "+h.getHausnummer()+", "+h.getPlz()+" "+h.getOrt(), h.getId()); + for (Haus h : haeuser) { + addEntry(h.getStrasse() + " " + h.getHausnummer() + ", " + h.getPlz() + " " + h.getOrt(), h.getId()); } addEntry("Zurück", BACK); } diff --git a/03/src/de/dis2013/menu/MaklerSelectionMenu.java b/03/src/de/dis2013/menu/MaklerSelectionMenu.java index e39d303..6428903 100644 --- a/03/src/de/dis2013/menu/MaklerSelectionMenu.java +++ b/03/src/de/dis2013/menu/MaklerSelectionMenu.java @@ -14,9 +14,7 @@ public class MaklerSelectionMenu extends Menu { public MaklerSelectionMenu(String title, Set makler) { super(title); - Iterator it = makler.iterator(); - while(it.hasNext()) { - Makler m = it.next(); + for (Makler m : makler) { addEntry(m.getName(), m.getId()); } addEntry("Zurück", BACK); diff --git a/03/src/de/dis2013/menu/Menu.java b/03/src/de/dis2013/menu/Menu.java index def21c9..c7b22c2 100644 --- a/03/src/de/dis2013/menu/Menu.java +++ b/03/src/de/dis2013/menu/Menu.java @@ -50,7 +50,7 @@ public class Menu { */ public void addEntry(String label, int returnValue) { this.labels.add(label); - this.returnValues.add(new Integer(returnValue)); + this.returnValues.add(returnValue); } /** @@ -73,9 +73,7 @@ public class Menu { System.out.print("-- "); try { selection = Integer.parseInt(stdin.readLine()); - } catch (NumberFormatException e) { - e.printStackTrace(); - } catch (IOException e) { + } catch (NumberFormatException | IOException e) { e.printStackTrace(); } diff --git a/03/src/de/dis2013/menu/PersonSelectionMenu.java b/03/src/de/dis2013/menu/PersonSelectionMenu.java index d477393..1ecc336 100644 --- a/03/src/de/dis2013/menu/PersonSelectionMenu.java +++ b/03/src/de/dis2013/menu/PersonSelectionMenu.java @@ -14,10 +14,8 @@ public class PersonSelectionMenu extends Menu { public PersonSelectionMenu(String title, Set personen) { super(title); - Iterator it = personen.iterator(); - while(it.hasNext()) { - Person p = it.next(); - addEntry(p.getVorname()+" "+p.getNachname(), p.getId()); + for (Person p : personen) { + addEntry(p.getVorname() + " " + p.getNachname(), p.getId()); } addEntry("Zurück", BACK); }