if - else Baum vereinfachen

[DaveIT]

Hallo,

es geht um Folgendes:

Ich habe ein Programm mit vier Grundelementen (Arbeitszeiten, Regiezeiten, Pausenzeiten, Fehlerzeiten) geschrieben. Jedes Element hat eine Startzeit. Mein Programm verfügt über ein Archiv in dem ich alle gespeicherten Werte auswählen kann.

Zum Auslesen hab ich mir ein eigenes JTextPane gebaut welches die Elemente nach der Zeit geordnet ausgeben soll. Hier ist dann auch mein Problem:

Für dieses ordnen brauche ich 200 Codezeilen if-else Verschachtelung. Zuerst prüfe ich ob noch Elemente vorhanden sind. Allein dadurch entstehen 16 Fälle.

if(gesA > stepA){
                if(gesR > stepR){
                    if(gesP > stepP){
                        if(gesF > stepF){

usw...

Danach ermittle ich den nächsten Wert der ausgegeben werden muss. Hier muss ich auf alle Elemente prüfen die noch vorhanden sind.

 if((arbeit.getArbeitszeiten().get(stepA).getStart().compareTo(arbeit.getRegiezeiten().get(stepR).getStart()) < 1)){
                                if((arbeit.getArbeitszeiten().get(stepA).getStart().compareTo(arbeit.getPausen().get(stepP).getStart()) < 1)){
                                    if((arbeit.getArbeitszeiten().get(stepA).getStart().compareTo(arbeit.getFehler().get(stepF).getStart()) < 1)){
                                        outA(arbeit.getArbeitszeiten().get(stepA));
                                        stepA++;

Nach der Ausgabe durchlaufe ich alles solange bis keine Elemente mehr vorhanden sind.


Diese Lösung funktioniert zwar, ist aber sehr aufwendig und unübersichtlich. Bis jetzt ist mir noch keine bessere Methode eingefallen. Bin für Anregungen und Tipps dankbar.

LG

 

[Camino]

In was für einer Art "Archiv" liegen die Daten vor? Datenbank, Textdatei, XML...? Wie sieht denn die Struktur der einzelnen Datensätze aus? Wie ist die Startzeit strukturiert? Wenn es eine Datenbank ist, würde ich das sortieren der Startzeiten von der DB übernehmen lassen. Wenn nicht, dann einlesen in eine einheitliche Struktur (Klasse) und diese dann im Programm sortieren.

Zuerst prüfe ich ob noch Elemente vorhanden sind.

Wozu das? Les doch einfach alle Daten ein, die vorhanden sind. Warum prüfst du vorher, ob noch welche da sind? Versteh ich nicht.

 

[DaveIT]

Die Daten liegen in einer XML Datei. Struktur wie folgt:

[XML]<..zeiten>
<...zeit>
<startzeit> "Wert" </startzeit>
...weitere Informationen
</...zeit>
</..zeiten>
[/XML]

Wozu das? Les doch einfach alle Daten ein, die vorhanden sind. Warum prüfst du vorher, ob noch welche da sind? Versteh ich nicht.

Weil ich z.B.:

arbeit.getArbeitszeiten().get(stepA).getStart()

nur aufrufen kann wenn ich einen gültigen Index habe. Sonst gibts eine Exception.

 

[HoaX]

Schreib dir eine Klasse die Comparator implementiert, bzw lasse deine Arbeitszeit-Klasse Comparable implementieren. Dort implementierst du dann den Vergleich für zwei Objekte.
Danach musst du nurnoch

List<Arbeitszeit> zeiten = arbeit.getArbeitszeiten();
Collections.sort(zeiten);

aufrufen und der Inhalt ist sortiert.

 

[Camino]

Weil ich z.B.:

arbeit.getArbeitszeiten().get(stepA).getStart()

nur aufrufen kann wenn ich einen gültigen Index habe. Sonst gibts eine Exception.

Dann hast du da wohl irgendwas falsch programmiert. Was für ein gültiger Index überhaupt??? Schreib deinen Code doch so um, dass du alle vorhandenen Datensätze einliest und dann sortierst. Ohne vorher zu überprüfen, ob die Datensätze auch vorhanden sind. Das siehst du ja, wenn du sie eingelesen hast.