CompareTo soll Datum sortieren

[mkwin]

Hi,
ich versuche die compareTo() Methode zu überschreiben um Datums nach Kalenderwoche und nach Jahr zu sortieren.
Ich versuche mit daran schon seit längerem, nur irgendwo ist ein Fehler in meiner Logik.
Vielleicht könnt ihr mir ja weiter helfen...

@Override
	public int compareTo(Week week) {
		if (this.startDate.getWeekOfWeekyear() < week.getStartDate().getWeekOfWeekyear() && this.startDate.getYear() < week.getStartDate().getYear()){
			return -1;
		}else if (this.startDate.getWeekOfWeekyear() > week.getStartDate().getWeekOfWeekyear() && this.startDate.getYear() > week.getStartDate().getYear()){
			return 1;
		}
			return 0;
	}

getWeekOfWeekYear () gibt die Kalenderwoche zurück

Meine Ausgabe sieht so aus, wenn ich mit Daten teste.
Davor sagen ich natürlich Collections.sort(weeks)

26
2014
18
2014
5
2014
26
2014
16
2015
16
2013

 

[Joose]

Generell hast noch nicht genau gesagt was nicht funktioniert bzw. falsch läuft.
Welche compareTo Methode versuchst du zu überschreiben?
Was genau sind deine Anfangsdaten und was erwartest du dir wie es nachher ausschaut?

Ansonsten ein Fehler ist mir so schon aufgefallen:
Überprüfe zuerst ob das Jahr > oder < oder = ist, dann erst die Woche.

this.startDate.getWeekOfWeekyear() < week.getStartDate().getWeekOfWeekyear() && this.startDate.getYear() < week.getStartDate().getYear()

Wenn die Woche nun nicht kleiner ist, ist dieser Ausdruck und durch das "&&" der Ganze Ausdruck ==> false! Auch wenn das Jahr kleiner wäre

Hier arbeitet Java vielleicht anders als von dir erwartet. Es wertet die Ausdrücke der Reihe nach aus.
=> Ausdruck1 && Ausdruck2

Zuerst wird Ausdruck1 ausgewertet, ist dieser false wird Ausdruck2 gar nicht ausgewertet => da false && etwas anderes immer false ist!

 

[mkwin]

Ah alles klar, hab gerade auch den Fehler behoben, funzt nun, danke!