2martens
/
dis
1
0
Fork 0
Code Issues 1 Pull Requests Projects Releases Wiki Activity

Improved coding 

- changed while to foreach loop where possible
- made methods private where possible
- removed unnecessary boxing

Signed-off-by: Jim Martens <github@2martens.de>
This commit is contained in:
Jim Martens 2017-05-01 17:25:59 +02:00
parent 2deebe0bff
commit b9d84e797b
10 changed files with 114 additions and 147 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
03/src/de/dis2013/Main.java
View File

@ -25,7 +25,7 @@ public class Main {
/** /**
* Zeigt das Hauptmenü * Zeigt das Hauptmenü
*/ */
public static void showMainMenu() { private static void showMainMenu() {
//Menüoptionen //Menüoptionen
final int MENU_MAKLER = 0; final int MENU_MAKLER = 0;
final int MENU_PERSON= 1; final int MENU_PERSON= 1;

176
03/src/de/dis2013/core/ImmoService.java
View File

@ -56,14 +56,12 @@ public class ImmoService {
* @return Makler mit der ID oder null * @return Makler mit der ID oder null
*/ */
public Makler getMaklerById(int id) { public Makler getMaklerById(int id) {
Iterator<Makler> it = makler.iterator();
for (Makler m : makler) {
while(it.hasNext()) { if (m.getId() == id) {
Makler m = it.next(); return m;
}
if(m.getId() == id) }
return m;
}
return null; return null;
} }
@ -74,14 +72,12 @@ public class ImmoService {
* @return Makler mit der ID oder null * @return Makler mit der ID oder null
*/ */
public Makler getMaklerByLogin(String login) { public Makler getMaklerByLogin(String login) {
Iterator<Makler> it = makler.iterator();
for (Makler m : makler) {
while(it.hasNext()) { if (m.getLogin().equals(login)) {
Makler m = it.next(); return m;
}
if(m.getLogin().equals(login)) }
return m;
}
return null; return null;
} }
@ -99,14 +95,12 @@ public class ImmoService {
* @return Person mit der ID oder null * @return Person mit der ID oder null
*/ */
public Person getPersonById(int id) { public Person getPersonById(int id) {
Iterator<Person> it = personen.iterator();
for (Person p : personen) {
while(it.hasNext()) { if (p.getId() == id) {
Person p = it.next(); return p;
}
if(p.getId() == id) }
return p;
}
return null; return null;
} }
@ -181,14 +175,12 @@ public class ImmoService {
*/ */
public Set<Haus> getAllHaeuserForMakler(Makler m) { public Set<Haus> getAllHaeuserForMakler(Makler m) {
Set<Haus> ret = new HashSet<Haus>(); Set<Haus> ret = new HashSet<Haus>();
Iterator<Haus> it = haeuser.iterator();
for (Haus h : haeuser) {
while(it.hasNext()) { if (h.getVerwalter().equals(m)) {
Haus h = it.next(); ret.add(h);
}
if(h.getVerwalter().equals(m)) }
ret.add(h);
}
return ret; return ret;
} }
@ -199,13 +191,11 @@ public class ImmoService {
* @return Das Haus oder null, falls nicht gefunden * @return Das Haus oder null, falls nicht gefunden
*/ */
public Haus getHausById(int id) { public Haus getHausById(int id) {
Iterator<Haus> it = haeuser.iterator();
while(it.hasNext()) { for (Haus h : haeuser) {
Haus h = it.next(); if (h.getId() == id) {
if(h.getId() == id)
return h; return h;
}
} }
return null; return null;
@ -234,14 +224,12 @@ public class ImmoService {
*/ */
public Set<Wohnung> getAllWohnungenForMakler(Makler m) { public Set<Wohnung> getAllWohnungenForMakler(Makler m) {
Set<Wohnung> ret = new HashSet<Wohnung>(); Set<Wohnung> ret = new HashSet<Wohnung>();
Iterator<Wohnung> it = wohnungen.iterator();
for (Wohnung w : wohnungen) {
while(it.hasNext()) { if (w.getVerwalter().equals(m)) {
Wohnung w = it.next(); ret.add(w);
}
if(w.getVerwalter().equals(m)) }
ret.add(w);
}
return ret; return ret;
} }
@ -252,14 +240,12 @@ public class ImmoService {
* @return Die Wohnung oder null, falls nicht gefunden * @return Die Wohnung oder null, falls nicht gefunden
*/ */
public Wohnung getWohnungById(int id) { public Wohnung getWohnungById(int id) {
Iterator<Wohnung> it = wohnungen.iterator();
for (Wohnung w : wohnungen) {
while(it.hasNext()) { if (w.getId() == id) {
Wohnung w = it.next(); return w;
}
if(w.getId() == id) }
return w;
}
return null; return null;
} }
@ -296,14 +282,12 @@ public class ImmoService {
*/ */
public Set<Mietvertrag> getAllMietvertraegeForMakler(Makler m) { public Set<Mietvertrag> getAllMietvertraegeForMakler(Makler m) {
Set<Mietvertrag> ret = new HashSet<Mietvertrag>(); Set<Mietvertrag> ret = new HashSet<Mietvertrag>();
Iterator<Mietvertrag> it = mietvertraege.iterator();
for (Mietvertrag v : mietvertraege) {
while(it.hasNext()) { if (v.getWohnung().getVerwalter().equals(m)) {
Mietvertrag v = it.next(); ret.add(v);
}
if(v.getWohnung().getVerwalter().equals(m)) }
ret.add(v);
}
return ret; return ret;
} }
@ -315,14 +299,12 @@ public class ImmoService {
*/ */
public Set<Kaufvertrag> getAllKaufvertraegeForMakler(Makler m) { public Set<Kaufvertrag> getAllKaufvertraegeForMakler(Makler m) {
Set<Kaufvertrag> ret = new HashSet<Kaufvertrag>(); Set<Kaufvertrag> ret = new HashSet<Kaufvertrag>();
Iterator<Kaufvertrag> it = kaufvertraege.iterator();
for (Kaufvertrag k : kaufvertraege) {
while(it.hasNext()) { if (k.getHaus().getVerwalter().equals(m)) {
Kaufvertrag k = it.next(); ret.add(k);
}
if(k.getHaus().getVerwalter().equals(m)) }
ret.add(k);
}
return ret; return ret;
} }
@ -333,14 +315,12 @@ public class ImmoService {
* @return Der Mietvertrag oder null, falls nicht gefunden * @return Der Mietvertrag oder null, falls nicht gefunden
*/ */
public Mietvertrag getMietvertragById(int id) { public Mietvertrag getMietvertragById(int id) {
Iterator<Mietvertrag> it = mietvertraege.iterator();
for (Mietvertrag m : mietvertraege) {
while(it.hasNext()) { if (m.getId() == id) {
Mietvertrag m = it.next(); return m;
}
if(m.getId() == id) }
return m;
}
return null; return null;
} }
@ -352,14 +332,12 @@ public class ImmoService {
*/ */
public Set<Mietvertrag> getMietvertragByVerwalter(Makler m) { public Set<Mietvertrag> getMietvertragByVerwalter(Makler m) {
Set<Mietvertrag> ret = new HashSet<Mietvertrag>(); Set<Mietvertrag> ret = new HashSet<Mietvertrag>();
Iterator<Mietvertrag> it = mietvertraege.iterator();
for (Mietvertrag mv : mietvertraege) {
while(it.hasNext()) { if (mv.getWohnung().getVerwalter().getId() == m.getId()) {
Mietvertrag mv = it.next(); ret.add(mv);
}
if(mv.getWohnung().getVerwalter().getId() == m.getId()) }
ret.add(mv);
}
return ret; return ret;
} }
@ -371,14 +349,12 @@ public class ImmoService {
*/ */
public Set<Kaufvertrag> getKaufvertragByVerwalter(Makler m) { public Set<Kaufvertrag> getKaufvertragByVerwalter(Makler m) {
Set<Kaufvertrag> ret = new HashSet<Kaufvertrag>(); Set<Kaufvertrag> ret = new HashSet<Kaufvertrag>();
Iterator<Kaufvertrag> it = kaufvertraege.iterator();
for (Kaufvertrag k : kaufvertraege) {
while(it.hasNext()) { if (k.getHaus().getVerwalter().getId() == m.getId()) {
Kaufvertrag k = it.next(); ret.add(k);
}
if(k.getHaus().getVerwalter().getId() == m.getId()) }
ret.add(k);
}
return ret; return ret;
} }
@ -389,14 +365,12 @@ public class ImmoService {
* @return Der Kaufvertrag oder null, falls nicht gefunden * @return Der Kaufvertrag oder null, falls nicht gefunden
*/ */
public Kaufvertrag getKaufvertragById(int id) { public Kaufvertrag getKaufvertragById(int id) {
Iterator<Kaufvertrag> it = kaufvertraege.iterator();
for (Kaufvertrag k : kaufvertraege) {
while(it.hasNext()) { if (k.getId() == id) {
Kaufvertrag k = it.next(); return k;
}
if(k.getId() == id) }
return k;
}
return null; return null;
} }

6
03/src/de/dis2013/editor/MaklerEditor.java
View File

@ -58,7 +58,7 @@ public class MaklerEditor {
* Legt einen neuen Makler an, nachdem der Benutzer * Legt einen neuen Makler an, nachdem der Benutzer
* die entprechenden Daten eingegeben hat. * die entprechenden Daten eingegeben hat.
*/ */
public void newMakler() { private void newMakler() {
Makler m = new Makler(); Makler m = new Makler();
m.setName(FormUtil.readString("Name")); m.setName(FormUtil.readString("Name"));
@ -73,7 +73,7 @@ public class MaklerEditor {
/** /**
* Berarbeitet einen Makler, nachdem der Benutzer ihn ausgewählt hat * Berarbeitet einen Makler, nachdem der Benutzer ihn ausgewählt hat
*/ */
public void editMakler() { private void editMakler() {
//Menü zum selektieren des Maklers //Menü zum selektieren des Maklers
Menu maklerSelectionMenu = new MaklerSelectionMenu("Makler editieren", service.getAllMakler()); Menu maklerSelectionMenu = new MaklerSelectionMenu("Makler editieren", service.getAllMakler());
int id = maklerSelectionMenu.show(); int id = maklerSelectionMenu.show();
@ -108,7 +108,7 @@ public class MaklerEditor {
* Löscht einen Makler, nachdem der Benutzer * Löscht einen Makler, nachdem der Benutzer
* ihn ausgewählt hat. * ihn ausgewählt hat.
*/ */
public void deleteMakler() { private void deleteMakler() {
//Menü zum selektieren des Maklers //Menü zum selektieren des Maklers
Menu maklerSelectionMenu = new MaklerSelectionMenu("Makler löschen", service.getAllMakler()); Menu maklerSelectionMenu = new MaklerSelectionMenu("Makler löschen", service.getAllMakler());
int id = maklerSelectionMenu.show(); int id = maklerSelectionMenu.show();

6
03/src/de/dis2013/editor/PersonEditor.java
View File

@ -58,7 +58,7 @@ public class PersonEditor {
* Legt eine neue Person an, nachdem der Benutzer * Legt eine neue Person an, nachdem der Benutzer
* die entprechenden Daten eingegeben hat. * die entprechenden Daten eingegeben hat.
*/ */
public void newPerson() { private void newPerson() {
Person p = new Person(); Person p = new Person();
p.setVorname(FormUtil.readString("Vorname")); p.setVorname(FormUtil.readString("Vorname"));
@ -72,7 +72,7 @@ public class PersonEditor {
/** /**
* Editiert eine Person, nachdem der Benutzer sie ausgewählt hat * Editiert eine Person, nachdem der Benutzer sie ausgewählt hat
*/ */
public void editPerson() { private void editPerson() {
//Personenauswahlmenü //Personenauswahlmenü
Menu personSelectionMenu = new PersonSelectionMenu("Person bearbeiten", service.getAllPersons()); Menu personSelectionMenu = new PersonSelectionMenu("Person bearbeiten", service.getAllPersons());
int id = personSelectionMenu.show(); int id = personSelectionMenu.show();
@ -102,7 +102,7 @@ public class PersonEditor {
* Löscht eine Person, nachdem der Benutzer * Löscht eine Person, nachdem der Benutzer
* die entprechende ID eingegeben hat. * die entprechende ID eingegeben hat.
*/ */
public void deletePerson() { private void deletePerson() {
//Auswahl der Person //Auswahl der Person
Menu personSelectionMenu = new PersonSelectionMenu("Person bearbeiten", service.getAllPersons()); Menu personSelectionMenu = new PersonSelectionMenu("Person bearbeiten", service.getAllPersons());
int id = personSelectionMenu.show(); int id = personSelectionMenu.show();

43
03/src/de/dis2013/editor/VertragsEditor.java
View File

@ -70,19 +70,22 @@ public class VertragsEditor {
} }
} }
public void zeigeVertraege() { private void zeigeVertraege() {
//Mietverträge anzeigen //Mietverträge anzeigen
System.out.println("Mietverträge\n-----------------"); System.out.println("Mietverträge\n-----------------");
Set<Mietvertrag> mvs = service.getAllMietvertraegeForMakler(verwalter); Set<Mietvertrag> mvs = service.getAllMietvertraegeForMakler(verwalter);
Iterator<Mietvertrag> itmv = mvs.iterator(); for (Mietvertrag mv : mvs) {
while(itmv.hasNext()) { System.out.println("Mietvertrag " + mv.getVertragsnummer() + "\n" +
Mietvertrag mv = itmv.next(); "\tGeschlossen am " + Helper.dateToString(mv.getDatum()) + " in " + mv.getOrt() + "\n" +
System.out.println("Mietvertrag "+mv.getVertragsnummer()+"\n"+ "\tMieter: " + mv.getVertragspartner().getVorname() + " " +
"\tGeschlossen am "+Helper.dateToString(mv.getDatum())+" in "+mv.getOrt()+"\n"+ mv.getVertragspartner().getNachname() + "\n" +
"\tMieter: "+mv.getVertragspartner().getVorname()+" "+mv.getVertragspartner().getNachname()+"\n"+ "\tWohnung: " + mv.getWohnung().getStrasse() + " " +
"\tWohnung: "+mv.getWohnung().getStrasse()+" "+mv.getWohnung().getHausnummer()+", "+mv.getWohnung().getPlz()+" "+mv.getWohnung().getOrt()+"\n"+ mv.getWohnung().getHausnummer() + ", " + mv.getWohnung().getPlz() + " " +
"\tMietbeginn: "+Helper.dateToString(mv.getMietbeginn())+", Dauer: "+mv.getDauer()+" Monate\n"+ mv.getWohnung().getOrt() + "\n" +
"\tMietpreis: "+mv.getWohnung().getMietpreis()+" Euro, Nebenkosten: "+mv.getNebenkosten()+" Euro\n"); "\tMietbeginn: " + Helper.dateToString(mv.getMietbeginn()) + ", Dauer: " +
mv.getDauer() + " Monate\n" +
"\tMietpreis: " + mv.getWohnung().getMietpreis() + " Euro, Nebenkosten: " +
mv.getNebenkosten() + " Euro\n");
} }
System.out.println(""); System.out.println("");
@ -90,15 +93,15 @@ public class VertragsEditor {
//Kaufverträge anzeigen //Kaufverträge anzeigen
System.out.println("Kaufverträge\n-----------------"); System.out.println("Kaufverträge\n-----------------");
Set<Kaufvertrag> kvs = service.getAllKaufvertraegeForMakler(verwalter); Set<Kaufvertrag> kvs = service.getAllKaufvertraegeForMakler(verwalter);
Iterator<Kaufvertrag> itkv = kvs.iterator(); for (Kaufvertrag kv : kvs) {
while(itkv.hasNext()) { System.out.println("Kaufvertrag " + kv.getVertragsnummer() + "\n" +
Kaufvertrag kv = itkv.next(); "\tGeschlossen am " + Helper.dateToString(kv.getDatum()) + " in " + kv.getOrt() + "\n" +
System.out.println("Kaufvertrag "+kv.getVertragsnummer()+"\n"+ "\tMieter: " + kv.getVertragspartner().getVorname() + " " +
"\tGeschlossen am "+Helper.dateToString(kv.getDatum())+" in "+kv.getOrt()+"\n"+ kv.getVertragspartner().getNachname() + "\n" +
"\tMieter: "+kv.getVertragspartner().getVorname()+" "+kv.getVertragspartner().getNachname()+"\n"+ "\tHaus: " + kv.getHaus().getStrasse() + " " + kv.getHaus().getHausnummer() +
"\tHaus: "+kv.getHaus().getStrasse()+" "+kv.getHaus().getHausnummer()+", "+kv.getHaus().getPlz()+" "+kv.getHaus().getOrt()+"\n"+ ", " + kv.getHaus().getPlz() + " " + kv.getHaus().getOrt() + "\n" +
"\tKaufpreis: "+kv.getHaus().getKaufpreis()+" Euro\n"+ "\tKaufpreis: " + kv.getHaus().getKaufpreis() + " Euro\n" +
"\tRaten: "+kv.getAnzahlRaten()+", Ratenzins: "+kv.getRatenzins()+"%\n"); "\tRaten: " + kv.getAnzahlRaten() + ", Ratenzins: " + kv.getRatenzins() + "%\n");
} }
} }
@ -106,7 +109,7 @@ public class VertragsEditor {
/** /**
* Menü zum anlegen eines neuen Mietvertrags * Menü zum anlegen eines neuen Mietvertrags
*/ */
public void newMietvertrag() { private void newMietvertrag() {
//Alle Wohnungen des Maklers finden //Alle Wohnungen des Maklers finden
Set<Wohnung> wohnungen = service.getAllWohnungenForMakler(verwalter); Set<Wohnung> wohnungen = service.getAllWohnungenForMakler(verwalter);

6
03/src/de/dis2013/menu/AppartmentSelectionMenu.java
View File

@ -14,10 +14,8 @@ public class AppartmentSelectionMenu extends Menu {
public AppartmentSelectionMenu(String title, Set<Wohnung> wohnungen) { public AppartmentSelectionMenu(String title, Set<Wohnung> wohnungen) {
super(title); super(title);
Iterator<Wohnung> it = wohnungen.iterator(); for (Wohnung w : wohnungen) {
while(it.hasNext()) { addEntry(w.getStrasse() + " " + w.getHausnummer() + ", " + w.getPlz() + " " + w.getOrt(), w.getId());
Wohnung w = it.next();
addEntry(w.getStrasse()+" "+w.getHausnummer()+", "+w.getPlz()+" "+w.getOrt(), w.getId());
} }
addEntry("Zurück", BACK); addEntry("Zurück", BACK);
} }

6
03/src/de/dis2013/menu/HouseSelectionMenu.java
View File

@ -14,10 +14,8 @@ public class HouseSelectionMenu extends Menu {
public HouseSelectionMenu(String title, Set<Haus> haeuser) { public HouseSelectionMenu(String title, Set<Haus> haeuser) {
super(title); super(title);
Iterator<Haus> it = haeuser.iterator(); for (Haus h : haeuser) {
while(it.hasNext()) { addEntry(h.getStrasse() + " " + h.getHausnummer() + ", " + h.getPlz() + " " + h.getOrt(), h.getId());
Haus h = it.next();
addEntry(h.getStrasse()+" "+h.getHausnummer()+", "+h.getPlz()+" "+h.getOrt(), h.getId());
} }
addEntry("Zurück", BACK); addEntry("Zurück", BACK);
} }

4
03/src/de/dis2013/menu/MaklerSelectionMenu.java
View File

@ -14,9 +14,7 @@ public class MaklerSelectionMenu extends Menu {
public MaklerSelectionMenu(String title, Set<Makler> makler) { public MaklerSelectionMenu(String title, Set<Makler> makler) {
super(title); super(title);
Iterator<Makler> it = makler.iterator(); for (Makler m : makler) {
while(it.hasNext()) {
Makler m = it.next();
addEntry(m.getName(), m.getId()); addEntry(m.getName(), m.getId());
} }
addEntry("Zurück", BACK); addEntry("Zurück", BACK);

6
03/src/de/dis2013/menu/Menu.java
View File

@ -50,7 +50,7 @@ public class Menu {
*/ */
public void addEntry(String label, int returnValue) { public void addEntry(String label, int returnValue) {
this.labels.add(label); this.labels.add(label);
this.returnValues.add(new Integer(returnValue)); this.returnValues.add(returnValue);
} }
/** /**
@ -73,9 +73,7 @@ public class Menu {
System.out.print("-- "); System.out.print("-- ");
try { try {
selection = Integer.parseInt(stdin.readLine()); selection = Integer.parseInt(stdin.readLine());
} catch (NumberFormatException e) { } catch (NumberFormatException | IOException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace(); e.printStackTrace();
} }

6
03/src/de/dis2013/menu/PersonSelectionMenu.java
View File

@ -14,10 +14,8 @@ public class PersonSelectionMenu extends Menu {
public PersonSelectionMenu(String title, Set<Person> personen) { public PersonSelectionMenu(String title, Set<Person> personen) {
super(title); super(title);
Iterator<Person> it = personen.iterator(); for (Person p : personen) {
while(it.hasNext()) { addEntry(p.getVorname() + " " + p.getNachname(), p.getId());
Person p = it.next();
addEntry(p.getVorname()+" "+p.getNachname(), p.getId());
} }
addEntry("Zurück", BACK); addEntry("Zurück", BACK);
} }